search

roxlukas / lmeve

LMeve - industry manager and contribution tracker for EVE Online
https://pozniak.pl/wp/
Other
52 stars 13 forks source link

New Esi token #64

Closed erikterr closed 5 years ago

erikterr commented 5 years ago

O7, new fresh installation, I receive
Database error. Contact your administrator and report the problem. When I try to add new ESI key. I checked on settings.conf but seem to be db and login are correct. Also when I press new ESI token the site tell "I'm in maintenance". Using last beta .59

roxlukas commented 5 years ago

Please go to file:

config/config.php

Find this:

//debug database queries and PHP errors? (include additional information in error messages)
$LM_DEBUG=FALSE;

Change it to:

$LM_DEBUG=TRUE;

Then try to add new ESI token again and let me know what the error message is (it should say which SQL query fails).

Also, could you please confirm you've followed the SSO integration part (developers.eveonline.com) ?

erikterr commented 5 years ago

Error in query: SELECT cet.*,aps.date,apc.corporationName FROM cfgesitoken cet LEFT JOIN (SELECT tokenID,MAX(date) AS date FROM esistatus GROUP BY tokenID) aps ON cet.tokenID=aps.tokenID LEFT JOIN apicorps apc ON cet.tokenID=apc.tokenID GROUP BY cet.tokenID; MySQL reply: SQLSTATE[42000]: Syntax error or access violation: 1055 Expression #4 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'Lmeve.apc.corporationName' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by

On Sat, Jan 12, 2019 at 6:07 PM Łukasz Poźniak notifications@github.com wrote:

Please go to file:

config/config.php

Find this:

//debug database queries and PHP errors? (include additional information in error messages) $LM_DEBUG=FALSE;

Change it to:

$LM_DEBUG=FALSE;

Then try to add new ESI token again and let me know what the error message is (it should say which SQL query fails).

Also, could you please confirm you've followed the SSO integration part ( developers.eveonline.com) ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/roxlukas/lmeve/issues/64#issuecomment-453763784, or mute the thread https://github.com/notifications/unsubscribe-auth/Aaes3Ql_0pjlGVNpFJX9c8UiV5xEqtXsks5vChZGgaJpZM4Z8vYl .

-- Terrenzani Erik From Gmail

roxlukas commented 5 years ago

The problem seems to be with different behaviour of mysql 5.7. Please check issue #58 look for workaround regarding sql_mode=only_full_group_by:

https://www.sitepoint.com/quick-tip-how-to-permanently-change-sql-mode-in-mysql/

I'll mark this as bug, and will modify the queries so they won't require workaround

Note for myself: https://stackoverflow.com/questions/34115174/error-related-to-only-full-group-by-when-executing-a-query-in-mysql

erikterr commented 5 years ago

Confirmed that sql mode change worked. Now ESI are ok. I'll wait for pull data

On Sat, Jan 12, 2019 at 7:26 PM Łukasz Poźniak notifications@github.com wrote:

The problem seems to be with different behaviour of new mysql version. Please check issue #58 https://github.com/roxlukas/lmeve/issues/58 look for workaround regarding sql_mode=only_full_group_by

I'll mark this as bug, and will modify the queries so they won't require workaround

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/roxlukas/lmeve/issues/64#issuecomment-453770274, or mute the thread https://github.com/notifications/unsubscribe-auth/Aaes3VkF6yzPIb_liLl0TCtgIpEbDwRfks5vCijmgaJpZM4Z8vYl .

-- Terrenzani Erik From Gmail