Skip to:
Content

Opened 3 years ago

Closed 2 years ago

#2938 closed defect (fixed)

Append `&view=all` to trashed, pending, and spam statuses topic and reply links

Reported by: netweb Owned by: johnjamesjacoby
Milestone: 2.6 Priority: high
Severity: major Version:
Component: Component - Any/All Keywords: needs-patch
Cc:

Description

With the introduction of "pending" topics and replies when attempting to navigate to these posts from the back end by default the topic or reply is not viable, they need &view=all appened to the "View" row action link.

A Pending Topic as seen from the topics back end

  • Current "View" row action URL: http://example.com/?post_type=topic&p=12
  • This should have &view=all appended, http://example.com/?post_type=topic&p=12&view=all

A Pending Reply as seen from the replies back end

  • Current "View" row action URL: http://example.com/forums/topic/2934/#post-28
  • This should have &view=all appended, http://example.com/forums/topic/2934/#post-28&view=all

In the image on the left is without &view=all and on the right includes &view=all
https://cldup.com/LumdFbwdeO.png

Similar behaviour is seen when the topic or reply is trashed or spammed.

Change History (3)

#2 @netweb
2 years ago

  • Milestone changed from Under Consideration to 2.6

#3 @johnjamesjacoby
2 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 6312:

Moderation: Maybe add view=all query-arguments to view links in row actions.

Fixes #2938.

Note: See TracTickets for help on using tickets.