-
Notifications
You must be signed in to change notification settings - Fork 178
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
Fix ONLY_FULL_GROUP_BY #80 #406
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You disabled some travis settings but to merge this those need to remain.
.travis.yml
Outdated
@@ -41,6 +41,8 @@ before_script: | |||
- if [[ "$DB" == 'mysql' ]]; then mysql -u root -e 'CREATE DATABASE oc_autotest;'; fi | |||
- if [[ "$DB" == 'mysql' ]]; then mysql -u root -e "CREATE USER 'oc_autotest'@'localhost' IDENTIFIED BY 'oc_autotest';"; fi | |||
- if [[ "$DB" == 'mysql' ]]; then mysql -u root -e "GRANT ALL ON oc_autotest.* TO 'oc_autotest'@'localhost';"; fi | |||
- if [[ "$DB" == 'mysql' ]]; then mysql -u root -e "SET GLOBAL sql_mode = 'ONLY_FULL_GROUP_BY';"; fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we make this an extra test? Maybe also one with STRICT_MODE?
4693972
to
d5f5c7c
Compare
Switch to xenial. Set sql_mode to `STRICT_ALL_TABLES, ONLY_FULL_GROUP_BY`
d5f5c7c
to
f1eb28c
Compare
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
…xtcloud#406) Signed-off-by: Talon <talontalon24@gmail.com>
Changed - Basic Media-RSS support (#599) - Database index improvements (#637) Fixed - Call to a member function getUrlHash() on null" when adding a feed (#640) - Don't install symfony/console via composer (#636) - Fix for for ONLY_FULL_GROUP_BY (see #406) (Issue #80) (#407) - Catch invalid feeds (#646) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
I'm not really sure about this.
But it seems to work.
I didn't get my local mysql to enforce
ONLY_FULL_GROUP_BY
.This might affect the speed of the query. I didn't find another way yet.
I will do some cleanup/rebase on the commits and travis file later.