Datenbankfehler - Dislay.php

[Sternchen]

YaBB SE Version: 1.5.2 to 1.5.4
PHP Version: 4.3.4
MySQL Version: 4.0.x
Server Plattform: Windows 2000, XP, or 2003

Problem Beschreibung:
Hallo zusammen,

ich habe im ErrorLog dieses hier gefunden. Wie kann ich das abstellen. Hab seit gut einem halben Jahr nichts mehr verändert.

Datenbankfehler: You have an error in your SQL syntax.  Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 4
Datei: C:\www\ob0509\Sources\Display.php
Zeile: 95

Gruss Sternchen

[P_man]

hi sternchen,

poste mal bitte paar zeilen aus dem angegebenen bereich deiner display herüber. (sagen wir zw 80-110)

grüßle P

[Sternchen]

Code Select Expand


   //Redirect to page & post with new messages -- Omar Bazavilvazo
   if ($start == 'new')
   {
      // Check if a log exists, so we can go to next unreaded message page
      $request = mysql_query("
         SELECT GREATEST(IFNULL(lt.logTime, 0), IFNULL(lmr.logTime,0)) AS logtime, COUNT(m.ID_MSG), MAX(m.ID_MSG)
         FROM {$db_prefix}topics AS t
            LEFT JOIN {$db_prefix}log_topics AS lt ON (lt.ID_TOPIC=t.ID_TOPIC AND lt.ID_MEMBER=$ID_MEMBER)
            LEFT JOIN {$db_prefix}log_mark_read AS lmr ON (lmr.ID_BOARD=t.ID_BOARD AND lmr.ID_MEMBER=$ID_MEMBER)
            LEFT JOIN {$db_prefix}messages AS m ON (m.ID_TOPIC=t.ID_TOPIC)
         WHERE (t.ID_TOPIC = $viewnum)
         GROUP BY t.ID_TOPIC") or database_error(__FILE__, __LINE__);

      list($ltLastRead, $numMessages, $newestMessage) = mysql_fetch_row($request);

      $request = mysql_query("
         SELECT COUNT(*)
         FROM {$db_prefix}messages
         WHERE ID_TOPIC = $viewnum
            AND posterTime <= $ltLastRead") or database_error(__FILE__, __LINE__);
            
      list($numReadMessages) = mysql_fetch_row($request);
      $numUnreadMessages = $numMessages - $numReadMessages;

      if ($modSettings['viewNewestFirst'] == '1')
         $Page2Show = floor(($numUnreadMessages == 0 ? 0 : $numUnreadMessages - 1) / $maxmessagedisplay) * $maxmessagedisplay;
      else
         $Page2Show = floor(($numReadMessages  == $numMessages ? $numReadMessages - 1 : $numReadMessages) / $maxmessagedisplay) * $maxmessagedisplay;

      if ($numUnreadMessages > 0)
      {
         $request = mysql_query("
            SELECT MIN(ID_MSG)
            FROM {$db_prefix}messages
            WHERE ID_TOPIC = $viewnum
               AND posterTime > $ltLastRead") or database_error(__FILE__, __LINE__);
         list($firstUnreadMessage) = mysql_fetch_row($request);
         $newMsgID = "#msg$firstUnreadMessage";
      }
      elseif ($modSettings['viewNewestFirst'] == '1')
         $newMsgID = "#msg$newestMessage";
      else
         $newMsgID = '#lastPost';


und das ist die Zeile 95:
AND posterTime <= $ltLastRead") or database_error(__FILE__, __LINE__);

das ist die 8 letzte Zeile von unten nach oben gezählt.
Hoffentlich kannst Du etwas damit anfangen.

Gruss Sternchen

[P_man]

also der ganze code sieht bei mir haar genauso aus und läuft.

kann keinen fehler finden. außer das die betreffende zeile weiter oben steht

AND posterTime <= $ltLastRead") or database_error(__FILE__, __LINE__);

[SnowCrash]

Hmm... Sternchen? Bist Du Dir mit der MySQL Version sicher?

Ansonsten hätte ich zwei Ideen...

Einmal könntest Du den Teil

Code Select Expand

SELECT GREATEST(IFNULL(lt.logTime, 0), IFNULL(lmr.logTime,0)) AS logtime, COUNT(m.ID_MSG), MAX(m.ID_MSG)

gegen

Code Select Expand

SELECT MAX(IFNULL(lt.logTime, 0), IFNULL(lmr.logTime,0)) AS logtime, COUNT(m.ID_MSG), MAX(m.ID_MSG)

austauschen und dann noch

Code Select Expand

     WHERE ID_TOPIC = $viewnum
              AND posterTime > $ltLastRead") or database_error(__FILE__, __LINE__);

gegen

Code Select Expand

     WHERE ID_TOPIC = $viewnum
              AND posterTime > ".(int)$ltLastRead) or database_error(__FILE__, __LINE__);

und vielleicht noch

Code Select Expand

        WHERE ID_TOPIC = $viewnum
           AND posterTime <= $ltLastRead") or database_error(__FILE__, __LINE__);

gegen

Code Select Expand

        WHERE ID_TOPIC = $viewnum
           AND posterTime <= ".(int)$ltLastRead) or database_error(__FILE__, __LINE__);

... vorher aber ne Sicherheitskopie Deiner Display.php anlegen!

Der Fehler könnte 2 Gründe haben... einmal das GREATEST in der ersten DB-Abfrage (GREATEST wird nicht von allen MySQL-Versionen unterstützt) und einmal die $ltLastRead Variable... wenn die nicht gefüllt wird, dann kommt nen SQL-Fehler... durch das (int) ist die Variable = 0 wenn sie nicht gefüllt ist...

Aber ob das Dein Problem 100% löst... keine Ahnung... dazu kenn ich Dein Boar dnicht gut genug.... (installierte Mods, etc.)

[P_man]

ich würde mal auf letzteres tippen, daß die lastread vereinzeld nich gefüllt ist. sternchens frage sah nich so dramatisch aus, als daß das öfter wäre und ihr forum zusammenbräche.

P