Eigene Bestätigungsseite bzw. Bestätigungsmail

Hallo miteinander,

momentan ist der Hinweis nach Klicken des Bestätigungslinks ja sehr unscheinbar (...ja, ich weiß CSS kann da was machen...).
Schön wäre es, wenn man bei der Bestätigung eine eigene Seite zur Verfügung stellen könnte bzw. nach Bestätigung des Abos nochmal eine Dankes-Mail einrichten könnte.

Für den normalen Newsletter habe ich hierzu auch schon eine Anleitung gefunden.
Meine Frage wäre nun, ist das für Avisota in dieser Form auch möglich, wenn ja - wie?

Vielen lieben Dank schon einmal :-)

Marie

EDIT by tril: Eine Lauffähige Lösung ist hier zu finden: https://www.contao-community.de/show...l=1#post223409

Eine eigene Bestätigungsseite gibt es zur Zeit nicht, das lässt sich aber einbauen.
http://contao-forge.org/issues/1913

Zur Zeit ließe sich aber alles auch über die HOOKs umsetzen.

config.php

PHP-Code:

---

<?php

$GLOBALS['TL_HOOKS']['avisotaSubscribe'][] = array('MyClass', 'hookAvisotaSubscribe');
$GLOBALS['TL_HOOKS']['avisotaActivateSubscribtion'][] = array('MyClass', 'hookAvisotaActivateSubscribtion');

?>

---

MyClass.php

PHP-Code:

---

<?php

class MyClass
{
    function hookAvisotaSubscribe($arrRecipient, $arrTokens)
    {
        foreach ($_SESSION['avisota_subscription'] as $strMessage)
        {
            // break on error
            if (preg_match('#|error$#', $strMessage))
            {
                return;
            }
        }
        // remove the messages, alternative show this messages on the confirmation page
        unset($_SESSION['avisota_subscription']);
        // everythink is allright, redirect to confirmation page
        $this->redirect($this->getPageDetails(X)->row()); // replace X with the page id of the confirmation page
    }

    function hookAvisotaActivateSubscribtion($id, $strEmail, $strToken)
    {
        // send a confirmation email to $strEmail
    }
}

?>

---

füge ich den code von der "config.php" in die config.php des avisota modules ein? und wenn ja, ganz unten oder spielt es keine rolle wo?

und wo finde ich denn die myClass.php? ist es dort dann ebenfalls egal wo ich den code reinbaue?

viele grüsse und danke,
david

Du musst ein eigenes Modul dafür erstellen, die Dateien kommen dann z.B. hierhin:
system/modules/AvisotaCustom/config/config.php
system/modules/AvisotaCustom/MyClass.php

eigenes modul erstellen oder einfach nur die von dir genannten ordner und dort dann die zwei dateien rein? oder wie genau meinst du das?

Zitat:

---

Zitat von tril

system/modules/AvisotaCustom/config/config.php
system/modules/AvisotaCustom/MyClass.php

---

Da stehen doch die Dateipfade wo du die ablegen musst ;)

habe jetzt die pfade erstellt und auch die dateien. die dateien natürlich noch etwas angepasst dass es "Unsubscribtion" heisst:

config.php:

Code:

---

<?php

$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnsubscribe');
$GLOBALS['TL_HOOKS']['avisotaActivateUnsubscribtion'][] = array('MyClass', 'hookAvisotaActivateUnsubscribtion');

?>

---

MyClass.php:

Code:

---

<?php

