-
Notifications
You must be signed in to change notification settings - Fork 35
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
using mysql_connect in installation - unsupported since php7 #79
Comments
okay, like @skenow already mentioned, this is not a simple fix. PHP7 support is a requirement for 2.0 (as it will be the only version still supported by the time we release), so I propose to rewrite this part and make it simpler : only PDO. |
I already have half needed code in my DB rewrite branch (mongodb and pdo) :) I just need somehow just to finish. |
Great news, I'll have a look at what you did there
Op do 26 jan. 2017 09:18 schreef Raimondas Rimkevičius <
[email protected]>:
… I already have half needed code in my DB rewrite branch (mongodb and pdo)
:) I just need somehow just to finish.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#79 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADkFcrzrWcqaKxE492or9E6z669QGH-Bks5rWFa-gaJpZM4LaXnN>
.
|
I think you should wait a bit. At current stage my code is messed up ;( I need to find some time to fix it. |
Even though I'm using the PDO connection, the installer still tries to connect to mysql with an unsupported function. When I do the setup on PHP 7.0.9 I get the following :
in file icms1310/install/page_dbconnection.php
The text was updated successfully, but these errors were encountered: