Patch: Sortierung in diversen Listen anpassen

compoundbow83 · 12. Juni 2016

Zitat von Wiimm

Den Gedanken hatte ich auch schon. Ich bin aber der falsche Ansprechpartner, da ich nicht zum Entwickler-Team gehöre und mich erst in die Tiefen einarbeiten müsste.

Ok, aber trotzdem danke für deine Antwort.

Wiimm · 12. Juni 2016

@oldie
Das mit Spiel 14 ist schon längst gegessen, war aber letzte Nacht aktuell, wie man an den vielen Themen sieht.

Zitat von oldie

@ Wiimm
mal wieder
....
auserdem verführt mir Wimme die User viel zu viel zum herum Murksen im code

Willst Du mich mit "Murks" beleidigen?
Meine Patches sind kein Murks, sondern gut getestete Tipps, die etwas an der Reihenfolge der Tabellen ändern. Und die Auswirkung ist auch gut dokumentiert. So habe z.B. ich nie gesagt, dass der Kode hier irgendwelche Fehler behebt oder besser ist, sondern nur als Alternative angepriesen. Außerdem ist es hier im Forum üblich, Änderungen am Quelltext als Fehlerbereinigung zu posten.

Als Murks könnte man höchstens soetwas wie deinen Tipp zum Ändern eines Regex nehmen, der syntaktisch falsch ist und damit scheitern muss. Oder aber die Aussage, dass es unnötig ist, ein Sicherheitsloch zum Indentätsdiebstahl zu schließen, obwohl es sehr einfach möglich ist. Dieses würde ich aber nie so schreiben, es sei denn ich werde wie hier angegriffen.

Und das "mal wieder" ist eine herabsetzende Bemerkung, die ich nicht verdient habe.

Im übrigen versuche ich immer höflich und sachlich zu sein, niemanden zu diffamieren oder herabzusetzen und kann mich auch für Fehler entschuldigen.

NACHTRAG:
Ich entwickle auch OpenSource und habe mit Wiimmfi ein großes offenes Projekt. ich freue mich immer auf Code-Vorschläge, egal, ob sich sie annehme oder zurückweise. Allein für das Gespräch und eine versuchte Anregung bin ich dankbar.

Horst · 12. Juni 2016

Zitat von oldie

die User viel zu viel zum herum Murksen im code

@Wiimm
Immer locker bleiben, er hat sich deutlich ausgedrückt und die User angesprochen die Deinen Code verwenden. Diese murksen dann an Sachen rum die Sie meistens nict verstehen.
Nicht von Ungefähr gibt es hier so viele Nachfragen!
User, wie ich, die das Standard Skript samt Updates einsetzen haben so gut wie keine Probleme! 2,3 kleinere Sachen die aber nicht bedeutend waren.

floriber · 12. Juni 2016

Eine Idee zur Rangliste mit den Siegertreppchen.
Könnte man da noch angeben, welches Spiel als letztes gewertet wurde und wieviele Spiele bereits gespielt wurden (x von 51 gespielt und gewertet).
In der Rangliste (Auf einen Blick) steht ja das letzte gewerte Spiel drin, das könnte man rüberziehen oder?

Kassi · 28. Juni 2016

@Wiimm:
Weißt du was ganz schlecht bei der Sortierung von oben ist?
Die Sortierung greift auch, bevor die Ergebnisse angezeigt werden - das bedeutet, dass ich die Tipps der anderen herausfinden kann, da ich ja noch umtippen kann und damit sehe, wer vor mir bzw. hinter mir einsortiert wird.
Es muss also noch abgefragt werden, ob die Spiele schon gesperrt sind bevor sortiert wird...
Ich habe das noch nicht gemacht, weil ich die Sortierung nicht nutze.

Wiimm · 3. Juni 2018

Zitat von floriber

Danke nochmals hierfür. was man natürlich nun vor den spielen schon sieht ist wer quasi für welche mannschaft getippt hat.
steht ein spieler auf ersten seite ganz oben dann hat er wohl für die heimmannschaft getippt und jemand der auf der letzten seite steht eben für die gastmannschaft...

In diesem Jahr habe ich auch dieses Problem gelöst. Der folgende Patch für Datei statistik.spiel.php sortiert die Tipps für ein Spiel nach Ergebnis, sobald die Ergebnisse sichtbar sind. Ich habe ihn gut getestet, der finale Test findet dann aber vor dem Eröffnungsspiel statt.

Diff

