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

Alternative login mechanism #1406

Open
castamir opened this issue Feb 14, 2014 · 4 comments
Open

Alternative login mechanism #1406

castamir opened this issue Feb 14, 2014 · 4 comments

Comments

@castamir
Copy link

There is a note in David's TODO list about alternative way to user authentication and one part of it will by quite handy to me - login from database.

I use nice ORM but entities cannot be serialized (what a surprise) so I have to have one more unwanted property profile which is set in a secured BasePresenter manualy. It contains all data I need, but there is still $presenter::$user which becames confusing now because it no longer holds all proper data.

This situation was quite schizophrenic to me so I decided to modify Identity and UserStorage. There is no problem to rewrite Identity, but when I tried to modify (inheritted) UserStorage, I have realized that there is only few lines to rewrite but due to private fields I had to copy almost whole implementation.

So, is it possible to change these 3 fileds to protected or at least could you answer me, why there is such a condition? I would like to store only id, but to modify this condition, I have to rewrite so much... for what reason?

@enumag
Copy link
Contributor

enumag commented Feb 15, 2014

@castamir
Copy link
Author

thx.

It's quite dirty workaround to me, but it's working =)

@enumag
Copy link
Contributor

enumag commented Feb 15, 2014

Yeah. I know it's dirty. Better than nothing though. ;-)

@fprochazka
Copy link
Contributor

I disagree, Majkl578/nette-identity-doctrine is awesome :)

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

4 participants