search

isotope / core

Core repository of Isotope eCommerce, an eCommerce extension for Contao Open Source CMS
https://isotopeecommerce.org
135 stars 107 forks source link

Product Page shows error when page/article tree is filtered by a specific node (Contao 2.10.4, Isotope 1.3.0 RC1) #323

Closed ghost closed 11 years ago

ghost commented 11 years ago

When you're not at the main tree for pages/articles, you cannot view the main.php?do=iso_products and get this error:

Fatal error: Uncaught exception Exception with message Query error: Unknown column 'sorting' in 'order clause' (SELECT id, pid FROM tl_iso_products WHERE pid IN(0) ORDER BY FIND_IN_SET(pid, '0'), sorting) thrown in […]system/libraries/Database.php on line 689

http://www.contao-forge.org/issues/ […]/system/libraries/Database.php(636): Database_Statement->query() http://www.contao-forge.org/issues/ […]/system/libraries/Database.php(164): Database_Statement->execute() http://www.contao-forge.org/issues/ […]/system/libraries/Controller.php(2811): Database->execute('SELECT id, pid ...') http://www.contao-forge.org/issues/ […]/system/libraries/Controller.php(2953): Controller->getChildRecords(0, 'tl_iso_products', true) http://www.contao-forge.org/issues/ […]/system/drivers/DC_Table.php(225): Controller->eliminateNestedPages(Array, 'tl_iso_products', true) http://www.contao-forge.org/issues/ […]/system/drivers/DC_ProductData.php(71): DC_Table->__construct('tl_iso_products') http://www.contao-forge.org/issues/ […]/system/modules/backend/Backend.php(210): DC_ProductData->__construct('tl_iso_products') http://www.contao-forge.org/issues/ […]/contao/main.php(101): Backend->getBackendModule('iso_products') http://www.contao-forge.org/issues/ […]/contao/main.php(286): Main->run() http://www.contao-forge.org/issues/ {main}

see http://www.contao-forge.org/issues/2451)#note-8 and https://www.contao-community.de/showthread.php?24978-Isotope-2-10-3-1-3-Beta-2-Problem-mit-tl_iso_products&p=174683&viewfull=1#post174683 (german).

--- Originally created by michelb on 2012-01-03 16:13:40 (ID 2750)

Toflar commented 11 years ago

Isotope is NOT GOING TO WORK WITH 2.10.4 either! Leo committed the patch for 2.11 only :( I notified him about the error: https://github.com/contao/core/issues/3638

--- Originally created by Toflar on 2012-01-04 17:43:09

aschempp commented 11 years ago

I can confirm the issue. The filter-by-node should be removed anyway, but the problem still exists when a non-admin is logged in.

--- Originally created by andreas.schempp on 2012-01-05 08:42:46

aschempp commented 11 years ago

I have added the field "sorting" to the table as a temporary solution. The bug will be fixed in Contao 2.11. If we drop support for 2.10 we can remove that field.

--- Originally created by andreas.schempp on 2012-01-08 16:18:00