MODUL: Produkterinnerung für modified eCommerce Shopsoftware

[fiveBytes]

[...]
Kann es sein, dass zweimal ein n fehlt bei customers_remind.php?
[...]

Ups, das kann sein kann ich jetzt leider nicht mehr sehen, sorry dafür.

[Karl1]

Hallo fiveBytes,
danke für den Tipp - ist bei GitHub als "unreleased" zu finden.

Gruß Karl

P.S.: Im Code Antwort #448 fehlen zwei geschweifte Klammern vor "else".

[fiveBytes]

Hallo Karl1,

die Klammern sind überflüssig, eigentlich auch beim else-Zweig, da es sich ja beide Male lediglich um einen Einzeiler handelt.

Könnte also ganz schlank auch so aussehen:

Code: PHP  [Auswählen]

if ( defined('MODULE_CUSTOMERS_REMIND_STATUS') && MODULE_CUSTOMERS_REMIND_STATUS=== 'true' )
{
        if (!isset($forbidden_history_sites) || !is_array($forbidden_history_sites)) $forbidden_history_sites = array('customers_remind.php');
        else $forbidden_history_sites[] = 'customers_remind.php';
}

oder wer es lieber mit Umbrüchen und Einrückung mag, auch so:

Code: PHP  [Auswählen]

if ( defined('MODULE_CUSTOMERS_REMIND_STATUS') && MODULE_CUSTOMERS_REMIND_STATUS=== 'true' )
{
        if (!isset($forbidden_history_sites) || !is_array($forbidden_history_sites))
                $forbidden_history_sites = array('customers_remind.php');
        else
                $forbidden_history_sites[] = 'customers_remind.php';
}

Die Klammern sind an sich nur bei Anweisungen erforderlich, die aus mehreren Zeilen bestehen.

Gruß

[HaldOn]

Hallo Karl1,
nochmals vielen Dank für das Modul!

Modul Version 1.0.1
In die Emails
/templates/tpl_modified_nova/mail/german/remind_mail.html und
/templates/tpl_modified_nova/mail/german/remind_mail.txt
würde ich aus rechtlichen Gründen noch

Code: PHP  [Auswählen]

[SIGNATUR]

einfügen.

In der /templates/tpl_modified_nova/mail/german/remind_activate_mail.txt sind noch fehlerhafte Umlaute.

[noRiddle (revilonetz)]

@ fiveBytes
Ich möchte bitten die Kirche im Dorf zu lassen.
Jeder hat seinen Programmierstil und, ob man Klammern dort setzt wo man sie theoretisch auch weglassen könnte oder nicht, gehört dazu. Ist auch eine Frage der Lesbarkeit des Codes, die natürlich auch subjektiv beurteilt wird.
Dein, mit Verlaub, belehrender Post, der nichts zum eigtl. Thema beiträgt, ist also eigtl. überflüssig.

Gruß,
noRiddle

[Tomcraft]

Ich mag mich irren, aber wenn überhaupt, dann "belehrt" er sich doch selber? Es war doch sein eigener Code aus Antwort #448, den er nochmal optimiert hat.

Ich bin übrigens Freund von Klammern aufgrund der Leserlichtkeit und daher wäre ich bei:

Code: PHP  [Auswählen]

if (defined('MODULE_CUSTOMERS_REMIND_STATUS') && MODULE_CUSTOMERS_REMIND_STATUS=== 'true') {
  if (!isset($forbidden_history_sites) || !is_array($forbidden_history_sites)) {
    $forbidden_history_sites = array('customers_remind.php');
  } else {
    $forbidden_history_sites[] = 'customers_remind.php';
  }
}

Grüße

Torsten

[noRiddle (revilonetz)]

Oh, sich selbst belehren ist natürlich immer willkommen.
Dachte Karl sollte belehrt werden, sorry.

Gruß,
noRiddle

[Karl1]