--- statistik.spiel.php.orig 2018-05-30 08:38:28.495815158 +0200
+++ statistik.spiel.php 2018-06-03 09:11:22.220624350 +0200
@@ -698,6 +698,25 @@
AND users.isUnlocked = 1
LIMIT ".$begin.",".$length;
} else {
+ #----- by dc
+ $match_order = '';
+ $match_id = intval(ltrim($RUNTIME['PATH'][2],' 0'));
+ $query = "SELECT date FROM `{$CONFIG['MYSQL']['PREFIX']}spiele` WHERE id=$match_id";
+ $sql = $CONFIG['MYSQL']['CONNECT']->query($query);
+ if ( $sql && $sql->num_rows )
+ {
+ $row = $sql->fetch_row();
+ $match_date = intval($row[0]);
+ $now = time(); // + 11*86400 + 8*3600;
+ #error_log(sprintf('match=%d, now=%d, date=%d, delta=%d',$match_id,$now,$match_date,$now-$match_date));
+ if ( $now > $match_date - $CONFIG['TIPTIME'] )
+ $match_order = 'tipps.gast - tipps.heim, tipps.gast * tipps.gast'
+ .' - tipps.heim * tipps.heim, tipps.heim DESC,';
+
+ }
+ #error_log(sprintf('match=%d, order=%s',$match_id,$match_order));
+ #-----
+
$query = "SELECT tipps.id FROM ". $CONFIG['MYSQL']['PREFIX'] ."tipps AS tipps
LEFT JOIN ". $CONFIG['MYSQL']['PREFIX'] ."users AS users ON users.id = tipps.usr
WHERE spiel = '".$CONFIG['MYSQL']['CONNECT']->real_escape_string($RUNTIME['PATH'][2])."' AND users.isUnlocked = 1";
@@ -708,7 +727,7 @@
LEFT JOIN ". $CONFIG['MYSQL']['PREFIX'] ."spiele AS spiele ON spiele.id = tipps.spiel
WHERE tipps.spiel = '".$CONFIG['MYSQL']['CONNECT']->real_escape_string($RUNTIME['PATH'][2])."'
AND users.isUnlocked = 1
- ORDER BY name ASC
+ ORDER BY $match_order name
LIMIT ".$begin.",".$length;
}

Alles anzeigen

Die error_log()-Zeilen sind nur noch für den finalen Test drin, falls es wider Erwartens Probleme gibt.

Wiimm · 8. Juni 2018

Update:

Ein Tipper machte mich darauf aufmerksam, dass Tippübersicht bei einem Spiel unter Verwendung der "persönlichen Rangliste" gänzlich unsortiert ist. Daher habe ich obiges etwas angepasst und die Vorberechnung vor IF-THEN-ELSE gelegt.

Diff

--- statistik.spiel.php.orig 2018-05-30 08:38:28.495815158 +0200
+++ statistik.spiel.php 2018-06-08 08:38:00.842996441 +0200
@@ -674,7 +674,28 @@
WHERE spiel = '".$CONFIG['MYSQL']['CONNECT']->real_escape_string($RUNTIME['PATH'][2])."' AND users.isUnlocked = 1";
$sql = $CONFIG['MYSQL']['CONNECT']->query($query);
$num = $sql->num_rows;
-
+
+
+ #----- by dc
+ $match_order = '';
+ $match_id = intval(ltrim($RUNTIME['PATH'][2],' 0'));
+ $query = "SELECT date FROM `{$CONFIG['MYSQL']['PREFIX']}spiele` WHERE id=$match_id";
+ $sql = $CONFIG['MYSQL']['CONNECT']->query($query);
+ if ( $sql && $sql->num_rows )
+ {
+ $row = $sql->fetch_row();
+ $match_date = intval($row[0]);
+ $now = time(); // + 11*86400 + 8*3600;
+ #error_log(sprintf('match=%d, now=%d, date=%d, delta=%d',$match_id,$now,$match_date,$now-$match_date));
+ if ( $now > $match_date - $CONFIG['TIPTIME'] )
+ $match_order = 'tipps.gast - tipps.heim, tipps.gast * tipps.gast'
+ .' - tipps.heim * tipps.heim, tipps.heim DESC,';
+
+ }
+ #error_log(sprintf('match=%d, order=%s',$match_id,$match_order));
+ #-----
+
+ #error_log(print_r($RUNTIME['PATH'],1));
if ($RUNTIME['PATH'][3] == "custom") {
$t_query = "SELECT uid FROM ". $CONFIG['MYSQL']['PREFIX'] ."rangliste_custom WHERE owner = '".$_SESSION['s_id']."'";
$t_sql = $CONFIG['MYSQL']['CONNECT']->query($t_query);
@@ -696,6 +717,7 @@
WHERE tipps.spiel = '".$CONFIG['MYSQL']['CONNECT']->real_escape_string($RUNTIME['PATH'][2])."'
AND (".$t_query.")
AND users.isUnlocked = 1
+ ORDER BY $match_order name
LIMIT ".$begin.",".$length;
} else {
$query = "SELECT tipps.id FROM ". $CONFIG['MYSQL']['PREFIX'] ."tipps AS tipps
@@ -708,7 +730,7 @@
LEFT JOIN ". $CONFIG['MYSQL']['PREFIX'] ."spiele AS spiele ON spiele.id = tipps.spiel
WHERE tipps.spiel = '".$CONFIG['MYSQL']['CONNECT']->real_escape_string($RUNTIME['PATH'][2])."'
AND users.isUnlocked = 1
- ORDER BY name ASC
+ ORDER BY $match_order name
LIMIT ".$begin.",".$length;
}

Alles anzeigen

floriber · 10. Juni 2018

Hi WIIMM,

könntest Du hier bitte auch Deine Datei statistik.spiel.php zur Verfügung stellen?

Das wäre klasse!!

Ciao

florian

Wiimm · 10. Juni 2018

Der obige Patch beschreibt genau die Veränderungen, die ich vorgenommen habe.

Hier ist die komplette Datei: https://download.wiimm.de/misc…6/statistik.spiel.php.txt

Tags