[buddypress-trac] [BuddyPress Trac] #6014: Include BP Edit Group Slug into the core

buddypress-trac noreply at wordpress.org
Tue Apr 4 21:06:37 UTC 2017


#6014: Include BP Edit Group Slug into the core
-------------------------+-----------------------------
 Reporter:  slaFFik      |       Owner:  dcavins
     Type:  enhancement  |      Status:  accepted
 Priority:  normal       |   Milestone:  Future Release
Component:  Groups       |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  needs-patch  |
-------------------------+-----------------------------

Comment (by dcavins):

 I'm attaching a fresh version of the patch that moves the routing logic
 into `BP_Groups_Component::setup_globals()`. The current behavior is that
 attempting to visit a URL based on an old slug (like
 http://mysite.org/groups/oldslug/members) causes a 301 redirect to the
 correct URL (like http://mysite.org/groups/currentslug/members). In the
 case of several groups having had the same previous slug, the group that
 most recently used that slug wins.

 In addition to the points in
 https://buddypress.trac.wordpress.org/ticket/6014#comment:8, this patch
 also adds support for `orderby = meta_id` in BP_Groups_Group::get() and a
 number of new tests.

 Note that this patch relies on the patch at #7496 (adding `slug` parameter
 support in `BP_Groups_Group::get()`), which I'm planning to commit soon.

 Thanks again for your feedback!

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6014#comment:15>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list