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

PHP 5.5 Deprecated mysql #58

Closed
larjohn opened this issue Sep 29, 2013 · 6 comments
Closed

PHP 5.5 Deprecated mysql #58

larjohn opened this issue Sep 29, 2013 · 6 comments

Comments

@larjohn
Copy link

@larjohn larjohn commented Sep 29, 2013

The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead

Would you?

@pietercolpaert
Copy link
Contributor

@pietercolpaert pietercolpaert commented Oct 24, 2014

👍 Problem which is very important for http://github.com/tdt/triples

@craigdietrich
Copy link
Contributor

@craigdietrich craigdietrich commented Oct 28, 2014

Curious if there's movement on this ticket? Or should I fork and run a pull request with the change?

@pietercolpaert
Copy link
Contributor

@pietercolpaert pietercolpaert commented Oct 28, 2014

A pull request would be great! They're willing to merge afaik

@larjohn
Copy link
Author

@larjohn larjohn commented Oct 28, 2014

I 've used a tool I found online to make the conversion in a past project.
I can't recall the tool, but it went all well!

ʌἁɀɑƿѻѕ ɪɷɑⱴⱴἱɗʜѕ

2014-10-28 21:44 GMT+02:00 Craig Dietrich notifications@github.com:

Curious if there's movement on this ticket? Or should I fork and run a
pull request with the change?


Reply to this email directly or view it on GitHub
#58 (comment).

@craigdietrich
Copy link
Contributor

@craigdietrich craigdietrich commented Oct 30, 2014

@larjohn I think this is the tool you mention: https://github.com/philip/MySQLConverterTool ... using it to work on the patch now.

@bnowack
Copy link
Collaborator

@bnowack bnowack commented Oct 31, 2014

Sigh, I wish I had some time to refactor the whole beast to an ARC3 and use PDO and stuff... But thanks for the patch, just merged in the pull request :-)

@bnowack bnowack closed this Oct 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants