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

Update Model.php #315

Closed
wants to merge 1 commit into from
Closed

Update Model.php #315

wants to merge 1 commit into from

Conversation

Yacine-krk
Copy link
Contributor

it looks like it is possible to use multiple db connection in one single request, instead of reusing the PDO class instance.
This ensure to use instantiate one single time the Database.

it looks like it is possible to use multiple db connection in one single request, instead of reusing the PDO class instance.
This ensure to use instantiate one single time the Database.
@Dominic28
Copy link
Contributor

Hey, please take a look at the develop branch. The whole script was refactored and Model.php no longer exists.

@panique
Copy link
Owner

panique commented Dec 15, 2013

Thanks @Yacine-krk , but this (or something similar) has already been done in the develop branch. Please never push into master branch, as the master always contains a stable release. The develop branch holds the in-development version. Currently the entire script collection gets a massive makeover, therefore a lot of things will change. Sorry, normally I would thankfully merge your commit, but i this case it's not possible :/

@panique panique closed this Dec 15, 2013
@Yacine-krk Yacine-krk deleted the patch-1 branch December 16, 2013 22:40
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

Successfully merging this pull request may close these issues.

None yet

3 participants