isotope / core

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

Warum schafft ihr den Options Wizard ab? #1468

Open felixpfeiffer opened 9 years ago

felixpfeiffer commented 9 years ago

Der Options Wizard wird in den Attribute-Einstellungen als deprecated angezeigt? Warum?

Nur dieser bietet die Möglichkeit einfach freie Wert-Label-Paare zu definieren.

Beim Options-Manager, der nebenbei gesagt echt cool ist, kann man nur ein Label definieren, aber keinen extra Wert.

In meinem Fall ist es so, dass ich die Auswahl des Attributs als CSS-Klasse verwenden möchte. Und das bei mehreren Eigenschaften eines Produkts. Dabei heißen die Klassen aber alle gleich. Beim Options Wizard kann ich einfach die Klasse und ein passendes Label definieren und die Klasse im Frontend-Template ausgeben lassen.

Beim Options Manager müsste ich eine extra Routine haben, die die ID der Option in eine entsprechende CSS-Klasse umwandelt. Das könnte ich aber nur im Template umsetzen.

Meine Frage nun also, warum den Wizard rauswerfen, der firsst doch kein Brot :-D

aschempp commented 9 years ago

Gutes Argument… :-)

aschempp commented 9 years ago

@Toflar wie siehst du das?

Toflar commented 9 years ago

Kann man schon drin lassen, ist einfach viel weniger mächtig, da keine Übersetzungen etc. Aber wem's dient... :)

aschempp commented 9 years ago

Übersetzungen gehen schon (mittels dem Übersetzungs-Manager von Isotope). Dann werde ich das "deprecated" entfernen :-)

aschempp commented 9 years ago

Hmm, ich habe mir gerade den Code angeschaut, und es gibt einige "komische" Routinen nur um die Options Wizard zu unterstützen. Vielleicht wäre es sinnvoller ein "Value"-Feld beim Manager hinzuzufügen, oder seht ihr da ein Problem/Nachteil?

felixpfeiffer commented 9 years ago

Das wäre die andere Variante. Wie wäre das mit der Abwärtskompatibilität?

Ansonsten finde ich den Wizard schön einfach und schnell zu konfigurieren. Keine Kindtabelle usw. Eben wirklich für einfach Wert-Label-Paare. Und Mehrsprachigkeit bekommt man da ja zur Not auch mit Insert-Tags hin.