Suche

[gabin]

Hi allesamt,

einen kurzen Vorschlag.
Mein Vorschalg bezieht sich auf die folgende Zuweisung:
cgiLinkM("index", "def", _new_clickhere); der function Action - Modul Newsletter - Zeile 86

Wenn man einen Newsletter bei pragmaMX abonniert, erscheint folgende Meldung:

Wir haben Ihnen ein Bestätigungsmail geschickt!
Bitte klicken Sie hier

Nun das Problem ist, meiner Meinung nach,  dass der linke- und der rechte Block nicht angezeigt werden.
Das lässt sich hier auf der Webseite von pragmaMX nachweisen.

Mit einiges Ausprobieren konnte ich es hinkriegen, dass beide Blöcken zu sehen sind.

Beispiel:

if (!eregi("^[0-9a-z]([-_.]?[0-9a-z])*@[0-9a-z]([-.]?[0-9a-z])*\\.[a-z]{2,4}$", $new_email)) {
        cgiCenter();
        echo "" . _new_noemail . "\n";
        cgiBR();
        cgiLinkM("index", "def", _new_clickhere);
        cgiCenter(1);
        CloseTable();
        return;
    }

Wie folgt abändern für jede if

if (!eregi("^[0-9a-z]([-_.]?[0-9a-z])*@[0-9a-z]([-.]?[0-9a-z])*\\.[a-z]{2,4}$", $new_email)) {
       
cgiCenter();       
#cgiBR();
       
    include_once("header.php");

      echo "" . _new_noemail . "\n";
      cgiLinkM("index", "def", _new_clickhere);

    include ("footer.php");

cgiCenter(1);

CloseTable();

return;
}

Es sind 4 Änderungen bei der function Action($new_email, $new_sub, $new_type)( Zeile 86)

das betrifft die die folgenden Sprachkonstanten:
" . _new_noemail . ",  " . _new_already . ", ". _new_senok." und "._new_unsbuok."

und 2 bei der function Confirm($new_email, $new_check)(Zeile 236)

das betrifft folgende Sprachkonstanten:
" . _new_subnot . " und " . _new_subok . "

Nach der Änderungen sind die Blöcke zu sehen.

Gruß
gabin

[jubilee]

Hallo !
Du hast recht.
Es reicht jedoch in dieser Funktion Action()
jeweils VOR die return ein

include_once("footer.php");
zu setzen.
Beipspiel :

cgiCenter(1);
        CloseTable();
        include_once("footer.php");       
   return;

der header wird ja oben eingebunden. Das Problem ist, das wegen den return's der footer nicht eingebunden wird.

[jubilee]

Hier mal eine geänderte modules/Newsletter/index.php zum ausprobieren:


MfG
jubilee

[gelöscht durch Administrator]

[gabin]

Hi jubilee,

Hier mal eine geänderte modules/Newsletter/index.php zum ausprobieren:

Es funktioniert. 
Beide Blöcke sind zu sehen.

Nur in Datei modules/Newsletter/index.php function Admin() fehlen folgende Anweisungen:

   $send = (!isset($send)) ? "" : $send;
   $sub = (!isset($sub)) ? "" : $sub;
   $text = (!isset($text)) ? "" : $text;
   $htmltext = (!isset($htmltext)) ? "" : $htmltext;

Wenn diese Anweisungen nicht da stehen, dann sind folgende Fehler im Adminmenü zu sehen:

Undefined variable: send
Undefined variable: sub
Undefined variable: text
Undefined variable: htmltext

Diese Anweisungen hatte ich von Dir von einen anderem Tread erhalten.

Muster: soneros dot de

Gruß
gabin

[jubilee]

Ja, hab ich auch gerad gesehen, da das noch fehlt.
Ist wieder mal etwas für das nächste Release.
Wollt das Newslettermodul eh nochmal durchsehen ....

[Andi]

Hi

Ist wieder mal etwas für das nächste Release.

Spielst du das bitte gleich ins CVS, damit wir es nicht vergessen 

[jubilee]

Spielst du das bitte gleich ins CVS

Schon erledigt .... :-*


MfG
jubilee