Opened 7 years ago
Closed 7 years ago
#3181 closed defect (bug) (fixed)
Wrong trailing slash on reply link on top of topic page
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.6 | Priority: | high |
Severity: | normal | Version: | 2.5.14 |
Component: | Component - Replies | Keywords: | needs-patch |
Cc: |
Description
On top of topic pages, where the freshness info are added, there is a link wrapping them with an hash to move the reader to the new reply form.
That link is generated on file
wp-content/plugins/bbpress/includes/replies/template.php at lines 495-496:
if ( 1 >= $reply_page ) {
$url = trailingslashit( $topic_url ) . $reply_hash;
Adding a slash to a permalink, where the blog permalinks DO NOT end with slash creates a different url which forces the page reload to reply (and a lot of new links scanned by search engines bots).
Since an hash is added (#post-...) there is no need to add a slash.
Thank you, Stefano.
Change History (3)
Note: See
TracTickets for help on using
tickets.
Probably should use
user_trailingslashit()
instead.Might be other places this is needed.