class MyClass
{
    function hookAvisotaUnsubscribe($arrRecipient, $arrTokens)
    {
        foreach ($_SESSION['avisota_unsubscription'] as $strMessage)
        {
            // break on error
            if (preg_match('#|error$#', $strMessage))
            {
                return;
            }
        }
        // remove the messages, alternative show this messages on the confirmation page
        unset($_SESSION['avisota_unsubscription']);
        // everythink is allright, redirect to confirmation page
        $this->redirect($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page
    }

    function hookAvisotaActivateUnsubscribtion($id, $strEmail, $strToken)
    {
        // send a confirmation email to $strEmail
    }
}

?>

---

dennoch funktioniert es irgendwie nicht. es kommt sowohl im backend als auch im frontend im header folgende fehlermeldung.

Zitat:

---

Warning: Cannot modify header information - headers already sent by (output started at /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/config/config.php:6) in /www/htdocs/w00ef2fd/system/libraries/Template.php on line 285

#0 /www/htdocs/w00ef2fd/system/libraries/Template.php(285): __error(2, 'Cannot modify h...', '/www/htdocs/w00...', 285, Array)
#1 /www/htdocs/w00ef2fd/system/modules/frontend/FrontendTemplate.php(234): Template->output('Vary: User-Agen...', false)
#2 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(171): FrontendTemplate->output()
#3 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate()
#4 /www/htdocs/w00ef2fd/index.php(401): Index->run(Object(DB_Mysql_Result))
#5 {main}


Warning: Cannot modify header information - headers already sent by (output started at /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/config/config.php:6) in /www/htdocs/w00ef2fd/system/libraries/Template.php on line 286

#0 /www/htdocs/w00ef2fd/system/libraries/Template.php(286): __error(2, 'Cannot modify h...', '/www/htdocs/w00...', 286, Array)
#1 /www/htdocs/w00ef2fd/system/modules/frontend/FrontendTemplate.php(234): Template->output('Content-Type: t...')
#2 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(171): FrontendTemplate->output()
#3 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate()
#4 /www/htdocs/w00ef2fd/index.php(401): Index->run(Object(DB_Mysql_Result))
#5 {main}

---

was mache ich falsch bzw. ist falsch?

Klingt schwer danach, als ob du eine Leerzeile/Leerzeichen am Ende deiner PHP Datei, nach dem schließenden ?> Tag hast. Irgendwo wird halt eine Ausgabe erzeugt, die zu dem Fehler führt.

An dem Lag es wohl. Jetzt aber die Frage:
wie wird verwende ich das ganze? Also dass die Bestätigungsseite für die Unsubscription angezeigt wird? muss ich das irgendwie noch einstellen?

Grüsse und danke,
David

Wenn du den Hook in der config.php registriert hast, dann ist der aktiv.

was meinst du mit registriert? habe einfach die zwei dateien von oben angelegt, hochgeladen und sonst nichts ;) reicht das dann schon oder muss ich noch weitere schritte machen, um den "hook zu registrieren"? Ich kenne mich damit halt leider nicht aus...

Das da reicht aus ->

PHP-Code:

---

$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnsubscribe');
$GLOBALS['TL_HOOKS']['avisotaActivateUnsubscribtion'][] = array('MyClass', 'hookAvisotaActivateUnsubscribtion'); 


---

Zitat:

---

Zitat von tril

Das da reicht aus ->

PHP-Code:

---

$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnsubscribe');
$GLOBALS['TL_HOOKS']['avisotaActivateUnsubscribtion'][] = array('MyClass', 'hookAvisotaActivateUnsubscribtion'); 


---

---

du meinst also den code von dir in die datei

Zitat:

---

system/config/config.php

---

eintragen, oder?
Wenn ich das getan habe, wie kann ich dann die weiterleitungsseite für abmelden festlegen?

Also system/config/config.php ist schon mal völlig falsch.

Die richtigen Pfade stehen bereits hier:
https://www.contao-community.de/show...l=1#post206959

Und was du dort rein zu schreiben hast steht hier (in dem Code kannst du auch die Weiterleitungsseite festlegen):
https://www.contao-community.de/show...l=1#post150434

Hmm...also ich fasse jetzt nochmal zusammen was ich bisher getan habe, weil ich irgendwie nicht ganz schlau aus dem ganzen wirr-warr werde, sorry.

1. Modul "Abonnement verwalten" angelegt mit einer Weiterleitungsseite ("Newsletter abonniert")
2. Die o.g. Dateien erstellt:
   config.php:
   
   Code:

   ---

   <?php

$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnsubscribe');
$GLOBALS['TL_HOOKS']['avisotaActivateUnsubscribtion'][] = array('MyClass', 'hookAvisotaActivateUnsubscribtion');

?>

   ---

   und MyClass.php:
   
   Code:

   ---

   <?php

class MyClass
{
    function hookAvisotaUnsubscribe($arrRecipient, $arrTokens)
    {
        foreach ($_SESSION['avisota_unsubscription'] as $strMessage)
        {
            // break on error
            if (preg_match('#|error$#', $strMessage))
            {
                return;
            }
        }
        // remove the messages, alternative show this messages on the confirmation page
        unset($_SESSION['avisota_unsubscription']);
        // everythink is allright, redirect to confirmation page
        $this->redirect($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page
    }

    function hookAvisotaActivateUnsubscribtion($id, $strEmail, $strToken)
    {
        // send a confirmation email to $strEmail
    }
}

?>

   ---

