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

2 possible (unrelated) issues in PropelBundle\Security #60

Closed
drjones69 opened this issue Oct 10, 2011 · 1 comment
Closed

2 possible (unrelated) issues in PropelBundle\Security #60

drjones69 opened this issue Oct 10, 2011 · 1 comment

Comments

@drjones69
Copy link

Afternoon!,

After much (more) struggling to get symfony2 to use Propel as a provider for security, I believe I've come across a further couple of issues, I'm listing them here should anyone wish to investigate.

  1. Within : Propel\PropelBundle\Security\User\ModalUserProvider (line 93)
    This is given as 'if ($user instanceof $this->class) {', are you sure it shouldn't be 'if (!($user instanceof $this->class)) {' ?
    It continually throws a "There is no user provider for user" exception in it's initial state (via (sf) firewall ContextListener), but works as expected when negated.

  2. Within : Propel\PropelBundle\Security\User\ModalUserProvider (line 64 & 65)

ucfirst($property) should be (I believe), ucfirst($this->property) (causes fatal error in current form)

Feel free to drop me an email or catch me in #propel (nick=drjones69) for any clarification.

Regards,
Ryan

@willdurand
Copy link

Fixed, thanks!

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

No branches or pull requests

2 participants