Isotope 2.10.3 + 1.3 Beta 2 - Problem mit tl_iso_products

**Wian** · 15.11.2011, 12:02

Ergänzung 06.12.2011: Bitte lest das Thema bis ganz zum Schluss! Die gefundene Lösung mit dem "sorting" war nicht korrekt! Die B2 von Isotope läuft maximal unter 2.10.2!
Ergänzung 11.01.2012: Bitte lest bis ganz zum Schluss!

Hallo Forum!

Kämpfe gerade mit der tl_iso_products. Wenn ich im Backend auf Produkte anlegen gehe, kommt folgende Fehlermeldung:

Code:

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)

Im englischsprachigen Forum habe ich eine Info gefunden http://www.contao-community.org/view...hp?f=36&t=3623 , wo man das Feld "sorting" manuel ergänzen soll. Das hatte ich gemacht und es kam eine Fehlermedlung mit unkorrektem Syntax. Ich habe jetzt ewig auf den Code gestarrt und nix gefunden. Also habe ich die alte Datenbank wieder installiert und will nun nochmal von vorn beginnen.

Hat irgendjemand einen Tipp, was falsch war:

Code:

-- Tabellenstruktur fÃ¼r Tabelle `tl_iso_products`
--

CREATE TABLE IF NOT EXISTS `tl_iso_products` (
  `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
  `pid` int(10) unsigned NOT NULL DEFAULT '0',
  `sorting` int(10) unsigned NOT NULL DEFAULT '0',
  `gid` int(10) unsigned NOT NULL DEFAULT '0',
  `tstamp` int(10) unsigned NOT NULL DEFAULT '0',
  `type` int(10) unsigned NOT NULL DEFAULT '0',
  `language` varchar(2) NOT NULL DEFAULT '',
  `pages` blob,
  `inherit` blob,
  `alias` varchar(128) NOT NULL DEFAULT '',
  `sku` varchar(128) NOT NULL DEFAULT '',
  `name` varchar(255) NOT NULL DEFAULT '',
  `images` blob,
  `teaser` text,
  `description` text,
  `keywords_meta` text,
  `description_meta` text,
  `price` decimal(12,2) NOT NULL DEFAULT '0.00',
  `shipping_weight` varchar(255) NOT NULL DEFAULT '',
  `shipping_exempt` char(1) NOT NULL DEFAULT '',
  `tax_class` int(10) unsigned NOT NULL DEFAULT '0',
  `archive` int(1) unsigned NOT NULL DEFAULT '0',
  `published` char(1) NOT NULL DEFAULT '',
  `start` varchar(10) NOT NULL DEFAULT '',
  `stop` varchar(10) NOT NULL DEFAULT '',
  `displaytausch` varchar(255) NOT NULL DEFAULT '',
  PRIMARY KEY (`id`),
  KEY `gid` (`gid`),
  KEY `pid` (`pid`,`language`,`published`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;

Oder muss man das sorting-Feld gar nicht einfügen??? Im deutschen Forum habe ich dazu nichts gefunden.

Danke im Voraus!

Antje

***lucina*** · 15.11.2011, 12:51

Für mich hat der Hinweis aus dem internationalen Forum mit dem Sorting-Feld gerade einwandfrei funktioniert. Hast Du es über die Ergänzung der entsprechenden database.sql des Moduls probiert oder direkt in der Datenbank?

Carolina.

**Wian** · 15.11.2011, 13:12

Ich habe den Code über den Editor eingefügt und wieder hochgeladen und dann die install.php aufgerufen. Nun bekomme ich beim Klick auf Produkte folgenden Fehler:

Code:

Fatal error: Uncaught exception Exception with message Query error: 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 ') ORDER BY FIND_IN_SET(pid, ''), sorting' at line 1 (SELECT id, pid FROM tl_iso_products WHERE pid IN() ORDER BY FIND_IN_SET(pid, ''), sorting) thrown in .../system/libraries/Database.php on line 677

Wie hast du die sql-Datei geändert? Über das Backend? Ich muss gestehen, dass ich erst seit 1 Jahr mit Contao arbeite und mir einige Dinge bei Fehlermeldungen ziemliches Kopfzerbrechen bereiten.

Grüsse!

Antje

***Nina*** · 15.11.2011, 13:19

Ich habe die Datei /system/modules/isotope/config/database.sql mit einem UTF-8 fähigen Browser geöffnet und am Anfang die nachfolgend markierte Zeile eingefügt:

Code:

CREATE TABLE `tl_iso_products` (
  `id` int(10) unsigned NOT NULL auto_increment,
  `pid` int(10) unsigned NOT NULL default '0',
  `sorting` int(10) unsigned NOT NULL default '0',
  `gid` int(10) unsigned NOT NULL default '0',
  `tstamp` int(10) unsigned NOT NULL default '0',

Dann habe ich im Backend die Erweiterungsverwaltung aufgerufen und bin auf den Link "Datenbank aktualisieren" gegangen. Dort zeigt Contao an, dass es nun das neue Feld hinzufügen will. Das habe ich bestätigt. Danach kann ich wieder normal auf Isotope zugreifen.

**Wian** · 15.11.2011, 14:22

Also ich habe die Lösung noch nicht gefunden.

Bevor der Fehler kam, hatte ich schon ein Produkt eingepflegt. Als der Fehler dann kam (nach dem Update auf 2.10.3) habe ich den Produkteintrag in der Datenbank gelöscht (direkt vor Ort) und unter operations wieder auf 1 gesetzt. - weil ich dachte, dass ich erstmal die Shop-Konfiguration machen muss, bevor ich ein Product anlegen darf. Sicherlich liegt es an diesem manuellen Löschen in der Datenbank, dass es jetzt nicht funktioniert. Geduld ist leider nicht meine Stärke. Irgendwie müßte man alles nochmal neu machen.

Danke für Eure Hilfe. Es ist gut zu wissen, dass es bei anderen läuft. Zudem hoffe ich auf ein baldiges RC1.

Grüsse!

Antje

***Nina*** · 15.11.2011, 14:54

Die "Lösung" aus dem englischen Forum behebt das Problem nicht wirklich. Der Fehler taucht zwar im backend nicht gleich wieder auf, tritt aber z. B. im Frontend in der Listenansicht auf und kann ev. an anderen Stellen noch vorkommen.

Es gibt bereits ein Ticket dazu, das hoffentlich möglichst schnell zu einer richtigen Lösung führt.

**Wian** · 15.11.2011, 18:55

Also alles wieder zurück! Und dabei war der Fehler dann weg. Mein Mann - ein Programmierer mit heutigem Erstkontakt zu Contao - hat sich solange durch den Code gewühlt, bis er letztlich in der Controller.php was geändert hat. (get Child Records) Aber er meinte, er hätte das Problem damit nur umgangen und nicht gelöst. Und beim nächsten Update wäre es auch wieder weg. Schade!

Kann man dann eigentlich einen Shop problemlos von 0.2.11 auf 1.3 updaten?

Grüsse!

Antje

P.S. Seit fast einem Jahr suche ich nun nach einer Shoplösung. Im Februar habe ich Isotope getestet und bin dann wegen den rechtlichen Dingen wieder zurück zu xtcommerce. Bei xtcommerce vermisste ich die vielen netten Contao-Dinge. Dann habe ich ein Projekt mit dem Contao-Webshop getestet - aber der Demoshop von Isotope hat mir dann doch besser gefallen. Nun möchte ich gern doch mit Isotope arbeiten, aber wo beginnen? 2.10 oder doch 2.9.5 mit 0.2.11. Für einen kleinen Tipp wäre ich dankbar.

**Toflar** · 16.11.2011, 21:56

Ich hab das Ticket heute aktualisiert. Mit entsprechender Beschreibung des Problems, temporärer Lösung und Bugreport im offiziellen Tracker.

**bizzyd** · 05.12.2011, 14:03

Zitat von Nina

Ich habe die Datei /system/modules/isotope/config/database.sql mit einem UTF-8 fähigen Browser geöffnet und am Anfang die nachfolgend markierte Zeile eingefügt:

Code:

CREATE TABLE `tl_iso_products` (
  `id` int(10) unsigned NOT NULL auto_increment,
  `pid` int(10) unsigned NOT NULL default '0',
  `sorting` int(10) unsigned NOT NULL default '0',
  `gid` int(10) unsigned NOT NULL default '0',
  `tstamp` int(10) unsigned NOT NULL default '0',

Dann habe ich im Backend die Erweiterungsverwaltung aufgerufen und bin auf den Link "Datenbank aktualisieren" gegangen. Dort zeigt Contao an, dass es nun das neue Feld hinzufügen will. Das habe ich bestätigt. Danach kann ich wieder normal auf Isotope zugreifen.

Danke

**Toflar** · 05.12.2011, 19:27

Nicht! Das ist, wie im Ticket beschrieben, nicht die Lösung. Ich hab alles im Ticket beschrieben.

**Kahmoon** · 16.12.2011, 19:12

Servus beisammen,

mich hats jetzt auch damit erwischt....irgendwie werde ich aus den Lösungvorschlägen hier nicht schlau

tl_iso_products.php habe ich wohl schon die Aktuelle. Die Zeile wie oben beschrieben gibt es dort nicht. Wo krieg ich die gepatchte DC_TABLE (http://www.contao-forge.org/issues/2451)?

Gruß
Frank

**Kahmoon** · 29.12.2011, 17:57

Darf ich hier noch mal nachhaken?

**andreas.schempp** · 30.12.2011, 07:16

Wie du im Ticket nachlesen kannst, wird der Fehler in der nächsten Version von Contao behoben. Isotope wird nicht mit Contao 2.10.3 funktionieren, wir bauen keinen Spezialpatch für eine alte Version ein ;-)

**michelb** · 03.01.2012, 13:09

Ich hab den Fehler im RC1 Build 15 und Contao 2.10.4 wieder drin. :/

**Toflar** · 03.01.2012, 13:55

Genau derselbe Fehler ist unmöglich. Evtl. andere Fehler, aber wenn Du genau den gleichen hast, läuft nicht Version 2.10.4

**michelb** · 03.01.2012, 14:35

Ich habe nochmal alle Dateien mit denen aus 2.10.4 ersetzt (außer die config-Dateien) und das Install-Tool aufgerufen, RC1 von Isotope ist noch immer Build 15:

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

#0 […]/system/libraries/Database.php(636): Database_Statement->query()
#1 […]/system/libraries/Database.php(164): Database_Statement->execute()
#2 […]/system/libraries/Controller.php(2811): Database->execute('SELECT id, pid ...')
#3 […]/system/libraries/Controller.php(2953): Controller->getChildRecords(0, 'tl_iso_products', true)
#4 […]/system/drivers/DC_Table.php(225): Controller->eliminateNestedPages(Array, 'tl_iso_products', true)
#5 […]/system/drivers/DC_ProductData.php(71): DC_Table->__construct('tl_iso_products')
#6 […]/system/modules/backend/Backend.php(210): DC_ProductData->__construct('tl_iso_products')
#7 […]/contao/main.php(101): Backend->getBackendModule('iso_products')
#8 […]/contao/main.php(286): Main->run()
#9 {main}

Keine Ahnung was da noch schiefläuft.

**michelb** · 03.01.2012, 15:00

Aha, lag daran, dass ich was in der Seitenstruktur ausgewählt hatte, siehe http://www.contao-forge.org/issues/2451)#note-8 :

This error occurs in backend only, if you have the page/article tree filtered by a specific node.

Bug-Report kommt.

**andreas.schempp** · 03.01.2012, 19:15

Der Installations-Fehler müsste nun behoben sein.

**deerwood** · 07.01.2012, 22:52

Weil Leo gesagt hat, dass es wahrscheinlich keine 2.10.5 Version geben wird; nur noch mal zur Sicherheit die Frage: wenn ich DC_Table.php in Contao 2.10.4 so patche, wie in den beiden Commits Consistent "hasSorting" behaviour #3638 und Consistent "hasSorting" behavior fix #3638 zusammen, also so (siehe auch Textfarben rechts im diff):

Code:

--- DC_Table.php-revBASE.svn000.tmp.php    Sa  7. Jan 21:22:48 2012
+++ system/drivers/DC_Table.php    Sa  7. Jan 21:19:46 2012
@@ -210,7 +210,7 @@
              // Unless there are any root records specified, use all records with parent ID 0
             if (!isset($GLOBALS['TL_DCA'][$table]['list']['sorting']['root']) || $GLOBALS['TL_DCA'][$table]['list']['sorting']['root'] === false)
             {
-                $objIds = $this->Database->prepare("SELECT id FROM " . $table . " WHERE pid=?" . ($this->Database->fieldExists('sorting', $strTable) ? ' ORDER BY sorting' : ''))
+                $objIds = $this->Database->prepare("SELECT id FROM " . $table . " WHERE pid=?" . ($this->Database->fieldExists('sorting', $table) ? ' ORDER BY sorting' : ''))
                                          ->execute(0);
 
                 if ($objIds->numRows > 0)
@@ -222,7 +222,7 @@
             // Get root records from global configuration file
             elseif (is_array($GLOBALS['TL_DCA'][$table]['list']['sorting']['root']))
             {
-                $this->root = $this->eliminateNestedPages($GLOBALS['TL_DCA'][$table]['list']['sorting']['root'], $table, true);
+                $this->root = $this->eliminateNestedPages($GLOBALS['TL_DCA'][$table]['list']['sorting']['root'], $table, $this->Database->fieldExists('sorting', $table));
             }
         }

dann sollte ich Isotope problemlos von "1.3.0 beta2 Build 17" zu "1.3.0 rc1 Build 20" aktualisieren können und Isotope sollte dann laufen?

Danke, Georg

**Toflar** · 07.01.2012, 22:54

Ja, das ist richtig Georg

**andreas.schempp** · 08.01.2012, 12:38

Oder du fügst einfach das Feld "sorting" in die entsprechende Tabelle ein... Wird zwar nicht genutzt, aber der Fehler ist dann weg.

**deerwood** · 08.01.2012, 16:50

Zitat von andreas.schempp

Oder du fügst einfach das Feld "sorting" in die entsprechende Tabelle ein... Wird zwar nicht genutzt, aber der Fehler ist dann weg.

Uh, äh, hmmm, wenn das richtig ist, dann verstehe ich gar nichts mehr, siehe besonders Beiträge #1 (erster Absatz), #6 und #10 in dieser Diskussion. Und vor allem verstehe ich dann nicht, warum Ihr Isotope nicht das Feld mitgebt und den Shop stattdessen dick und fett als nicht kompatibel mit 2.10.3 und 2.10.4 deklarieren müsst?

LG, Georg

**Toflar** · 08.01.2012, 18:06

Weil der Fix erst in der RC2 kommt

Warum das mit dem Feld "sorting" hinzufügen jetzt plötzlich funktionieren soll, muss ich allerdings Andreas beantworten lassen. Ich meine jemand hätte da dann Probleme im FE gehabt.

**andreas.schempp** · 08.01.2012, 21:23

Oh, echt? Dessen war ich nicht bewusst. Das Feld ist Isotope egal... dachte ich?

Bezüglich "warum nicht bisher", wir dachten das Problem ist mit Contao 2.10.4 behoben. Weil das nun nicht geschehen ist, werden wir ihn umgehen müssen.