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.

Sign up for GitHub

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

Jump to bottom

alt_auth plugin and not able to create user from other db + FIX #3982

Closed
Jimmi08 opened this issue Oct 25, 2019 · 1 comment
Closed

alt_auth plugin and not able to create user from other db + FIX #3982

Jimmi08 opened this issue Oct 25, 2019 · 1 comment
Labels
plugin: alt_auth status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing.

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Oct 25, 2019 •
edited
Loading

This code
$userMethods->addNonDefaulted($newUser);

causes wrong array for insert user:

image

Insert debug didn't return any error
Changing to:
$userMethods->addNonDefaulted($newUser['data']);

it fixed it. Again, no idea if this is correct and why this change.
@Moc Moc added status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. plugin: alt_auth labels Oct 27, 2019
@Jimmi08 Jimmi08 changed the title alt_auth plugin and not able to create user from other db alt_auth plugin and not able to create user from other db + FIX Oct 29, 2019
CaMer0n added a commit that referenced this issue Dec 28, 2019
@CaMer0n
Merge pull request #3992 from Jimmi08/patch-3
bcab517 
#3982 fix for wrong user data while creating user
@CaMer0n
Copy link
Member

CaMer0n commented Dec 28, 2019

@Jimmi08 Closing. Thank you. Please use "Fixes #(number)" in the description when submitting pull-requests to trigger auto-closing.

@CaMer0n CaMer0n closed this as completed Dec 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
plugin: alt_auth status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Moc @CaMer0n @Jimmi08