Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#5607 closed defect (bug) (fixed)

Remove ?view=all param from profile thread responses

Reported by: jonoaldersonwp's profile jonoaldersonwp Owned by: dd32's profile dd32
Milestone: Priority: lowest
Component: Support Forums Keywords: seo performance


When a user comments on a thread, it's recorded in their profile.
The link to that thread appends a ?view=all parameter, which is redundant. This should be removed (in existing, and new response records).

Change History (7)

#1 @fierevere
3 years ago

It still can be a good idea to automatically append "?view=all" for moderators.

Last edited 3 years ago by fierevere (previous) (diff)

#2 @Otto42
3 years ago

Where is this view=all at? Because I don't see it in any links on profiles.

#3 @jonoaldersonwp
3 years ago

Most recent reply on, for example.

Posted a <a href="">reply</a> to <i>Database Error</i>, on the site Forums:

#4 @Otto42
3 years ago

Yeah, that's not universal.

I think that is only happening on moderator's replies, when they are using the view=all view that lets them see the archived replies as well.

If this is undesirable, then the correct thing to do would be to find the piece of the forums that sends the reply notification to the profiles and remove the data from it there, so that it's sending the desired URL.

#5 @dd32
3 years ago

  • Component changed from Profiles to Support Forums

#6 @dd32
3 years ago

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

In 10663:

Support: Don't include moderator query vars in the permalink sent to the Profiles feed.

Fixes #5607.

#7 @dd32
3 years ago

Note that the URL sent over by [10663] will now be .../topic-name/?#post-1234 with the extra ? in there, that's due to #core52503.

Note: See TracTickets for help on using tickets.