[buddypress-trac] [BuddyPress Trac] #5557: BP_Group_Extension settings_screen_save

[buddypress-trac]

#5557: BP_Group_Extension settings_screen_save
-------------------------+------------------
 Reporter:  dcavins      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  low          |   Milestone:  2.1
Component:  Groups       |     Version:
 Severity:  minor        |  Resolution:
 Keywords:  needs-patch  |
-------------------------+------------------

Comment (by dcavins):

 Something else to consider is that if the plugin author (me) is only using
 the catch-all `settings_screen_save` and forces a redirect at the end,
 that redirect will also be fired when saving from the WordPress group
 admin interface, which is not ideal, since it usually redirects the user
 to the group's front-end admin screen. And I imagine that if it occurs
 before some other piece is saved, there could be conflicts and lost data.

 I'm mostly noting this for myself, so that the new "should we redirect"
 logic takes into account where the save request is coming from.

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