   Die Seite "newsletter gekündigt" hat die ID 61 und dort soll der User dann weitergeleitet werden, wenn er sich über das Modul aus Pkt. 1 vom Newsletter abmeldet.
3. Die zwei Dateien von Punkt 2 wie folgt hochgeladen: system/modules/AvisotaCustom/ für die MyClass.php und system/modules/AvisotaCustom/config/ für die config.php

Jetzt ist die Frage, was ich noch machen muss, damit der User richtig weitergeleitet wird, wenn er auf den Button "Kündigen" klickt von dem Modul aus Punkt 1. Bisher wird er weiterhin auf die weiterleitungsseite des Modul geschickt.

Viele Grüsse,
David

keiner einen hinweis? denke wenn man hier mal eine komplette anleitung hat, die auch für php-nichtkenner taugt, dürfte die frage auch nicht mehr kommen...

Ah sry, hatte deine Antwort übersehen.

Prüfe doch mal bitte, ob er überhaupt in die Methode rein kommt, z.B. durch eine Ausgabe:

PHP-Code:

---

class MyClass
{
    function hookAvisotaUnsubscribe($arrRecipient, $arrTokens)
    {
        var_dump($arrRecipient);

        ...
    }
} 


---

Zitat:

---

Zitat von tril

Ah sry, hatte deine Antwort übersehen.

Prüfe doch mal bitte, ob er überhaupt in die Methode rein kommt, z.B. durch eine Ausgabe:

PHP-Code:

---

class MyClass
{
    function hookAvisotaUnsubscribe($arrRecipient, $arrTokens)
    {
        var_dump($arrRecipient);

        ...
    }
} 


---

---

Hallo tril,

danke für deinen ganzen support und hilfe :) nur ich zähle zu denen in meinem letzten post genannten php-nichtkenner ;) Daher müsste ich wissen:

1. In welche Datei muss der Code?
2. Wo muss die Datei eingebunden werden, falls es eine neue Datei ist?
3. Wie bekomme ich dann die Ausgabe?

Viele Grüsse u Danke,
David

Du sollst einfach nur am Anfang der Methode hookAvisotaUnsubscribe den folgenden Code einfügen:

PHP-Code:

---

var_dump($arrRecipient); 


---

So wie in meinem primitiven Beispiel gezeigt, dass zeigt nämlich die Klasse MyClass die du bereits angelegt hast ;)

so, erledigt. und nun? ;)

Bekommst du bei der Abmeldung eine gesonderte Debug Ausgabe?

nee bekomme nichts. es kommt ganz normal wieder die weiterleitungsseite nach der anmeldung. kannst mal selbst kurz anschauen, wenn du zeit hast:

http://www.beerpong.de/de/service/newsletter.html (tester ¦ tester)

VG,
David

öhm, mach nach dem var_dump mal ein exit(); rein.

passiert das selbe - werde wieder auf die weiterleitungsseite des modul weitergeleitet. der code ist so ja richtig, oder?

Code:

---

<?php

class MyClass
{
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens)
    {
            var_dump($arrRecipient);
                exit();
        foreach ($_SESSION['avisota_unsubscription'] as $strMessage)
        {
            // break on error
            if (preg_match('#|error$#', $strMessage))
            {
                return;
            }
        }
        // remove the messages, alternative show this messages on the confirmation page
        unset($_SESSION['avisota_unsubscription']);
        // everythink is allright, redirect to confirmation page
        $this->redirect($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page
    }

    function hookAvisotaActivateUnSubscribtion($id, $strEmail, $strToken)
    {
        // send a confirmation email to $strEmail
    }
}

?>

---

Ja, die Änderung die du gemacht hast sind richtig, aber der Callback wird nicht aufgerufen.

Du hast aber die aktuellste Version aus dem ER?

Und in der dazugehören config.php steht auch das drin?

PHP-Code:

---

$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnSubscribe'); 


---

PS: benutze doch bitte PHP Code Blöcke :)

Jap genau. Habe die letzte Version aus dem ER. Gerade nochmal extra im Backend geupdated.
die config.php hat ebenfalls den von dir genannten code:

PHP-Code:

---

<?php

$GLOBALS['TL_HOOKS']['avisotaUnSubscribe'][] = array('MyClass', 'hookAvisotaUnSubscribe');
$GLOBALS['TL_HOOKS']['avisotaActivateUnSubscribtion'][] = array('MyClass', 'hookAvisotaActivateUnSubscribtion');

?>

---

Da ist der Fehler, du hast geschrieben:

PHP-Code:

---

$GLOBALS['TL_HOOKS']['avisotaUnSubscribe'][] = array('MyClass', 'hookAvisotaUnSubscribe'); 


---

richtig ist aber

PHP-Code:

---

$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnSubscribe'); 


---

-> avisotaUnsubscribe

na da schau an :) danke. sorry für den fehler. wenn ich mich jetzt abmelde, kommt folgendes im browser:

Zitat:

---

