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

Repair_settings.php - 1.1 compat logic vs php 7.x #33

Closed
sbulen opened this issue Nov 9, 2019 · 1 comment · Fixed by #35
Closed

Repair_settings.php - 1.1 compat logic vs php 7.x #33

sbulen opened this issue Nov 9, 2019 · 1 comment · Fixed by #35

Comments

@sbulen
Copy link
Contributor

sbulen commented Nov 9, 2019

I think we're seeing some last stragglers finally moving off of 1.x... Some moving to new hosts.

One problem seen is if they restore files & db onto a new host, they need to run repair_settings.php to get the settings pointing to the new locations. Although repair_settings.php has a 1.x compatibility mode, that mode assumes mysql (not mysqli) calls. If that host is pushing folks to php7, those mysql calls won't work (mysql_connect doesn't exist).

To fix, there are a few options -

  • add mysqli support to the 1.x compatibility logic
  • build a legacy 1.x-only version of repair_settings.php
  • support upgrading to the new code before running repair_settings.php (which doesn't work today, reason unknown as the error reporting is erroring out...)

Originally reported here:
https://www.simplemachines.org/community/index.php?topic=561873.0

@sbulen
Copy link
Contributor Author

sbulen commented Nov 28, 2019

Another issue with 1.1 - if db name or prefix are wrong, you error out (any php version):
image

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 a pull request may close this issue.

1 participant