@alle:
Alles gut, fühle mich nicht belehrt.
Umgesetzt habe ich es wie Torsten das geschrieben hat.

@HaldOn:
Danke für den Hinweis und den Tipp, Selbstlaute sind immer wieder ein Problem beim Dateiabgleich.
Die Dateien sind als "unreleased" mit Selbstlauten und Signatur bei GitHub zu finden.

Gruß Karl

[HaldOn]

Hallo Karl1,

bei mir beim Livetest folgender Fehler aufgetreten:
Hatte mich eingetragen, aber Double-Opt-in der Email nicht bestätigt, Uhrsymbol in Kundenerinnerung Empfänger.
Das gewünschte Produkt habe ich in die Erinnerungsliste eingetragen.
Heute hatte ich Warenzulauf des gewünschten Produkts, und wurde dennoch per Email über das Eintreffen  informiert.

Das dürfte aber nach den unbestätigten Double-Opt-in nicht passieren.

Gruß Andre

[DerNachbar]

Ich habe das eben mit einem Testartikel nachgestellt und kann den Fehler bestaetigen.
Die Mails gehen raus, ohne die Einstellung zu beruecksichtigen.

[Karl1]

Hallo Zusammen,
ein neues Release kann bei GitHub heruntergeladen werden.

Das hat sich geändert:
- Kundenerinnerung wurde auch ohne Opt-in versandt - behoben
- deutsche Selbstlaute in E-Mails
- customers_remind.php wird nach dem Login nicht mehr als "letzte aufgerufen Seite" behandelt - danke fiveBytes

Gruß Karl

P.S.: Danke für die Rückmeldungen.

[EDIT Tomcraft 24.04.2024: Modul in Beitrag 1 ergänzt.]

[HaldOn]

Moin Zusammen,

@Karl1
Vielen Dank für die schnelle Änderung!

@all
Zitat aus der Einbauanleitung:

Soll bei jedem Seitenaufruf die Tabelle "Kundenerinnerung" mit dem "Lagerbestand" abgeglichen werden und der Mailversand gestartet werden,
dann die Datei /includes/modules/customers_remind.php mit einem Editor öffnen und in Zeile 19 den Wert ändern in
$sendmail_asap = true;

Beim testen der Version 1.0.2 ist mir aufgefallen, dass trotz Einstellung

Code: PHP  [Auswählen]

if ($sendmail_asap === false) {

die Emails  bei jedem Seitenaufruf versandt werden.
Habe es auch nochmal mit v1.0.1 getestet, da ist genau so.

Vielleicht könnte das mal jemand gegenprüfen und gegebenenfalls bestätigen.

Gruß Andre

[Karl1]

Hallo Andre,
sorry, aber scheinbar war ich zu schnell und unaufmerksam.

Es ist nicht die If-Abfrage in Zeile 19, sondern in Zeile 17 der Wert auf "$sendmail_asap = true;" zu ändern.

Solche Fehler passieren nur, wenn man Anleitungen schreibt. 

Gruß Karl

[HaldOn]

Hallo Karl,

kein Problem.

In der Zeile 17 steht

Code: PHP  [Auswählen]

$sendmail_asap = false; // true - wenn bei jedem Seitenaufruf die Tabelle "Kundenerinnerung" mit dem "Lagerbestand" abgeglichen werden soll

auf "false", es wir aber bei jedem Seitenaufruf versendet nicht einmal am Tag.

Gruß Andre

[Karl1]

Hallo Andre,
an dieser Abfrage habe ich eigentlich nichts verändert und bei mir funktioniert das.

Wichtig ist, dass das false beide Male (Zeile 17 und 19) gleich geschrieben wird.

Einzige Fehlermöglichkeit, die ich sehe ist, dass in Zeile 44 das Datum in der Tabelle 'simulated_cron_records' nicht aktualisiert wird.
Kontrolliere bitte, ob in den Logs Einträge vorhanden sind.

Gruß Karl