array(5) { ["lists"]=> array(1) { [0]=> int(1) } ["email"]=> string(25) "meine.email@googlemail.com" ["firstname"]=> string(5) "Vorname" ["lastname"]=> string(4) "Nachname" ["gender"]=> string(4) "male" }

---

Ist doch bestimmt ein gutes Zeichen, oder? Wie schaffe ich es jetzt, dass er auf die page id 61 weiterleitet beim klick auf "Kündigen"?

Du musst nur noch den var_dump und exit entfernen :)

habe ich getan. aber es funktioniert nicht.
ich würde dann wieder auf die eingestellte weiterleitungsseite des moduls weitergeleitet ("newsletter-abonniert"). Das steht zumindest in der browserzeile. Doch die seite bleibt weiss, ohne inhalt.

Mäh, mein Fehler, die Zeile

PHP-Code:

---

$this->redirect($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page 


---

muss natürlich so lauten:

PHP-Code:

---

$this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row())); // replace X with the page id of the confirmation page 


---

Hmm...aber auch das geht leider nicht. Das Ergebnis ist das selbe wie oben beschrieben. Habe die Zeile 1:1 ersetzt...

dann setze am Anfang der Funktion bitte nochmal ein

PHP-Code:

---

var_dump($_SESSION['avisota_unsubscription']);
exit; 


---

ein und poste die Ausgabe nochmal.

Die Ausgabe ist einfach

Zitat:

---

NULL

---

als url wird oben wieder die weiterleitungsseite des moduls angezeigt.

Huh, dann versuch mal um das foreach eine IF Bedingung zu packen:

PHP-Code:

---

class MyClass
{
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens)
    {
    if (is_array($_SESSION['avisota_unsubscription'])) {
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage)
            {
                // break on error
                if (preg_match('#|error$#', $strMessage))
                {
                    return;
                }
            }
        }

        // remove the messages, alternative show this messages on the confirmation page
        unset($_SESSION['avisota_unsubscription']);
        // everythink is allright, redirect to confirmation page
        $this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row())); // replace X with the page id of the confirmation page
    }

    function hookAvisotaActivateUnSubscribtion($id, $strEmail, $strToken)
    {
        // send a confirmation email to $strEmail
    }
} 


---

kommt auch wieder das selbe ergebnis wie zuvor. habe gleich die

PHP-Code:

---

var_dump($_SESSION['avisota_unsubscription']);
exit; 


---

noch eingebaut und es kommt wieder NULL

füge das exit; mal am ENDE der Methode ein, also direkt nach dem $this->redirect(...

auch dieses mal kommt wieder das "NULL"

Du hast aber das exit am Anfang der Funktion entfernt?
Füge vor das exit am ENDE auch noch mal ein:

PHP-Code:

---

var_dump($this->generateFrontendUrl($this->getPageDetails(61)->row())); 


---

Mal schauen was da raus kommt.

also der code sieht jetzt so aus:

PHP-Code:

---

<?php

class MyClass
{
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens)
    {
    var_dump($_SESSION['avisota_unsubscription']); 
    if (is_array($_SESSION['avisota_unsubscription'])) {
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage)
            {
                // break on error
                if (preg_match('#|error$#', $strMessage))
                {
                    return;
                }
            }
        }

        // remove the messages, alternative show this messages on the confirmation page
        unset($_SESSION['avisota_unsubscription']);
        // everythink is allright, redirect to confirmation page
        $this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row())); // replace X with the page id of the confirmation page
        var_dump($this->generateFrontendUrl($this->getPageDetails(61)->row()));  
        exit;
    }

    function hookAvisotaActivateUnSubscribtion($id, $strEmail, $strToken)
    {
        // send a confirmation email to $strEmail
    }
} 

?>

---

als ergebnis kommt wiedereinmal "NULL" :(

OK, das sieht schon mal nicht schlecht aus, weil er rennt zumindest in den redirect und beendet darin. Schiebe das letzte var_dump mal VOR das $this->redirect und prüfe, ob die URL die richtige ist. Das var_dump am Anfang kannst du wieder entfernen.

Soeben probiert:

• URL leider immernoch die falsche
• Ausgabe ist leer, d.h. einfach weisse Seite

Also wenn du das so abänderst, ändert sich gar nichts außer einer weißen Seite?

PHP-Code:

---

class MyClass 
{ 
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens) 
    { 
        if (is_array($_SESSION['avisota_unsubscription'])) { 
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage) 
            { 
                // break on error 
                if (preg_match('#|error$#', $strMessage)) 
                { 
                    return; 
                } 
            } 
        } 

        // remove the messages, alternative show this messages on the confirmation page 
        unset($_SESSION['avisota_unsubscription']); 
        // everythink is allright, redirect to confirmation page 
        var_dump($this->generateFrontendUrl($this->getPageDetails(61)->row()));   
        $this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row())); // replace X with the page id of the confirmation page 
        exit; 
    } 
} 


