Bug im Gutscheinnodul

[Pseiko]

Hallo,

im Gutscheinmodul steckt ein Bug.
Auch in der Demo-Version passiert.

Wenn man einen Gutschein mit einem Tag Gültigkeit anlegt.
Und man springt dann in den Gutscheinmasken umher, wird die Gültigkeit bei jedem Seitenaufruf um einen Tag verlängert.

Oder man drückt einfach mehrmals hintereinander speichern und er verlängert sich immer wieder um einen Tag.

Jetzt weiß ich auch warum ich mich bei der 1.6er Version schon mal gewundert hatte dass die Gutscheinen über das Monatsende hinaus gültig waren.

Gruß
Pseiko

[schokoladentiger69]

Richtig, fiel mir auch auf bei der Gestaltung des diesjährigen "Adventskalenders" mit täglicher Gutscheinaktion. Und bei mir geht es auch nicht, dass nur einzelne Artikel mit Gutscheincode rabattiert werden, es muss die Kategorie sein. Wenn nur ein Artikel gewählt ist sagt das Modul bei der Prüfung im letzten Bestellschritt, dass kein gültiger Artikel ausgewählt wäre, was definitiv aber nicht stimmte.

[ablass]

@Pseiko

Konnte ich nachvollziehen, lässt sich relativ einfach beheben:
In der Datei functions.mod.gutscheinaktionen.inc.php etwa in Zeile 1062 die Zeile:

Code: Alles auswählen

        $GueltigBis = date("Y-m-d",mktime(0,0,0,$date[1],$date[0]+1,$date[2]))."\n";


so abändern:

Code: Alles auswählen

        $GueltigBis = date("Y-m-d",mktime(23,59,59,$date[1],$date[0],$date[2]))."\n";


@schokoladentiger69

Welche Shopversion verwendest du? Handelt es sich bei den Artikeln evtl. um Merkmalkombinationen ? Da gab es einen Bug der mit der Version 1.7 gefixt wurde.

[schokoladentiger69]

@ablass: Öhm, ehrlich gesagt weiß ich die Version gar nicht, kann ich das irgendwo sehen? Der letzte mitgemacht Update war der auf diese Bürgel-Boniprüfgeschichten, danach nur noch Eigenanpassungen. Shop wurde in 2009 installiert, das Gutscheinmodul kam dieses Jahr dazu (Version mit Shopgutscheinen).
Ja, es handelt sich um Artikel mit Merkmalkombinationen, ich hab fast nur solche Von daher hab ich mir spontan eben über die Kategorien geholfen aber wenn es da ne Möglichkeit wäre das natürlich nicht schlecht.

[mmaass]

Im Quelltext ganz untern sowie ab der V1.5 im Admin unter Einstellungen -> System

[ablass]

Ok wenn du selber Anpassungen gemacht hast ist natürlich ungünstig mit nem Update auf die 1.7.
Der Fix ist aber auch sehr übersichtlich... in functions.mod.gutscheinaktionen.inc.php in der Funktion CheckGutschein() müsste so etwa um die Zeile 206 herum folgendes stehen:

Code: Alles auswählen

         if ($GutscheinAktionObject->gueltigkeitsbereich == 2) {
            
            // Artikel aus dem Warenkorb
            $SQLString = "SELECT ";      
            $SQLString .= TABLE_WARENKORB . ".artikel_id ";      
            $SQLString .= "FROM ";      
            $SQLString .= TABLE_WARENKORB . " ";      
            $SQLString .= "WHERE ";      
            $SQLString .= TABLE_WARENKORB . ".session = '" . $Session . "' ";      
            
            $MySQLQueryReference = errorlogged_mysql_query($SQLString);
            
            while ($WarenkorbArtikelRow = mysql_fetch_array($MySQLQueryReference)) {
               
               $WarenkorbArtikelIDArray[] = $WarenkorbArtikelRow["artikel_id"];
               
            }


das muss so geändert werden:

Code: Alles auswählen

         if ($GutscheinAktionObject->gueltigkeitsbereich == 2) {
            
            // Artikel aus dem Warenkorb
            $SQLString = "SELECT ";      
            $SQLString .= TABLE_WARENKORB . ".artikel_id ";      
            $SQLString .= ', ' . TABLE_ARTIKEL . '.merkmalkombinationparentid ';
            $SQLString .= "FROM ";      
            $SQLString .= TABLE_WARENKORB . " ";      
            $SQLString .= ' INNER JOIN ' . TABLE_ARTIKEL . ' ON ' . TABLE_WARENKORB . '.artikel_id = ' . TABLE_ARTIKEL . '.id ';
            $SQLString .= "WHERE ";      
            $SQLString .= TABLE_WARENKORB . ".session = '" . $Session . "' ";      
            
            $MySQLQueryReference = errorlogged_mysql_query($SQLString);
            
            while ($WarenkorbArtikelRow = mysql_fetch_array($MySQLQueryReference)) {
               if($WarenkorbArtikelRow['merkmalkombinationparentid'] == 0)
                  $WarenkorbArtikelIDArray[] = $WarenkorbArtikelRow["artikel_id"];
               else
                  $WarenkorbArtikelIDArray[] = $WarenkorbArtikelRow['merkmalkombinationparentid'];
            }


dann sollte es eigentlich schon tun.

[schokoladentiger69]

@mmaas: Danke. Dann ist es 1.5.1 laut Systeminfo. Die Anpassungen habt ja eh ihr gemacht und von daher (danke @ablass trotzdem!) halt ich mich da raus - besser ist das - bevor ich mir wieder aufs WE den Shop komplett lahmlege
@ablass: Ich hab nämlich keine Ahnung von der Materie Bin nur Anwender und hab aus meinen Fehlern (s. Lahmlegen Shop am WE) gelernt!

[mmaass]

Hi Schokoladentiger, schick mir mal kommende Woche bitte eine Mail drüber, dann bau ich es bei Dir ein.

[schokoladentiger69]

@mmaass: Ja ist denn scho Weihnachten? Super, ich freu mich, danke fürs Angebot - nehm ich doch gerne an und meld mich nächste Woche!