[buddypress-trac] [BuddyPress] #3647: bp_forum_permalink() is missing a ’/’ (malformed)

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Mon Oct 3 13:08:44 UTC 2011


#3647: bp_forum_permalink() is missing a ’/’ (malformed)
---------------------+--------------------
 Reporter:  Grimbog  |       Owner:
     Type:  defect   |      Status:  new
 Priority:  normal   |   Milestone:  1.5.1
Component:  Forums   |     Version:  1.5
 Severity:  normal   |  Resolution:
 Keywords:           |
---------------------+--------------------

Comment (by boonebgorges):

 You're right that the link is being created incorrectly. We do need to add
 the slash.

 However, it's worth noting that bp-default, in BP 1.5, does not actually
 use this function at all (for whatever reason). It'll called in bp-themes
 /bp-default/forums/single/forum-header.php, which isn't actually called at
 the moment (the logic in bp_forums_screen_single_topic() never returns
 true).

 Also, trailingslashit() should not be allowing double-slashes. See
 http://core.trac.wordpress.org/browser/tags/3.2.1/wp-
 includes/formatting.php#L1170

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3647#comment:3>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list