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

Repair the application error loop #663

Open
wants to merge 87 commits into
base: master
Choose a base branch
from
Open

Conversation

domino91
Copy link

@domino91 domino91 commented Feb 21, 2017

Sorry to be deleted

Danielss89 and others added 30 commits June 8, 2014 22:38
…e class at this point.

Use Case: I have a guest login system which uses a separate route to login (zfcuser/guest) and the redirect plug-in is looking explicitly for zfcuser/login to do a redirect. I want to extend the RedirectCallback class with my own which does a check for the login/guest route, set the $currentRoute variable to zfcuser/login then call the parent function. No code duplication. At the moment it requires the entire class to be coppied/pasted to achieve the same functionality.
Changing the getRedirect function to protected as I need to extend the c...
ZF-Commons#520

If the option "login_after_registration" is set to "true", the redirection after register is invalid. Because in "RedirectCallback" file, in 'getRedirect" function
pass changePassword and changeEmail data to event
Remove default user state value check during registration process
Danielss89 and others added 30 commits August 17, 2015 13:49
Add zfcuser/authenticate to redirectCallback check
…sion

Update PHP version constraint to allow PHP 7
IsEmpty check in storage should not trigger reading from the mapper
Fix factories returning themselves
Fixed usage of zfcuser_login_form service in ZfcUserLoginWidget view helper
do not allow 5.6 failures and add php7 to test matrix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.