Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

queries with "group by" don't work on NAS #146

Closed
wants to merge 1 commit into from

Conversation

royvanmanen
Copy link

I've changed left join to count "anzahl" to a subquery and that works. Otherwise on a QNAP nas only the books list works, all the others do not.

I've changed left join to count "anzahl" to a subquery and that works. Otherwise on a QNAP nas only the books list works, all the others do not.
@rvolz
Copy link
Owner

rvolz commented Feb 22, 2015

Any specifics why this is necessary? Does QNAP NAS don't have SQLite 3?

@royvanmanen
Copy link
Author

Hi Rainer,

[cid:image001.png@01D04EC5.0F7ABC10]

This is what phpinfo shows.

I'm a windows system administrator and SQL server dba, my linux knowledge is better than beginner but I'm not a linux expert.

The "group by" statement doesn't work. In other SQl languages, the group by should contain all fields in the selection list, but that didn't work either.

Other bug:

It seems the "LOGIN_REQUIRED" variable is not retrieved from the database. I've checked, after saving …
[cid:image002.png@01D04ED1.9DF33EC0]

If I refresh the admin page, I see the login required is "no" while there is a 1 at "must_login" in the database. If I change the kindle setting, this is written but does work okay.
I've changed the default value in my index php to 1 ($globalSettings[LOGIN_REQUIRED] = 1;) that works.

Best regards,

Roy van Manen

Van: Rainer Volz [mailto:notifications@github.com]
Verzonden: zondag 22 februari 2015 15:51
Aan: rvolz/BicBucStriim
CC: Roy van Manen | Skyliner
Onderwerp: Re: [BicBucStriim] queries with "group by" don't work on NAS (#146)

Any specifics why this is necessary? Does QNAP NAS don't have SQLite 3?


Reply to this email directly or view it on GitHubhttps://github.com//pull/146#issuecomment-75438795.

@rvolz
Copy link
Owner

rvolz commented Feb 23, 2015

The pictures in your post didn't make it. Could you please just check the sqlite version on your QNAP?

sqlite3 --version and php5 --version

left join should be part of sqlite3. If this is an older version of SQLite then you might run into other problems too, sooner or later.

@fraubi
Copy link

fraubi commented Apr 1, 2015

Hallo Herr Volz,

ich habe das gleiche Problem auf meiner QNAP und konnte bislang keine Lösung finden.

reihen
schlagwoerter
fehlermeldung
Sobald die Auswahl in der Menüleiste nach "Autoren", "Schlagwörtern" oder "Reihen" erfolgt, ist die Anzeige fehlerhaft und es gibt, wenn man eine Auswahl trifft, eine Fehlermeldung (siehe Anhang).

Auf der QNAP ist PHP 5.5.21 sowie SQLITE 3.8.1-1 installiert.

Es scheint sich um ein Problem der QNAP zu handeln, denn auf meiner Synology läuft es fehlerfrei.
Wie kann ich behilflich sein, das Problem zu lösen ? Kann ich Log-Dateien fertigen ? Wenn, wie ?

Ich würde mich über eine Rückinfo sehr freuen, da BicBucStriim eine super Anwendung ist, die ich gerne auch auf meiner QNAP nutzen würde.

Viele Grüße
Andreas Frauboes

@mustela7
Copy link

mustela7 commented Jun 2, 2015

Hi, I have the same problem (as in the screenshots above) on a Qnap TS 470 pro, with current Firmware QTS 4.1.3 (and also newest QTS 4.1.4 beta version, 2015/05/22). BicBucStriim installation check said it is all ok, except no PHP module intl.
So it seems to be an unsolved Qnap problem? ... still any help appreciated!

@joelang1699
Copy link

Same issue here on TS-1079P 4.1.3

@rvolz
Copy link
Owner

rvolz commented Aug 18, 2015

I verified that the sqlite problem doesn't appear in version 3.8.2, on my test system. Does QTS 4.2 contain 3.8.2+?

@rvolz
Copy link
Owner

rvolz commented Sep 28, 2015

I heard there is now the QNAP 4.2 firmware. Did they finally update the buggy sqlite3 version, or is the problem still there?

@rvolz rvolz closed this in 817728f Oct 1, 2015
@mustela7
Copy link

mustela7 commented Oct 2, 2015

Hey Mr Volz, I just tried the newest version of BBS v. 1.3.1 with newest QNAP QTS 4.2.0 (still beta):
I still have the same issue (identical to the screenshots of fraubi on 1.April above).

Just wondering, as the workaround above for QNAP firmware does not seem to work?

rvolz added a commit that referenced this pull request Oct 3, 2015
@rvolz
Copy link
Owner

rvolz commented Oct 3, 2015

Hm, I changed the joins into subselects, as proposed by @royvanmanen. I wasn't convinced that this is the real problem, because the screenshots show the numbers, which were produced by the joins.

However, I noticed that I didn't remove the GROUP BY statements in these queries. They didn't do anything, but who knows. Could someone please try the Github version? See, if it changes anything for you?

BTW: you only have to download lib/BicBucStriim/calibre.php to test this if you don't want to replace the whole installation.

@rvolz rvolz reopened this Oct 3, 2015
@mustela7
Copy link

mustela7 commented Oct 3, 2015

That's it, this does the trick, many thanks!
Authors, Tags and Series are shown now and are selectable.
(At least here on a QNAP 470 pro with QTS 4.2 beta :-))

@rvolz
Copy link
Owner

rvolz commented Oct 3, 2015

Thx for testing. I'll change the rest of the queries and publish 1.3.2.

rvolz added a commit that referenced this pull request Oct 3, 2015
@rvolz
Copy link
Owner

rvolz commented Oct 3, 2015

v1.3.2 with the fixes is published

@rvolz rvolz closed this Oct 3, 2015
@mustela7
Copy link

mustela7 commented Oct 3, 2015

v1.3.2 funktioniert nun auf QNAP, vielen Dank!

Rückmeldung: Dafür ist nun der Config-Button oben rechts plötzlich verschwunden?: Er war bis und mit Erstinstallation (Konfigurationsseite) von v1.3.2 noch sichtbar, nun ist er weg. Die Buttons Suchen und Abmelden sind ok.

Freundliche Grüsse

Am 03.10.2015 um 18:10 schrieb Rainer Volz notifications@github.com:

v1.3.2 with the fixes is published


Reply to this email directly or view it on GitHub #146 (comment).

@rvolz
Copy link
Owner

rvolz commented Oct 5, 2015

Das ist ein Übergangsproblem mit alten Sessions. Da ich die dafür zuständige Bibliothek austauschen musste, haben sich die Sessionstrukturen geändert. Das alte admin-flag wird somit nicht erkannt.

Sobald die aktuelle Sesson abläuft und der Browser eine neue anlegt, ist alles wieder normal. Wer es eiliger hat, löscht das Session-Cookie im Browser oder die Session-Dateien auf dem NAS.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants