[buddypress-trac] [BuddyPress Trac] #5831: Activation link - change from "?key=" to action variable

[buddypress-trac]

#5831: Activation link - change from "?key=" to action variable
-------------------------------------+-----------------------
 Reporter:  r-a-y                    |       Owner:  djpaul
     Type:  defect (bug)             |      Status:  reopened
 Priority:  normal                   |   Milestone:  2.2
Component:  Members                  |     Version:  1.5
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+-----------------------

Comment (by r-a-y):

 > This is certainly going to break backwards compatibility for anyone
 checking the $_GET on their own.

 If plugins were doing checks for the key on their own, wouldn't the plugin
 need to remove how BP did its activation process with:

 `remove_action( 'bp_screens', 'bp_core_screen_activation' );`

 > I think we should encapsulate the key getting logic in a function to
 direct people to

 Sure, I think that's fair.

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