[buddypress-trac] [BuddyPress Trac] #5926: getting an activity permalink instead of a shortlink

[buddypress-trac]

#5926: getting an activity permalink instead of a shortlink
-------------------------+-----------------
 Reporter:  DJPaul       |      Owner:
     Type:  enhancement  |     Status:  new
 Priority:  normal       |  Milestone:  2.2
Component:  Activity     |    Version:
 Severity:  normal       |   Keywords:
-------------------------+-----------------
 Our existing `bp_activity_get_permalink` method returns the permalink for
 a single activity item, right? Actually, that's not true, because if you
 look at the function, it returns what I'd actually describe as a short
 link. For example:

 `bp_activity_get_permalink(567)` will return
 `http://example.com/activity/p/567/`.
 That URL will redirect to
 `http://example.com/members/admin/activity/567/`.

 I think `bp_activity_get_permalink` should return the canonical permalink
 (the second link), not the first. It would be more intuitive, bring it
 inline with WP's `get_permalink` which I'd guess many people who extend BP
 are familiar with, and finally give us a way to get the canonical link for
 a piece of activity content without having to dig into `$_SERVER`.

 If this proposed change is accepted, I suggest we rename
 `bp_activity_get_permalink` to `bp_activity_get_shortlink`, and build a
 real `bp_activity_get_permalink`.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5926>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac