Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BuddyPress registration not picking up s2 role/caps #162

Closed
clavaque opened this issue May 22, 2014 · 12 comments
Closed

BuddyPress registration not picking up s2 role/caps #162

clavaque opened this issue May 22, 2014 · 12 comments
Assignees

Comments

@clavaque
Copy link
Contributor

I'm testing how the registration works with WP+BP+s2 Framework. I tried a signup link for Level 1, I got redirected to the BP registration page (BP replaces wp-login with its own pages) and registered. Then checked the new user's role and it's "subscriber", not level 1. s2 should be able to pick that up and give the new user the right role/ccaps.

@jaswrks jaswrks added this to the Next Release milestone Jun 3, 2014
@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

Moving this to the "Next Release" milestone.

@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

Attempting to reproduce...

jaswrks pushed a commit that referenced this issue Jun 3, 2014
jaswrks pushed a commit that referenced this issue Jun 3, 2014
jaswrks pushed a commit that referenced this issue Jun 3, 2014
jaswrks pushed a commit that referenced this issue Jun 3, 2014
@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

Confirmed, this is a bug. Investigating now.

jaswrks pushed a commit that referenced this issue Jun 3, 2014
@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

Cookies are being set, but not parsed during BP registration. Hmm...

2014-06-02_21-41-46

jaswrks pushed a commit that referenced this issue Jun 3, 2014
@clavaque
Copy link
Contributor Author

clavaque commented Jun 3, 2014

Probably related:
https://websharks.zendesk.com/agent/#/tickets/2849

@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

Yeah, something changed in the most recent version of BP apparently.

@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

It's only the role/caps being lost here; other meta values are not impacted by the bug. This tells me that BuddyPress might be overriding these details on activation. Still investigating.

@clavaque
Copy link
Contributor Author

clavaque commented Jun 3, 2014

Okay

@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

Confirmed. BP integrated several new routines dealing with registration in a much different way (internally) from previous releases. This will take some work to implement this into the latest release of BP. I will leave this open while work continues.

@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

We'll need to move our hook and cookie parser to
do_action( 'bp_core_activated_user', $user_id, $key, $user );

jaswrks pushed a commit that referenced this issue Jun 3, 2014
jaswrks pushed a commit that referenced this issue Jun 3, 2014
@jaswrks
Copy link
Contributor

jaswrks commented Jun 3, 2014

This has been resolved; coming in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants