Koupon Admin Modul funktioniert nicht richtig

[Rainman12]

Hallo,

seit den letztem Update funktioniert unser Koupon Admin Modul nicht mehr richtig.

Gutscheine kann ich anlegen, aber danach nicht mehr bearbeiten oder deaktivieren und folgende Meldung erscheint bei den Feldern "gültig ab" und "gültig bis" :
 

Code: PHP  [Auswählen]

Deprecated: mktime(): The is_dst parameter is deprecated in /var/www/clients/client1/web46/web/tags/modified eCommerce Shopsoftware-1.05-SP1/admin/coupon_admin.php on line 83

Im übrigen auch im modified eCommerce Shopsoftware-Demoshop!
Gruß
Rainman

Linkback: https://www.modified-shop.org/forum/index.php?topic=17818.0

[franky_n]

Hallo Rainman12,

ab PHP 5.3.0 wirft "mktime" eine deprecated Meldung aus wenn der Wert "is_dst" dort genutzt wird.

Ich sehe zwar dort in Z.83 gerade (auch in der SP1b) keinen mktime Befehl aber Du kannst einfach den letzten Wert des mktime Befehls:

Code: PHP  [Auswählen]

mktime (..,..,..,..,..,..,0);

also das

Code: PHP  [Auswählen]

,0

löschen.

Viele Grüße

Franky

[Rainman12]

Hallo Franky,
danke für den Hinweis, aber ist denn der Demoshop nicht auf der aktuellen Version 1.05b? -> https://demo.modified-shop.org/tags/modified-1.05-SP1

Leider hilft das auch nicht, beim bearbeiten von Coupons, das geht weiterhin nicht.
Hast du vielleicht hier eine Idee?

Gruß
Rainman

[franky_n]

Hallo rainman12,

es gibt mehrere mktime in der Datei, achte mal auf die Zeilenangabe.
Ich vermute das Du dort nach der ersten Fehlerkorrektur eine neue Zeilenangabe hattest.
Wenn der erste mktime verändert wurde und dann der nächste Fehler kommt musst Du dort auch nachbessern.
Der Demoshop läuft halt nicht auf PHP 5.3.0...

Wir wissen das der Shop Probleme mit dieser neuen PHP Version hat, warum das von uns auch öfter hier im Forum erwähnt wird...

Entweder auf PHP 5.2... runter stufen oder versuchen die Fehlermeldungen zu korrigieren.

Viele Grüße

Franky

[Rainman12]

Hallo Franky,

danke für deine schnelle Antwort.

Mein eigentlich und wichtigstes Problem bleibt dennoch das ich Gutscheine (siehe Screenshot) nicht bearbeiten kann und vermutlich wird das nicht mit der PHP Version zusammenhängen. 

Gruß
Rainman

[franky_n]

Hallo Rainman12,

und vermutlich auch nicht mit dem Update!
In der SP1a & SP1b wurde die Datei "coupon_admin.php" überhaupt gar nicht angepasst oder lag dem Update bei!

Viele Grüße

Franky

[Rainman12]

Hy Franky,

mehr oder weniger davor, nämlich von 1.05 auf 1.05 SP1. Mein Fehler, ich habe geschrieben seit dem letzten Update.

Wir haben nun aber die Lösung des Problems selbst gefunden:

In Zeile 1028 wird ein falsches Array abgefragt, es darf nicht "coupon_desc", sondern es muss "coupon_name" abgefragt werden.

Falsch

Code: PHP  [Auswählen]

if ($coupon_desc['coupon_name'] != '') {

Richtig

Code: PHP  [Auswählen]

if ($coupon_name['coupon_name'] != '') {

Gibt es denn einen Grund warum jetzt eine Namensprüfung für das anzeigen der Buttons eingefügt wurde? In den Versionen bis 1.05 konnte man Gutscheine auch ohne Namen vergeben (ich weiß, dass ist nicht schön aber hat funktioniert). All diese älteren Gutscheine kann man jetzt natürlich nicht mehr bearbeiten, da hier die Abfrage positiv ist und keine Buttons erstellt werden.

Gruß
Rainman

[franky_n]

Hallo Rainman12,

[...]
In Zeile 1028 wird ein falsches Array abgefragt, es darf nicht "coupon_desc", sondern es muss "coupon_name" abgefragt werden.
[...]

Nein das "coupon_desc" steht schon etwas weiter drüber und funktioniert auch...

Das hat was mit aktiven und nicht aktiven Coupons zu tun, habe ich mir aber nicht weiter angeschaut.

In Deinem Fall könntest Du einen SQL Befehl ausführen um die Sache zu umgehen:

Code: SQL  [Auswählen]

UPDATE coupons_description SET coupon_name=' ' WHERE coupon_name='';

Dann wird der Name mit einem Leerzeichen befüllt.

Vorher Backup damit nichts schief geht!

Viele Grüße

Franky

[Tomcraft]

Hallo Rainman12,

gib uns hier bitte noch Feedback.

Grüße

Torsten

[Rainman12]

Hi Tomcraft,

sorry, Alter Mann ist kein D-Zug.

Wir haben für uns die Abfrage nun einfach auskommentiert, damit ich meine alten Gutscheine nicht nur sehen, sondern auch bearbeiten und noch verwenden kann. Ist mit Sicherheit nicht die schönste Lösung, aber die praktikabelste und einfachste.

So schaut es nun mal bei mir aus:

Code: PHP  [Auswählen]

//NEW if ($coupon_name['coupon_name'] != '') {
          if ($coupon_active['coupon_active'] != 'N') {
            $contents[] = array('text'=>'<center><a class="button" onclick="this.blur();" href="'.xtc_href_link('coupon_admin.php','action=email&cid='.$cInfo->coupon_id,'NONSSL').'">'.BUTTON_EMAIL.'</a><br />' .
                                '<a class="button" onclick="this.blur();" href="'.xtc_href_link('coupon_admin.php','action=voucheredit&cid='.$cInfo->coupon_id,'NONSSL').'">'.BUTTON_EDIT.'</a> <br />' .
                                '<a class="button" onclick="this.blur();" href="'.xtc_href_link('coupon_admin.php','action=voucherdelete&cid='.$cInfo->coupon_id,'NONSSL').'">'.BUTTON_STATUS_OFF.'</a>' .
                                '<a class="button" onclick="this.blur();" href="'.xtc_href_link('coupon_admin.php','action=voucherreport&cid='.$cInfo->coupon_id,'NONSSL').'">'.BUTTON_REPORT.'</a></center>');
          }
       //NEW }

Gruß
Jan

[Tomcraft]

Was meinen denn die anderen aus dem Team dazu?

Sollen wir nicht den SQL-Befehl in die "update_1.0.5.0_to_1.0.6.0.sql" übernehmen?

[...]

Code: SQL  [Auswählen]

UPDATE coupons_description SET coupon_name=' ' WHERE coupon_name='';

[...]

Grüße

Torsten