OrderSprinter-Forum

Hallo Stefan,

ist denn das Sortieren implementiert inzwischen?

Viele Grüße

@Martin: vielen Dank, dass du hier im Forum mithilfst, die Fragen zu beantworten, das nimmt mir echt Arbeit ab :)

@udgesbou: Mir fehlte bisher die Zeit, aber jetzt habe ich mir den Thread nochmal angesehen. Das ursprüngliche Problem war doch, dass in deiner Tabelle die Einträge für die Sortierung einiger Tische aus irgendeinem Grund verloren gegangen sind und ich dir im Beitrag vom 27. Aug 2018, 11:14 beschrieben hatte, wie du die Sortierung wieder einrichten kannst. Wenn die Sortierung nach Update auf 1.5.32 wieder komplett durcheinander ist, deutet das für mich darauf hin, dass die sorting-Spalte von resttables immer noch NULL-Werte enthält. Ich würde dir empfehlen, dass wirklich geradezuziehen, damit du später keine Probleme bekommst.

Nun hast du ein weiteres Sortierkriterium eingebaut: die Summe der für einen Tisch bestellten Produkte. Ich habe mir notiert, dass ich dem Anwender in einer der zukünftigen Versionen als benutzerdefinierte Einstellung die Wahl lasse, ob er nach statischer Tischsortierung, oder zuerst noch nach Preis sortieren soll. Aber bis das ich das irgendwann einmal eingebaut habe, musst du eigene SQL-Tweaks immer wieder neu einbauen und ich kann dir nicht versprechen, dass die stets immer noch mit jeder Version funktionieren. Ich ändere die Sources sehr häufig, habe dann aber natürlich nicht im Kopf, welche Änderungen Anwender gemacht haben könnten. Nur so als Randbemerkung...

Gruß,

Stefan

[quote=MartinH post_id=1399 time=1563029988]Probier mal aus nach beiden Spalten zu sortieren:

ORDER BY pricesum DESC, R.sorting ASC

Dann wird zuerst nach Preis sortiert und bei gleichen Preisen nach "sorting".[/quote]
Servus Martin, das ist die Lösung, vielen Dank dafür. Jetzt arbeitet wieder alles so wie es soll :mrgreen: :mrgreen:


Grüße

Hi udgesbou,

du sortierst ja nur nach Preis. Bedeutet, dass die restlichen Tische (die ja alle den gleichen Preis haben, nämlich nix) in beliebiger Reihenfolge kommen dürfen.

Probier mal aus nach beiden Spalten zu sortieren:

ORDER BY pricesum DESC, R.sorting ASC

Dann wird zuerst nach Preis sortiert und bei gleichen Preisen nach "sorting".

Gruß

Martin

$sql = "SELECT R.id as id,R.tableno as name,R.sorting as sorting,";
	$sql .= " IFNULL(SUM(IF(Q.toremove='0' AND Q.paidtime is null AND Q.isclosed is null,Q.price,0.00)),0.00) as pricesum FROM %resttables% R";
	$sql .= " LEFT OUTER JOIN %queue% Q ON Q.tablenr=R.id WHERE R.removed is null AND active='1' AND ";
	$sql .= " R.roomid=? $areaWhere GROUP BY R.id,name ";
	$sql .= " ORDER BY R.sorting";

$sql = "SELECT R.id as id,R.tableno as name,R.sorting as sorting,";
	$sql .= " IFNULL(SUM(IF(Q.toremove='0' AND Q.paidtime is null AND Q.isclosed is null,Q.price,0.00)),0.00) as pricesum FROM %resttables% R";
	$sql .= " LEFT OUTER JOIN %queue% Q ON Q.tablenr=R.id WHERE R.removed is null AND active='1' AND ";
	$sql .= " R.roomid=? $areaWhere GROUP BY R.id,name ";
	$sql .= " ORDER BY pricesum DESC";

[quote=pichel post_id=1386 time=1562874538 user_id=2]Von 1.5.25 auf 1.5.32 habe ich so absolut gar nichts an der Tischsortierung gemacht.[/quote]
Das habe ich mir gedacht, deswegen finde ich es auch komisch.

[quote=pichel post_id=1386 time=1562874538 user_id=2]Bitte prüfe, ob du die modifizierte SQL-Abfrage wirklich an der richtigen Stelle eingebaut hast.[/quote]
Die von mir modifizierte Stelle findet sich ja in der Datei "roomtables.php" hier in Zeile 266.

Der Teil aus der o. g. Datei sieht so aus:

[code]$sql = "SELECT R.id as id,R.tableno as name,R.sorting as sorting,";
$sql .= " IFNULL(SUM(IF(Q.toremove='0' AND Q.paidtime is null AND Q.isclosed is null,Q.price,0.00)),0.00) as pricesum FROM %resttables% R";
$sql .= " LEFT OUTER JOIN %queue% Q ON Q.tablenr=R.id WHERE R.removed is null AND active='1' AND ";
$sql .= " R.roomid=? $areaWhere GROUP BY R.id,name ";
$sql .= " ORDER BY R.sorting";[/code]

Ich ändere die letzte Zeile wie folgt um, zumindest hat das bis jetzt immer funktioniert:

[code]$sql = "SELECT R.id as id,R.tableno as name,R.sorting as sorting,";
$sql .= " IFNULL(SUM(IF(Q.toremove='0' AND Q.paidtime is null AND Q.isclosed is null,Q.price,0.00)),0.00) as pricesum FROM %resttables% R";
$sql .= " LEFT OUTER JOIN %queue% Q ON Q.tablenr=R.id WHERE R.removed is null AND active='1' AND ";
$sql .= " R.roomid=? $areaWhere GROUP BY R.id,name ";
$sql .= " ORDER BY pricesum DESC";[/code]

