[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