---

Schalte über die Sicherheitseinstellungen vom BE doch mal bitte die Fehlermeldungen AN.

habe die Fehlermeldung jetzt mal angeschaltet. Dann kommt folgende Ausgabe, nach dem klick auf kündigen (falsche URL ist es weiterhin).

PHP-Code:

---

Warning: Missing argument 2 for MyClass::hookAvisotaUnSubscribe(), called in /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php on line 433 and defined in /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/MyClass.php on line 5

#0 /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/MyClass.php(5): __error(2, 'Missing argumen...', '/www/htdocs/w00...', 5, Array)
#1 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(433): MyClass->hookAvisotaUnSubscribe(Array)
#2 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(763): ModuleAvisotaSubscription->unsubscribe(Array)
#3 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleAvisotaSubscription->compile()
#4 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(548): Module->generate()
#5 /www/htdocs/w00ef2fd/system/modules/frontend/ContentModule.php(72): ModuleAvisotaSubscription->generate()
#6 /www/htdocs/w00ef2fd/system/libraries/Controller.php(511): ContentModule->generate()
#7 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(194): Controller->getContentElement('402')
#8 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleArticle->compile()
#9 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(71): Module->generate()
#10 /www/htdocs/w00ef2fd/system/libraries/Controller.php(443): ModuleArticle->generate(false)
#11 /www/htdocs/w00ef2fd/system/libraries/Controller.php(307): Controller->getArticle('59', false, false, 'main')
#12 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(100): Controller->getFrontendModule('0', 'main')
#13 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate(Object(DB_Mysql_Result))
#14 /www/htdocs/w00ef2fd/index.php(401): Index->run()
#15 {main}


Fatal error: Call to undefined method MyClass::generateFrontendUrl() in /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/MyClass.php on line 21 


---

Uh ja, wie dämlich von mir :eek:

Ändere die Klasse und Methode mal folgend ab:

PHP-Code:

---

class MyClass extends Controller
{  
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    {
        ...
    }
} 


---

nun kommt folgender fehler bei falscher url:

PHP-Code:

---

Fatal error: Call to a member function prepare() on a non-object in /www/htdocs/w00ef2fd/system/libraries/Controller.php on line 595 


---

Dann musst du vor dem $this->generateFrontendUrl noch folgendes einfügen:

PHP-Code:

---

$this->import('Database'); 


---

Zitat:

---

Zitat von tril

Dann musst du vor dem $this->generateFrontendUrl noch folgendes einfügen:

PHP-Code:

---

$this->import('Database'); 


---

---

also so?

PHP-Code:

---

$this->redirect($this->import('Database');$this->generateFrontendUrl($this->getPageDetails(61)->row())); 


---

damit kommt die fehlemeldung

PHP-Code:

---

Parse error: syntax error, unexpected ';' in /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/MyClass.php on line 22 


---

also kann der code so nicht stimmen, oder?

Sry, das war eher so gemeint :D

PHP-Code:

---

$this->import('Database');
$this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row())); 


---

ausgabe bei falscher url:

Code:

---

Fatal error: Call to a member function prepare() on a non-object in /www/htdocs/w00ef2fd/system/libraries/Controller.php on line 595

---

Hast Du noch weitere Ideen, damit wir das zum Laufen bringen können? :)

kann es sein dass du VOR dem $this->import('Database') immer noch das var_dump($this->generateFrontendUrl($this->getPageDetails(61)->row())); stehen hast? Ist dann logisch das der Fehler noch kommt :rolleyes:

Ok jetzt klappt es. sowohl die weiterleitung auf die richtige url als auch dass kein Fehler kommt.
Jetzt besteht jedoch das problem, dass der Abonnent nicht aus der Verteilerliste ausgetragen wird. er ist immer noch als "Bestätigt" hinterlegt, was ja bedeutet, dass er weiterhin den Newsletter bekommen würde.

Argh, ja stimmt, weil der Hook vor dem löschen getriggert wird.
Folgendes sollte aber funktionieren:

PHP-Code:

---

class MyClass extends ModuleAvisotaSubscription
{
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    {
        ...
        $this->remove_subscription($arrRecipient['email'], $arrRecipient['lists']);
        $this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row()));
    }
} 


---

dabei sieht jetzt so aus:

PHP-Code:

---

<?php