Sobald ich die Datei entsprechend ändere, stehen zwar die Tische auf denen etwas gebucht wurde oben, aber der Rest wird im regelmäßigen Abfrageintervall durcheinandergeworfen.

Ich habe die "ordersprinter_room" und "ordersprinter_resttables" aus der Datenbank im Anhang noch einmal mit angefügt, vielleicht hat sich ja doch ein Fehler in der Datenbank eingeschlichen und ich erkenne ihn nicht :geek:


Danke und Grüße

Von 1.5.25 auf 1.5.32 habe ich so absolut gar nichts an der Tischsortierung gemacht. Bitte prüfe, ob du die modifizierte SQL-Abfrage wirklich an der richtigen Stelle eingebaut hast.

Servus,
ich habe soeben von der Version 1.5.25 auf die neue Version 1.5.32 geupdatet und dann meine Eintragungen für die Sortierung in der roomtables.php wieder entsprechend getätigt:

[quote=udgesbou post_id=1071 time=1535198491 user_id=166]
[code]$sql = "SELECT R.id as id,R.tableno as name,R.sorting as sorting,";
$sql .= " IFNULL(SUM(IF(Q.ordertime is not null AND Q.paidtime is null AND Q.isclosed is null,Q.price,0.00)),0.00) as pricesum FROM %resttables% R";
$sql .= " LEFT OUTER JOIN %queue% Q ON Q.tablenr=R.id WHERE R.removed is null AND active='1' AND ";
$sql .= " R.roomid=? $areaWhere GROUP BY R.id,name ";
$sql .= " ORDER BY pricesum DESC";[/code][/quote]

Allerdings werden jetzt wieder, wie bereits in [url=http://wp12426429.server-he.de/forum3/posting.php?mode=quote&f=2&p=1071#pr1071]diesem Beitrag[/url] beschrieben, die einzelnen Tische wild durcheinander gewürfelt. Die Tische auf denen Buchungen getätigt wurden stehen allerdings wieder oben.

Ich habe in phpmyadmin bereits die Tabelleninhalte bzgl. der Sortierung kontrolliert, hier stimmt alles, in der Spalte "sorting" kommt jede Zahl nur einmal vor. Sprich ich habe wie in [url=http://wp12426429.server-he.de/forum3/posting.php?mode=quote&f=2&p=1071#pr1076]diesem Beitrag[/url] alles überprüft.

Woher könnte dieses Problem eventuell noch kommen?


Danke und Grüße

Grüß dich,
ich habe nun die entsprechenden Korrekturen vorgenommen und sie da, das Durcheinanderwürfeln hat ein Ende.
Vielen Dank für den schnellen Lösungsansatz.

Wie das passieren konnte kann ich mir auch nicht erklären. Ich habe manuell nie etwas an der Datenbank verändert. Eventuell doch ein Bug o. Ä.! Ich werde das ganze auch über kommende Updates beobachten und gebe natürlich Rückmeldung, sollte mir etwas auffallen.

Die für mich geeignete Sortierung habe ich auch schon eingepflegt.


Schönes Wochenende und Grüße

Hallo,

ich habe mir den Export angeschaut. Es fällt auf, dass die Tischsortierung für Raum Theke (mit der Id 5) durcheinander geraten ist. Die Ursache kann ich mir nicht erklären. Hast du mal manuell in der Datenbank editiert? Es mag ja sein, dass es auf einem Bug in OrderSprinter basiert, aber dann muss die fehlerhafte Sortierung bereits lange vor 1.5.11 passiert sein, denn an den möglichen Stellen im Source-Code habe ich seit langer Zeit nichts mehr verändert.

So, wie lösen wir das jetzt..

Du hast den Export mit phpMyAdmin erzeugt, kannst es also bedienen. In phpmyadmin kannst du auch Tabelleninhalte verändern. Schau dir also mal alle Einträge der Tabelle ordersprinter_resttables an, bei der roomid=5 ist und die Spalte removed NULL ist. Dort wird dir auffallen, dass in der Spalte "sorting" der Wert 10 doppelt auftritt. Ersetze alle bis auf einen der 10-er Werte aufsteigend mit Zahlen ab 48. Ziel: Jede Sortierungsnummer für einen Raum darf nur einmal vorkommen, die Sortierung beginnt ab "1".

Anschließend kannst du die Verwaltungsansicht -> Datenbank -> Raumplan aufrufen und die Sortierung der Einträge für den Raum "Theke" gerne anpassen.

Die Zeile "ORDER Y R.sorting" der oben zitierten SQL-Anweisung, wie sie im freigegebenen Download enthalten ist, ist richtig, denn damit wird die Tischsortierung vorgenommen. Das ORDER BY pricesum DESC hattest du wahrscheinlich selbst eingebaut. Damit wird dann aber nur nach Preis sortiert, alle anderen Einträge werden bunt zusammengewürfelt. Aber du kannst natürlich mehrere Sortierkriterien angeben.

Gruß,

Stefan

Danke. Ich schaue es mir Anfang kommender Woche an, bin dieses Wochenende nicht zuhause.

Aber zumindest kann ich schon sagen, von 1.5.11 auf 1.5.13 ist in dem Bereich keine Sourcecode-Änderung passiert.

Nächste Woche mehr...