Forums: Spam/Pending/Archived views should append ?view=all to topic/reply links

[SergeyBiryukov]

Since [4225], Spam/Pending/Archived views have some context for posts (i.e. forum name, topic name, topic link).

However, if you follow the topic/reply links from those views, you can't see the actual spam/pending/archived post on the topic page, because ?view=all is missing in the URL.

[SergeyBiryukov]

Related: #1984

[coffee2code]

In 4232:

Support Forums, Moderators: Add class constant VIEWS to list moderator-only views.

See #2134.

[coffee2code]

In 4233:

Forums Theme: Append '?view=all' to context forum and topic links for posts listed in moderator views (e.g. spam/pending/archived).

Fixes #2134.

[netweb]

Related: ​#bbPress2938

[SergeyBiryukov]

Noticed a couple of things:

• In reply to: is a link, As the topic: is not. It should be a link too for consistency.
• Forum and topic links now have ?view=all, but individual post links in .bbp-topic-post-date and .bbp-reply-post-date do not, so you have to additionally scroll to the post you clicked on, and it might not even be on the first page.

meta-2134.patch​ handles the first point. Should the second point wait for ​#bbPress2938?

[SergeyBiryukov]

In 4770:

Support Theme: Append '?view=all' to context forum and topic links for posts listed in moderator views (e.g. spam/pending/archived).

Merges [4233] to the new theme.

Props coffee2code.
See #2134.

[SergeyBiryukov]

meta-2134.patch​ was merged in [4770], so going to close as fixed.

A new ticket can be opened for the second point of comment:6.