class MyClass extends ModuleAvisotaSubscription
{  
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    { 
        if (is_array($_SESSION['avisota_unsubscription'])) { 
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage) 
            { 
                // break on error 
                if (preg_match('#|error$#', $strMessage)) 
                { 
                    return; 
                } 
            } 
        } 

        // remove the messages, alternative show this messages on the confirmation page 
        unset($_SESSION['avisota_unsubscription']); 
        // everythink is allright, redirect to confirmation page 
        $this->import('Database');
        $this->remove_subscription($arrRecipient['email'], $arrRecipient['lists']);
        $this->redirect($this->generateFrontendUrl($this->getPageDetails(61)->row())); // replace X with the page id of the confirmation page 
        exit; 
    } 
}  

?>

---

damit kommt dann aber leider wieder eine fehlermeldung:

PHP-Code:

---

Recoverable error: Argument 1 passed to ModuleAvisotaSubscription::__construct() must be an instance of Database_Result, none given, called in /www/htdocs/w00ef2fd/system/libraries/System.php on line 176 and defined in /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php on line 56

#0 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(56): __error(4096, 'Argument 1 pass...', '/www/htdocs/w00...', 56, Array)
#1 /www/htdocs/w00ef2fd/system/libraries/System.php(176): ModuleAvisotaSubscription->__construct()
#2 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(432): System->import('MyClass')
#3 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(763): ModuleAvisotaSubscription->unsubscribe(Array)
#4 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleAvisotaSubscription->compile()
#5 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(548): Module->generate()
#6 /www/htdocs/w00ef2fd/system/modules/frontend/ContentModule.php(72): ModuleAvisotaSubscription->generate()
#7 /www/htdocs/w00ef2fd/system/libraries/Controller.php(511): ContentModule->generate()
#8 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(194): Controller->getContentElement('402')
#9 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleArticle->compile()
#10 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(71): Module->generate()
#11 /www/htdocs/w00ef2fd/system/libraries/Controller.php(443): ModuleArticle->generate(false)
#12 /www/htdocs/w00ef2fd/system/libraries/Controller.php(307): Controller->getArticle('59', false, false, 'main')
#13 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(100): Controller->getFrontendModule('0', 'main')
#14 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate(Object(DB_Mysql_Result))
#15 /www/htdocs/w00ef2fd/index.php(401): Index->run()
#16 {main}


Warning: Missing argument 1 for ModuleAvisotaSubscription::__construct(), called in /www/htdocs/w00ef2fd/system/libraries/System.php on line 176 and defined in /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php on line 56

#0 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(56): __error(2, 'Missing argumen...', '/www/htdocs/w00...', 56, Array)
#1 /www/htdocs/w00ef2fd/system/libraries/System.php(176): ModuleAvisotaSubscription->__construct()
#2 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(432): System->import('MyClass')
#3 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(763): ModuleAvisotaSubscription->unsubscribe(Array)
#4 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleAvisotaSubscription->compile()
#5 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(548): Module->generate()
#6 /www/htdocs/w00ef2fd/system/modules/frontend/ContentModule.php(72): ModuleAvisotaSubscription->generate()
#7 /www/htdocs/w00ef2fd/system/libraries/Controller.php(511): ContentModule->generate()
#8 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(194): Controller->getContentElement('402')
#9 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleArticle->compile()
#10 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(71): Module->generate()
#11 /www/htdocs/w00ef2fd/system/libraries/Controller.php(443): ModuleArticle->generate(false)
#12 /www/htdocs/w00ef2fd/system/libraries/Controller.php(307): Controller->getArticle('59', false, false, 'main')
#13 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(100): Controller->getFrontendModule('0', 'main')
#14 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate(Object(DB_Mysql_Result))
#15 /www/htdocs/w00ef2fd/index.php(401): Index->run()
#16 {main}


Recoverable error: Argument 1 passed to Module::__construct() must be an instance of Database_Result, null given, called in /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php on line 58 and defined in /www/htdocs/w00ef2fd/system/modules/frontend/Module.php on line 73

