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

Fix GROUP BY clause errors #18

Merged
merged 2 commits into from Mar 16, 2017
Merged

Conversation

shemgp
Copy link
Contributor

@shemgp shemgp commented Mar 15, 2017

On default Ubuntu 16.04 install with MySQL 5.7, sip_additional.conf was not being populated. Traced it to mysql having error like:

mysql> SELECT data,id from sip where keyword='account' and flags <> 1 group by data;
ERROR 1055 (42000): Expression #2 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'asterisk.sip.id' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by

shemgp and others added 2 commits March 15, 2017 13:59
On default Ubuntu 16.04 install with MySQL  5.7, sip_additional.conf was not being populated.  Traced it to mysql having error like:
```
mysql> SELECT data,id from sip where keyword='account' and flags <> 1 group by data;
ERROR 1055 (42000): Expression FreePBX#2 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'asterisk.sip.id' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by
```
@tm1000
Copy link
Contributor

tm1000 commented Mar 16, 2017

Thank you for your code submission. We appreciate your participation in the efforts to improve FreePBX!

Code submissions usually requires a Code Licensing Agreement, but this patch has been deemed trivial, or obvious, and does NOT require you to submit a CLA for this code to be accepted. Thanks again!

We do encourage you to fill out and submit a CLA for any future submissions you may do, and you can read more about it in the wiki, and fill it out by completing this PDF and emailing it back to us at code@sangoma.com

@freepbx-tango freepbx-tango merged commit 57b0731 into FreePBX:release/13.0 Mar 16, 2017
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 this pull request may close these issues.

None yet

3 participants