[buddypress-trac] [BuddyPress Trac] #7386: BP_Group_Extension: Create screen is not passed the new group ID.

buddypress-trac noreply at wordpress.org
Tue Dec 20 02:58:06 UTC 2016


#7386: BP_Group_Extension: Create screen is not passed the new group ID.
------------------------------+--------------------
 Reporter:  dcavins           |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  2.8
Component:  Groups            |     Version:  2.7.2
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+--------------------

Comment (by boonebgorges):

 > I'm sort of in favor of doing a more wholesale strip of the oddly
 isolated groups_action_create_group(). Now that you've moved some logic
 into setup_globals(), moving more of the create screen setup into that
 method would make the create screens seem more like a part of the groups
 component.

 I agree - this would be a worthwhile thing to pursue, as it would make the
 internals of the Groups component more consistent and easier to debug. But
 it also has the potential to introduce bugs, so it should be pursued with
 more caution than we need to make this minor fix. So let's handle it
 separately.

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


More information about the buddypress-trac mailing list