#0 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(73): __error(4096, 'Argument 1 pass...', '/www/htdocs/w00...', 73, Array)
#1 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(58): Module->__construct(NULL)
#2 /www/htdocs/w00ef2fd/system/libraries/System.php(176): ModuleAvisotaSubscription->__construct()
#3 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(432): System->import('MyClass')
#4 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(763): ModuleAvisotaSubscription->unsubscribe(Array)
#5 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleAvisotaSubscription->compile()
#6 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(548): Module->generate()
#7 /www/htdocs/w00ef2fd/system/modules/frontend/ContentModule.php(72): ModuleAvisotaSubscription->generate()
#8 /www/htdocs/w00ef2fd/system/libraries/Controller.php(511): ContentModule->generate()
#9 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(194): Controller->getContentElement('402')
#10 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleArticle->compile()
#11 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(71): Module->generate()
#12 /www/htdocs/w00ef2fd/system/libraries/Controller.php(443): ModuleArticle->generate(false)
#13 /www/htdocs/w00ef2fd/system/libraries/Controller.php(307): Controller->getArticle('59', false, false, 'main')
#14 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(100): Controller->getFrontendModule('0', 'main')
#15 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate(Object(DB_Mysql_Result))
#16 /www/htdocs/w00ef2fd/index.php(401): Index->run()
#17 {main}


Fatal error: Call to a member function row() on a non-object in /www/htdocs/w00ef2fd/system/modules/frontend/Module.php on line 77 


---

Ok, füge am Anfang einfach noch einen Konstruktor ein:

PHP-Code:

---

class MyClass extends ModuleAvisotaSubscription 
{
    public function __construct()
    {
        $this->import('DomainLink');
        $this->import('FrontendUser', 'User');
    }
    ...
} 


---

Das funktioniert, aber jetzt wird man wieder auf die falsche seite weitergeleitet...

Noch eine Idee? :)

Sry, ich war übers WE nur unterwegs, da hab ich nicht in mein Mail Postfach geschaut *g*

Das Problem ist, dass $this->remove_subscription am Ende selbst einen Redirect macht, aber auch das dürfte sich "hacken" lassen :cool:

Probiere es mal so:

PHP-Code:

---

class MyClass extends ModuleAvisotaSubscription
{
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    {
        ...
        $this->Environment->request = $this->generateFrontendUrl($this->getPageDetails(61)->row());
    }
} 


---

EDIT: Besser ohne den Aufruf von remove_subscription, weil deinem Modul sonst alle Einstellungen fehlen :eek:

meinst du so wie im code unten? Weiss jett nämlich gerade nicht ob ich die public function löschen muss/soll.

PHP-Code:

---

class MyClass extends ModuleAvisotaSubscription
{   
    public function __construct()
    {
        $this->import('DomainLink');
        $this->import('FrontendUser', 'User');
    } 
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    { 
        if (is_array($_SESSION['avisota_unsubscription'])) { 
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage) 
            { 
                // break on error 
                if (preg_match('#|error$#', $strMessage)) 
                { 
                    return; 
                } 
            } 
        } 
        // remove the messages, alternative show this messages on the confirmation page 
        unset($_SESSION['avisota_unsubscription']); 
        // everythink is allright, redirect to confirmation page 
        $this->import('Database');
        $this->Environment->request = $this->generateFrontendUrl($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page 
        exit; 
    } 
} 


---

Fast richtig, nur das exit entfernen :)

PHP-Code:

---

class MyClass extends ModuleAvisotaSubscription
{   
    public function __construct()
    {
        $this->import('DomainLink');
        $this->import('FrontendUser', 'User');
    } 
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    { 
        if (is_array($_SESSION['avisota_unsubscription'])) { 
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage) 
            { 
                // break on error 
                if (preg_match('#|error$#', $strMessage)) 
                { 
                    return; 
                } 
            } 
        } 
        // remove the messages, alternative show this messages on the confirmation page 
        unset($_SESSION['avisota_unsubscription']); 
        // everythink is allright, redirect to confirmation page 
    $this->import('Database');
        $this->Environment->request = $this->generateFrontendUrl($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page 
    } 
} 


---

Mit dem Code werde ich aus dem Newsletter ausgetragen. Jedoch stimmt die URL oben nicht als auch wird eine Fehlermeldung ausgegeben anstatt die URL aufgerufen:

PHP-Code:

---

Runtime notice: Creating default object from empty value in /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/MyClass.php on line 26

