Remove ?view=all param from profile thread responses

[jonoaldersonwp]

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).

[fierevere]

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

[Otto42]

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

[jonoaldersonwp]

Most recent reply on ​https://profiles.wordpress.org/sterndata/, for example.

Posted a <a href="https://wordpress.org/support/topic/database-error-322/?view=all#post-14033241">reply</a> to <i>Database Error</i>, on the site WordPress.org Forums:

[Otto42]

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.

[dd32]

In 10663:

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

Fixes #5607.

[dd32]

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.