Smartor-Album / SQL-Fehler bei zurückblättern

[shadowrider]

Deine phpBB Version: Vanilla phpBB 2.0.17
MODs: Ja
Dein Wissensstand: Grundwissen
Link zu Deinem Forum: http://www.oldtimerfreunde-schmiechen.de

PHP Version:
MySQL Version:4.0.23


Was hast Du gemacht, bevor das Problem aufgetreten ist?
eigentlich nichts, der Fehler besteht wohl schon länger, ist jedoch nicht aufgefallen bzw. gemeldet worden


Was hast Du bereits versucht um das Problem zu lösen?
Foren durchsucht



Fehlerbeschreibung und Nachricht

Photo Album Addon 2.0.53 Smartor with Volodymyr (CLowN) Skoryk's SP1 addon

im Album funktioniert das zurück-blättern der Bilder nicht, vorwärts geht problemlos, ich erhalte bei Zurück folgenden Fehler:

Code: Alles auswählen

 Could not query pic information

DEBUG MODE

SQL Error : 1064 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 '(AND new.pic_id <= cur.pic_id ORDER BY pic_time DESC LIMIT 1' a

SELECT new.pic_id, new.pic_time FROM phpbb2_album AS new, phpbb2_album AS cur WHERE cur.pic_id = 964 AND new.pic_id <> cur.pic_id AND new.pic_cat_id = cur.pic_cat_id (AND new.pic_id <= cur.pic_id ORDER BY pic_time DESC LIMIT 1

Line : 178
File : album_showpage.php

Auszug der php:

Code: Alles auswählen

$sql .= ($HTTP_GET_VARS['mode'] == 'next') ? " AND new.pic_id >= cur.pic_id" : " (AND new.pic_id <= cur.pic_id";

		$sql .= ($row['pic_cat_id'] == PERSONAL_GALLERY) ? " AND new.pic_user_id = cur.pic_user_id" : "";

		$sql .= ($HTTP_GET_VARS['mode'] == 'next') ? " ORDER BY pic_time ASC LIMIT 1" : " ORDER BY pic_time DESC LIMIT 1";

		if( !($result = $db->sql_query($sql)) )
		{
			message_die(GENERAL_ERROR, 'Could not query pic information', '', __LINE__, __FILE__, $sql);
		}

		$row = $db->sql_fetchrow($result);

		if( empty($row) )
		{
			$sql = "SELECT new.pic_id, new.pic_time 
	         	FROM ". ALBUM_TABLE ." AS new, ". ALBUM_TABLE ." AS cur 
	         	WHERE cur.pic_id = $pic_id_temp 
	            AND new.pic_id <> $pic_id_temp 
	            AND new.pic_cat_id = $pic_cat_id_temp "; 

	         $sql .= ($pic_cat_id_temp == PERSONAL_GALLERY) ? " AND new.pic_user_id = cur.pic_user_id" : ""; 

	         $sql .= ($HTTP_GET_VARS['mode'] == 'next') ? " ORDER BY pic_time ASC LIMIT 0,1" : " ORDER BY pic_time DESC LIMIT 0,1"; 


	         if( !($result = $db->sql_query($sql)) ) 
	         { 
	            message_die(GENERAL_ERROR, 'Could not query pic information', '', __LINE__, __FILE__, $sql); 
	         } 

	         $row = $db->sql_fetchrow($result); 

	         if( empty($row) ) 
	         { 
	            message_die(GENERAL_ERROR, $lang['Pic_not_exist']); 
	         } 

			}

			$pic_id = $row['pic_id'];
	}
}

(ich hoffe dieser Auszug reicht den Profis hier, denn ich werd daraus nicht schlau...)
Link zum Album

[oxpus]

Gleich die erste Zeile des Blocks:

Code: Alles auswählen

$sql .= ($HTTP_GET_VARS['mode'] == 'next') ? " AND new.pic_id >= cur.pic_id" : " (AND new.pic_id <= cur.pic_id";

Da ist eine Klammerh zu viel.
So wäre diese Zeile korrekt:

Code: Alles auswählen

$sql .= ($HTTP_GET_VARS['mode'] == 'next') ? " AND new.pic_id >= cur.pic_id" : " AND new.pic_id <= cur.pic_id";

[shadowrider]

genau der war's - nun geht das, danke