#0 /www/htdocs/w00ef2fd/system/modules/AvisotaCustom/MyClass.php(26): __error(2048, 'Creating defaul...', '/www/htdocs/w00...', 26, Array)
#1 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(433): MyClass->hookAvisotaUnSubscribe(Array)
#2 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(763): ModuleAvisotaSubscription->unsubscribe(Array)
#3 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleAvisotaSubscription->compile()
#4 /www/htdocs/w00ef2fd/system/modules/Avisota/ModuleAvisotaSubscription.php(548): Module->generate()
#5 /www/htdocs/w00ef2fd/system/modules/frontend/ContentModule.php(72): ModuleAvisotaSubscription->generate()
#6 /www/htdocs/w00ef2fd/system/libraries/Controller.php(511): ContentModule->generate()
#7 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(194): Controller->getContentElement('402')
#8 /www/htdocs/w00ef2fd/system/modules/frontend/Module.php(140): ModuleArticle->compile()
#9 /www/htdocs/w00ef2fd/system/modules/frontend/ModuleArticle.php(71): Module->generate()
#10 /www/htdocs/w00ef2fd/system/libraries/Controller.php(443): ModuleArticle->generate(false)
#11 /www/htdocs/w00ef2fd/system/libraries/Controller.php(307): Controller->getArticle('59', false, false, 'main')
#12 /www/htdocs/w00ef2fd/system/modules/frontend/PageRegular.php(100): Controller->getFrontendModule('0', 'main')
#13 /www/htdocs/w00ef2fd/index.php(266): PageRegular->generate(Object(DB_Mysql_Result))
#14 /www/htdocs/w00ef2fd/index.php(401): Index->run()
#15 {main} 


---

Ich glaub ich weiß sogar schon was los ist, es gibt kein Environment Objekt ^^

->

PHP-Code:

---

class MyClass extends ModuleAvisotaSubscription
{   
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    { 
        ...
    $this->import('Database');
        $this->import('Environment');
        $this->Environment->request = $this->generateFrontendUrl($this->getPageDetails(61)->row()); // replace X with the page id of the confirmation page 
    } 
} 


---

Also ich meine wir haben es. Der Code sollte jetzt funktionieren :) Ich werde das über die kommenden Tage nochmals ein wenig testen, jedoch glaube ich, dass wir jetzt den richtigen Code gefunden haben :) Daher hier für alle noch kurz die Anleitung wie ihr das auch hinbekommen könnt:

-------------------------------------------------------------------------------------------

1. Weiterleitungsseite für Abmeldung erstellen
Erstelle in der Seitenstruktur eine neue Seite (z.B. erfolgreich abgemeldet) für alle User, die sich vom Newsletter abmelden. Notiere dir von dieser Seite die Page_ID. Diese brauchst Du für Schritt 2.

2. Zwei php-Datei erstellen mit folgendem Inhalt
config.php

PHP-Code:

---

<?php
$GLOBALS['TL_HOOKS']['avisotaUnsubscribe'][] = array('MyClass', 'hookAvisotaUnSubscribe'); 
?>

---

MyClass.php

PHP-Code:

---

<?php

class MyClass extends ModuleAvisotaSubscription
{   
    public function __construct()
    {
        $this->import('DomainLink');
        $this->import('FrontendUser', 'User');
    } 
    function hookAvisotaUnSubscribe($arrRecipient, $arrTokens = false)
    { 
        if (is_array($_SESSION['avisota_unsubscription'])) { 
            foreach ($_SESSION['avisota_unsubscription'] as $strMessage) 
            { 
                // break on error 
                if (preg_match('#|error$#', $strMessage)) 
                { 
                    return; 
                } 
            } 
        } 
        // remove the messages, alternative show this messages on the confirmation page 
        unset($_SESSION['avisota_unsubscription']); 
        // everythink is allright, redirect to confirmation page 
        $this->import('Database');
        $this->import('Environment'); 
        $this->Environment->request = $this->generateFrontendUrl($this->getPageDetails(X)->row()); // replace X with the page id of the confirmation page 
    } 
}

?>

---

Hierbei das "X" ersetzen durch die Page_ID deiner gewünschten Weiterleitungsseite beim Abmelden vom Newsletter.

3. Neue Ordner erstellen für das Modul (die Ordner mit roter Schrift sind neu zu erstellen):
a. /system/modules/ -> AvisotaCustom
b. /system/modules/AvisotaCustom/ -> config

4. erstelle php-Dateien hochladen:
a. Die "MyClass.php" - Datei in den Ordner /system/modules/AvisotaCustom/ laden
c. Die "config.php" - Datei in den Ordner /system/modules/AvisotaCustom/config/ laden

5. Melde Dich via dem Modul "Abonnenten verwalten" vom Newsletter ab (zuvor musst du natürlich angemeldet sein). Dann solltest Du auf die in 1. angelegte Seite weitergeleitet werden (diese vll noch mit Content füllen ;))!

-------------------------------------------------------------------------------------------

Vielen Dank an tril für die viele Unterstützung und die Geduld. Hoffen wir, dass das Ganze jetzt so funktioniert :)

VG,
David

Ich habe den Hook $GLOBALS['TL_HOOKS']['avisotaActivateUnSubscribtion'][] = array('MyClass', 'hookAvisotaActivateUnSubscribtion'); mal entfernt, den gibt es nämlich gar nicht und wird auch nicht benötigt :)