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

MySQL 8 "rank" setup issues #715

Closed
brett-mcgirr opened this issue Jul 18, 2021 · 5 comments
Closed

MySQL 8 "rank" setup issues #715

brett-mcgirr opened this issue Jul 18, 2021 · 5 comments

Comments

@brett-mcgirr
Copy link

Hi, I'm installing Jethro for the first time and running into a few issues regarding the use of rank as a column name on MySQL.

Would you like a pull request for the changes I made to get it working?

If no, here's a list of files I've changed:
age_bracket.class.php line 41
person_group.class.php line 104
roster_role_assignment.class.php line 13
db_object.class.php lines 62 and 66
installer.class.php lines 183 and 194

@tbar0970
Copy link
Owner

Hi there, thanks for this. I'm happy to apply these manually rather than doing a PR.
Just wanted to check: Are you running version 2.30.1? We fixed a bunch of these 'rank' issues recently, but perhaps there are some we missed.

@brett-mcgirr
Copy link
Author

Thanks Tom. I was running the latest version from the master branch, but I've checked the zipped source in 2.30.1 and the issue is still there. To reproduce the problem, do a clean, initial install with a MySQL 8 backend.

@tbar0970
Copy link
Owner

Thanks for this, I've done a search and added backticks for the rank keyword in SQL in many places.
The tricky thing is that MariaDB doesn't have rank as a reserved word!

@2SJH
Copy link

2SJH commented Oct 15, 2021

When installing the current master branch, I have one instance of this still causing problems. It's in service_item.class.php at line 77. It needs:
'servicerank' => Array('serviceid', 'rank'),

After that change, an install worked for me

@tbar0970
Copy link
Owner

I fixed the above more centrally in 79ddca1

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

No branches or pull requests

3 participants