-
Notifications
You must be signed in to change notification settings - Fork 32
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
Phinx migrations dies on fresh install step_3 #197
Comments
@prhost or @lucasjkr , can You, please, add MySQL version check? But, if this can't be ignored during the installation, then limitations must be applied during install. |
I'll work on the version check, I need it too. I don't think you should ignore the version though (which I'm assuming you might be thinking about since you're saying it won't have impact? maybe I'm wrong) |
Yes, this was my first thoughts. But of course, it shouldn't. :) |
One issue I realized. You can't check the MySQL version as a warning during install (like, where it says what PHP modules are installed), because you can't communicate with the DB to check the version until AFTER you've set up the connection. Also, you're still pretending to support other databases (MSSQL, Postgres). You should officially end that support if you're going to do this through SQL. Otherwise, wait for @prhost to show off his ORM, and have that do the checking against the DBs. |
Older mysql databases don't support full text indexes
There should be some kinda error handle for this
The text was updated successfully, but these errors were encountered: