id summary reporter owner description type status priority milestone component version resolution keywords cc blockedby blocking 164 """strpos(strtoupper("" in /includes/classes/split_page_results.php" Marcus Kreusch somebody "Gibt es einen Grund dafür dass strpos(strtoupper statt einfach stripos( verwendet wird? Wenn nicht, würde ich vorschlagen, das der Übersichtlichkeit zu liebe auszutauschen. Außerdem würde ich für diese drei Zeilen statt str(i)pos( vorschlagen strr(i)pos( zu verwenden: `$pos_group_by = strpos(strtoupper($this->sql_query), ' GROUP BY', $pos_from);`   `$pos_having = strpos(strtoupper($this->sql_query), ' HAVING', $pos_from);`      `$pos_order_by = strpos(strtoupper($this->sql_query), ' ORDER BY', $pos_from);` Also stattdessen: `$pos_group_by = strripos($this->sql_query, ' GROUP BY', $pos_from);`   `$pos_having = strripos($this->sql_query, ' HAVING', $pos_from);`      `$pos_order_by = strripos($this->sql_query, ' ORDER BY', $pos_from);` Der Grund: Wenn man in einer Produktlisting mal einen Subquery verwendet, der z.B. ein ""ORDER BY"" enthält, gibt das einen SQL-Fehler..." Erweiterung new niedrig Shop 2.0.1.0