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

Auth Automodeler ORM references ORM #34

Open
nanodocumet opened this issue Apr 22, 2011 · 2 comments
Open

Auth Automodeler ORM references ORM #34

nanodocumet opened this issue Apr 22, 2011 · 2 comments
Assignees
Labels
Milestone

Comments

@nanodocumet
Copy link

Is this expected?

https://github.com/zombor/Auto-Modeler/blob/master/classes/auth/automodeler/orm.php#L94

I thought that by having AM there is no need for ORM, however, I see some references to ORM in the auth driver. But in other places it references AM_ORM

https://github.com/zombor/Auto-Modeler/blob/master/classes/auth/automodeler/orm.php#L58

Shouldn't AM be self-contained and not require ORM?

@zombor
Copy link
Owner

zombor commented Apr 22, 2011

Yes, it should be self contained. I've never used tokens with AM auth, and it seems i missed those lines when converting the driver over.

@nanodocumet
Copy link
Author

Not tested, what about this?

https://gist.github.com/937184

@ghost ghost assigned zombor Apr 22, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants