Cross Selling Gruppen anzeigen

[demoncleaner]

Erstmal vielen Dank euch beiden.

@karsta.de: Das klingt deutlich besser. Bin aber leider gerade am Versuch gescheitert das duplizierte cross_selling2.php dann auch in das Shopsystem einzubinden. Normalerweise sollte das doch über /includes/extra/filenames/ problemlos gehen. Funzt aber bei mir nicht. Hab ich was übersehen? Ich habs dupliziert, umbenannt und dann in /includes/extra/filenames/ eine neue php Datei angelegt mit

Code: PHP  [Auswählen]

<?php
define('FILENAME_CROSS_SELLING2', 'cross_selling2.php');
?>

@awids: Das Angebot nehme ich gerne an. Wie kommen wir denn da am besten zusammen? Was Modul hat aber nichts mit den Cross-Selling Produkten zu tun oder? Das müsste ich mir dann selber noch basteln, richtig?

[karsta.de]

Alles korrekt, aber da fehlt noch eine weitere Datei in der includes\extra\modules\product_info_end mit dem Eintrag
include (DIR_WS_MODULES.FILENAME_CROSS_SELLING2);

BG Karsta

[demoncleaner]

na klar....muss ja auch irgendwo inkludiert werden  Vielen Dank!! Funzt.

[awids]

[...]
@awids: Das Angebot nehme ich gerne an. Wie kommen wir denn da am besten zusammen? Was Modul hat aber nichts mit den Cross-Selling Produkten zu tun oder? Das müsste ich mir dann selber noch basteln, richtig?

Das Modul ist unabhängig von "Cross-Selling", "Andere Kunden kauften auch...", etc. - die Templatedateien müssen nur entsprechend mit dem Einbau-Variablen laut Anleitung eingebaut werden.

Auf dieser MMLC-Seite erfährst du, wie du mich erreichen kannst. Wenn du den MMLC nicht installieren/nutzen willst, geht das auch ohne, nur profitierst du dann nicht von künftigen Updates.

[demoncleaner]

Kurz nochmal zur Updatesicherheit von karsta.de:
Da ich ja in meiner Variante noch die /includes/classes/product.php abgeändert habe ist das leider nicht updatesicher.
Um es updatesicher zu machen frage ich dann jetzt in der  cross_selling.html bzw. in der cross_selling2.html den Gruppennamen ab:

Code: PHP  [Auswählen]

{if $products_data.GROUP != '' && strpos($products_data.GROUP, 'Zubeh') !== false}

Nicht das gelbe vom Ei, da es eben dann in Englisch noch ein strpos braucht und nicht variabel ist. aber wenigstens updatesicher.
Ich glaube man kommt ansonsten nicht easy an die px.products_xsell_grp_name_id ran oder?
Oder man macht dafür halt extra einen modifier ... ? Aber das war mir zu doof, weil der Wert ja eigentlich schon da wäre.

[karsta.de]

Wenn du der Meinung bist, dass die Verwendung von  'GROUP_ID' => $cross_sells['products_xsell_grp_name_id']; einen Vorteil für die Shopsoftware und die Shopbetreiber darstellt, dann mach doch dafür ein Ticket auf, damit das vielleicht als Standard im Shop implementiert wird.

BG Karsta

[karsta.de]

Ansonsten fiele mir nur noch die Möglichkeit der Klassenerweiterung ein, aber das Einfachere wäre, wie von dir vorgeschlagen, dafür die includes/classes/product.php anzupassen und die GROUP_ID verfügbar zu machen.

[noRiddle (revilonetz)]

Wenn man sich die Funktion getCrossSells() in der product-Klasse anschaut müsste die Gruppen-ID doch bereits zur Verfügung stehen, und zwar als Schlüssel der Array-Elemente. Somit müsste man in der Template-Datei cross_selling.html aus dem

Code: XML  [Auswählen]

{foreach name=aussen item=products_data from=$module_content}

das machen können

Code: XML  [Auswählen]

{foreach name=aussen key=grp_id item=products_data from=$module_content}

und müsste dann so abfragen können:

Code: XML  [Auswählen]

{if $grp_id = 1}...{/if}

Somit bedürfte es keinerlei Änderungen, Anpassungen und Datei-Kopieren.
Kann's gerade nicht testen, probier mal aus.

Gruß,
noRiddle

[karsta.de]

Ja, super Idee. 
Aber muss heißen:

Code: PHP  [Auswählen]

{if $grp_id == 1}...{/if}

BG Karsta

[noRiddle (revilonetz)]

Upps, right. Danke für die Korrektur.

Gruß,
noRiddle

[demoncleaner]

Vielen Dank noRiddle und karsta.de. Funktioniert 1a. Hab´s gerade getestet.

[karsta.de]

@noRiddle

Somit bedürfte es keinerlei Änderungen, Anpassungen und Datei-Kopieren.

Naja, um die einzelnen CrossSellingGruppen an unterschiedlichen Stellen auf der Produkt-Seite anzuzeigen (um das nicht mit JavaScript zu tun) ist es meiner Meinung nach schon nötig zusätzliche Dateien zu erstellen. Wenn man nur eine andere Darstellung der einzelnen Gruppen haben will, ist dein Vorschlag top.

BG Karsta