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.

Sign up for GitHub

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

Jump to bottom

MySQLi update #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

MySQLi update #9

wants to merge 2 commits into from

Conversation

w1lla
Copy link
Member

@w1lla w1lla commented May 7, 2014

Usage of MySQLi

@NewboO
Copy link
Contributor

NewboO commented May 9, 2014

It should even completly replace the MySQL classes!

@w1lla
Copy link
Member Author

w1lla commented May 9, 2014

I agree as i did some tests with it and it seems to have good advantages

@magnetik
Copy link
Contributor

magnetik commented Jul 3, 2014

The long term solution would be to use PDO or sympfony dbal.
Anyway this is good enough for now.

Could you just update the pull request and rename the folder MySQLi ?
(You only have to update your own branch and it's automatic)

@w1lla
Copy link
Member Author

w1lla commented Jul 3, 2014

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@w1lla @NewboO @magnetik