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

Abstracted storage container interface. #11

Closed
CMCDragonkai opened this issue May 10, 2013 · 5 comments
Closed

Abstracted storage container interface. #11

CMCDragonkai opened this issue May 10, 2013 · 5 comments

Comments

@CMCDragonkai
Copy link

Would it be possible to provide a storage container interface. So that way you don't need to enforce PDO or MySQL?

Although php_rbac uses foreign keys, I'm sure if there was a storage container interface, people could code up plugins to support different databases and even NoSQL.

All you need to do is explicitly state what is expected of the current query. Such as the object relationship to the classes. I noticed you mapped the stuff coming out of the db to your classes.

As for foreign key support, that could be done via, some sort of application based mapping. So if there was a way to explicitly state the expectations of the database query (what to drop or cascade), then people could write plugins supporting NoSQL or databases with no foreign key support.

@CMCDragonkai
Copy link
Author

Any progress on this? BTW, I integrated your library into PolyAuth -> https://github.com/Polycademy/PolyAuth/

@leighmacdonald
Copy link
Owner

Looks Great :)

I've been a bit busy with other project things lately, ill try and get the branch completed and merged this weekend for you.

leighmacdonald pushed a commit that referenced this issue May 31, 2013
@CMCDragonkai
Copy link
Author

Hey Leigh, any updates on this. I see a new branch for it, is it currently working for MySQL? Cause that's all that's required to allow new pull requests.

leighmacdonald pushed a commit that referenced this issue Jul 2, 2013
leighmacdonald pushed a commit that referenced this issue Jul 2, 2013
leighmacdonald added a commit that referenced this issue Jul 2, 2013
@leighmacdonald
Copy link
Owner

Should be ready for testing now.

@CMCDragonkai
Copy link
Author

Does pgsql have foreign keys? Because if it doesn't then the sql and adapter needs some modifications.

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