search

zombor / Auto-Modeler

Simple and easy to use Kohana CRUD/ORM library
ISC License
66 stars 10 forks source link

Auth Automodeler ORM references ORM #34

Open nanodocumet opened 13 years ago

nanodocumet commented 13 years ago

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 commented 13 years ago

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 commented 13 years ago

Not tested, what about this?

https://gist.github.com/937184