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

[buddypress-trac]

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

Comment (by boonebgorges):

 Cool - thanks for sharing, r-a-y. This is really helpful.

 > However, since redirections would probably be done infrequently, it
 might be okay to use groupmeta.

 Yes, this. Won't perform super well, but like you say, it's fairly
 uncommon.

 I think we can do something a bit simpler for groups than what you've done
 for members. Member lookup is done during the parse_uri() routine. But
 group lookup is done well afterward, during
 `BP_Groups_Component::setup_globals()`. So we just do a check for
 `group_exists()`, and if it fails, try to look for an old group slug in
 groupmeta, then redirect.
 https://buddypress.trac.wordpress.org/browser/tags/2.1.1/src/bp-groups/bp-
 groups-loader.php#L177

 Of course, it would be nice to move this stuff - which is, essentially,
 "canonical" logic - into a centralized place.

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