ShoutBox-Fehler inkl Patch

Wiimm · 30. Mai 2018

Es gibt einen Fehler in content/shout.php, wenn man einen DB-Namen mit Minuszeichen verwendet. Es liegt an den fehlenden Quotes in de Abfrage. Dadurch meckert das System ständig mit "Beim Anlegen der MySQL-Tabelle xxx trat ein Fehler auf ...". Die Lösung ist der folgende Patch:

Diff

--- content/shout.php.orig 2018-05-30 08:38:28.495815158 +0200
+++ content/shout.php 2018-05-30 09:06:18.092380396 +0200
@@ -235,15 +235,10 @@
//MySQL-Tabelle anlegen, wenn sie noch nicht existiert:
-$sql = "SHOW TABLES FROM ". $database ." LIKE '%".$tabelle."%'";
+$sql = "SHOW TABLES FROM `". $database ."` LIKE '%".$tabelle."%'";
$result = $CONFIG['MYSQL']['CONNECT']->query($sql);
-$menge = $result->num_rows;
-if($menge > 0) {
- $check = 1;
-}
-
-if ( ! isset($check) || isset($check) && $check != 1)
+if ( !$result->num_rows )
{
$install_query = "CREATE TABLE IF NOT EXISTS ".$tabelle." (
`id` int(10) NOT NULL auto_increment,

Alles anzeigen

Im Prinzip reicht die Anpassung der ersten Code-Zeile (Zeilen 7+8, Ergänzung der Back-Hacks "`"). Weil ich dabei war, habe ich den überflüssigen Schnick-Schnack auch gleich entfernt.

Wiimm · 1. Juni 2018

Vorab:
Es geht hier nicht um anschwärzen, wie mir in den letzten Jahren vorgeworfen wurde, sondern um das Aufdecken von Fehlern und deren Korrektur. Ich kenne das aus meinen eigenen Open-Source-Projekten, bei denen ich solche Korrekturen und Hinweise immer gerne entgegen genommen habe und auch weiterhin werde.

Zum Thema:
Ich wollte die ShoutBox erweitern, so dass 2 Gruppen (Abteilungen) jeweils eine eigene haben. Bei der Umsetzung ist mir folgendes Kode-Stück als Ressourcen-Fresser aufgefallen:

Code

case('getposts'):
$mengequery = $CONFIG['MYSQL']['CONNECT']->query("SELECT * FROM ". $tabelle ."");
$menge = $mengequery->num_rows;
$maxsites = ceil($menge/$limit);
if (isset($_POST['shoutsite']) && $_POST['shoutsite'] != "" && $_POST['shoutsite'] != 0)
{
$l1 = $_POST['shoutsite']*$limit;
$l1 = $l1-$limit;
$mysqllimit = " LIMIT ".$l1.",".$limit."";
if($_POST['shoutsite'] > 1){ $zl = $_POST['shoutsite']-1; } else { $zl = 0;}
if($_POST['shoutsite'] < $maxsites){ $vl = $_POST['shoutsite']+1; }
}
else
{
$mysqllimit = " LIMIT ".$limit."";
$zl = 0;
if($menge > $limit){ $vl = 2; }
}
$postquery = "SELECT s.*, u.userpic FROM ".$tabelle." s left join "
.$tabelleuser." u on u.id = s.uid ORDER BY s.time DESC".$mysqllimit."";
$posts = $CONFIG['MYSQL']['CONNECT']->query($postquery);
....

Alles anzeigen

Zeile 2 ist der Ressourcen-Fresser: Hier wird der komplette Inhalt (alle Zeilen, alle Spalten) der Shout-Tabelle geladen, um die Anzahl der Zeilen zu bestimmen. Bei mehreren 1000 Beiträgen kann das schon spürbar Zeit und Speicher kosten. Der Einsatz von »count(*)« und die Abfrage des Ergebnisses ist hier klar besser.
Die Anzahl (maxsites) wird aber nur im IF-Zweig benötigt. Daher sollte die Abfrage aus Zeile 2ff in den IF-Zweig verschoben werden.
Fraglich ist, ob überhaupt die Anzahl benötigt wird. Man kann das OFFSET+LIMIT durchaus auch blind berechnen. Sollte dann z.B. Seite 5 angefordert werden, aber nicht existieren, dann werden halt Null Zeilen ausgegeben.

Wiimm · 9. Juni 2018

Zur Info:

Oben genannter Fehler und Ressourcen-Fresser wurden mit dem Update vom 8. Juni beseitigt.