Nach Absenden eines Formulas das Mitgliederkonto schliessen?

**dirksche** · 12.10.2018, 16:08

Hallo liebe Community,

ich habe in einem geschützten Bereich ein Formular eingebunden.
Nach dem Absenden des Formulars soll das Mitgliederkonto deaktiviert / geschlossen werden. Wie stelle ich das am besten an?
Ich denke mit dem processFormData Hook wäre dies möglich, oder?

Kann mit jemand einen Tipp geben, wie ich mittels processFormData Hook das Mitgliederkonto deaktiveren kann?

Grüsse,
Dirk

**fusch** · 13.10.2018, 13:25

Hallo Dirk,

wir haben neulich auch bei Absenden eines Formulars mittels Hook die Datenbank verändert. Evlt. kannst Du das für Dich nutzen, ist hier dokumentiert:
https://www.contao-bayern.de/newsrea...on-center.html
Wir haben allerdings den prepareFormData-Hook genommen, das hat den Vorteil, dass Du per Post dann auch gleich das Ergebnis ausgeben kannst.
Statt der Anmeldenummer im Beispiel müsstest Du halt "disable" auf 1 setzen, aber ich könnte mir vorstellen, dass das so klappt.

Und den Teil mit dem Notification-Center brauchst Du evlt. auch nicht, außer Du willst jemanden über die Deaktivierung benachrichtigen.

Gruß
Hella

**dirksche** · 13.10.2018, 16:05

Hallo Hella,

vielen Dank für Deine Hilfe bzw. die Dokumentation. Das hilft mir schon ein Stückchen weiter.
Kann es sein, dass die Datei MyHooks.php im Verzeichnis Contao-Verzeichnis/src/AppBundle/ gespeichert werden muss und nicht einfach unter Contao-Verzeichnis/src/ wie in der Doku beschrieben?

Leider habe ich mit der Datenbankabfrage noch so meine Probleme.
So sieht der Code derzeit aus:

Code:

<?php

namespace AppBundle;


use Contao\Database;
use Contao\Input;

class MyHooks {

    public function memberDeactivate(&$arrSubmitted, $arrLabels, $objForm)
    {   

        $sql = "SELECT disable FROM tl_member WHERE id=?";
        $result = Database::getInstance()->prepare($sql)->execute();
        $arrSubmitted['disable'] = 1;
    }
}

LG,
Dirk

**fusch** · 13.10.2018, 16:51

Hallo Dirk,

mit dem Pfad hast Du natürlich recht, danke für den Hinweis, das ändere ich gleich.

Die Abfrage brauchst Du m.E. so nicht, weil Dich ja gar nicht interessiert, was im Feld disable drin steht (das braucht es in dem Beispiel ja nur zum rechnen). Du willst ja lediglich den Wert disable=1 übergeben.

Wie Du zur ID des member kommst, steht z.B. hier

Code:

FrontendUser::getInstance()->id;

Gruß
Hella

**dirksche** · 13.10.2018, 17:31

Also müsste ich einfach nur folgendes eintragen ?

Code:

$arrSubmitted['disable'] = FrontendUser::getInstance()->1;

**fiedsch** · 13.10.2018, 18:55

Nein, eher

Code:

$user = FrontendUser::getInstance(); $user->disable = 1; $user->save();

Code:

$member = MemberModel::findById(FrontendUser::getInstance()->id);
$member->disable = 1;
$member->save();

Edit: Code korrigiert

Edit 2: "prepareFormData" ist m.E. nicht geeignet, da dies ausgeführt wird, "after a form has been submitted, but before it is processed". Wenn nun beim Verarbeiten des Formulars befunden wird, daß die Eingaben ungültig waren, wäre so das Mitglied dennoch bereits deaktiviert. Von daher klingt für mich "storeFormData" geeigneter, denn es werden keine nicht validen Formulardaten gespeichert -- der Hook wird also in so einem Fall nicht ausgeführt.

**dirksche** · 14.10.2018, 09:07

Ja, das klingt logisch. Leider funktioniert die Deaktivierung noch nicht so ganz.

Mein Code sieht derzeit so aus.

In der config.php steht:

Code:

<?php
$GLOBALS['TL_HOOKS']['storeFormData'][] = array('AppBundle\MyHooks', 'memberDeactivate');

In der MyHooks.php steht:

Code:

<?php

namespace AppBundle;

use Contao\Database;

class MyHooks {

    public function memberDeactivate($arrSet, $objForm)
    {   
		$member = MemberModel::findById(FrontendUser::getInstance()->id);
		$member->disable = 1;
		$member->save();
    }
}

**fiedsch** · 14.10.2018, 09:37

Zitat von dirksche

Ja, das klingt logisch. Leider funktioniert die Deaktivierung noch nicht so ganz.

Was heißt denn "nicht so ganz"? Was passiert denn?

Code:

<?php

namespace AppBundle;

use Contao\Database;

class MyHooks {

    public function memberDeactivate($arrSet, $objForm)
    {   
		$member = MemberModel::findById(FrontendUser::getInstance()->id);
		$member->disable = 1;
		$member->save();
    }
}

Ich würde mal sagen, Dir fehlt noch

PHP-Code:


use Contao\MemberModel;

use Contao\FrontendUser;

und

PHP-Code:


use Contao\Database;

kann entfallen, da Du Database ja nirgends benutzt.

Edit: Außerdem musst Du noch zwei Sachen beachten:

1. hat der Hook einen Rückgabewert https://docs.contao.org/books/api/ex...eFormData.html
D.h. Dir fehlt am Ende von memberDeactivate() noch ein return $arrSet;

2. wird das so für alle Formulare angewandt. Das ist wahrscheinlich nicht das, was Du willst. D.h. Du musst noch prüfen, ob es das Formular ist, bei dem Du das Mitglied deaktivieren willst. Ich würde Dir dringend empfehlen, diese Prüfung einzubauen, selbst wenn Du aktuell nur dieses eine Formular haben solltest. Irgendwann kommt ein zweites hinzu und dann "passieren komische Sachen" ...

**dirksche** · 14.10.2018, 09:59

Hab es angepasst:

Code:

<?php

namespace AppBundle;

use Contao\MemberModel; 
use Contao\FrontendUser;

class MyHooks {


    public function memberDeactivate($arrSet, $objForm)
    {   
		$member= MemberModel::findById(FrontendUser::getInstance()->id);
		$member->disable = 1;
		$member->save();
                return $arrSet;
    }
}

Das Mitglied ist nach dem Absenden leider immer noch aktiv :-(
Es kommt keine Fehlermeldung und auch im Log ist kein Eintrag.

Edit:
Das return $arrSet; habe ich eingebunden. Das Mitglied ist aber immer noch aktiv.
Liegt es vielleicht an der $member Variable?
Sollte vielleicht $arrSet['member'] verwendet werden?

**fiedsch** · 14.10.2018, 10:28

Zitat von dirksche

Liegt es vielleicht an der $member Variable?
Sollte vielleicht $arrSet['member'] verwendet werden?

Ich denke nicht, daß es daran liegt. Du missbrauchst ja gewisserweise den Hook dazu etwas ganz anders zu machen: Du bekommst die Daten in $arrSet, deaktivierst das Mitglied und gibts $arrSet unverändert zurück. Bzgl. der Daten in $arrSet verhält es sich also so, als gäbe es gar keinen registrierten Hook.

Evtl. musst Du

PHP-Code:


$member->disable = '1';

schreiben, denn die SQL-Definition von disable in dca/tl_member.php ist "char(1) NOT NULL default ''".

**Stefko** · 14.10.2018, 10:33

Hallo,

ich habe den Thread gerade eben erst gefunden.
Auf die ursprüngliche Frage hätte ich geantwortet:
Nach Formular absenden Weiterleitung auf eine Seite in welcher das Modul Auto-Logout eingebunden ist.
Oder denke ich hier zu einfach? [emoji848]

Gesendet von iPhone mit Tapatalk Pro

**fiedsch** · 14.10.2018, 10:37

So, wie ich es verstanden habe, geht es um deaktivieren und nicht abmelden des Mitglieds.

**Stefko** · 14.10.2018, 10:40

Zitat von fiedsch

So, wie ich es verstanden habe, geht es um deaktivieren und nicht abmelden des Mitglieds.

Stimmt, jetzt sehe ich es auch

**dirksche** · 14.10.2018, 10:47

Code:

$member->disable = '1';

hat leider auch nicht den gewünschten Effekt erzielt :-( Keine Fehlermeldung aber das Mitglied bleibt aktiv.

**fiedsch** · 14.10.2018, 11:18

Dann habe ich jetzt auch keine Idee mehr :-(

Vielleicht noch per Eintrag in den Systemlog prüfen, daß der Code überhaupt ausgeführt wird:

PHP-Code:


\System::log("Deine Log-Meldung",  __METHOD__, TL_GENERAL);

**dirksche** · 14.10.2018, 11:34

An welcher Stelle in der myHooks.php Datei muss ich den Sytem Log Eintag einfügen?

**fiedsch** · 14.10.2018, 11:48

Irgendwo innerhalb memberDeactivate(). Dann wird der befehl ausgeführt, wenn memberDeactivate() ausgeführt wird und Bingo: Du weißt, daß memberDeactivate() ausgeführt wurde.

**dirksche** · 14.10.2018, 15:18

Code:

<?php

namespace AppBundle;

use Contao\MemberModel; 
use Contao\FrontendUser;

class MyHooks {


    public function memberDeactivate($arrSet, $objForm)
    {   
		\System::log("Test",  __METHOD__, TL_GENERAL);
		$member = MemberModel::findById(FrontendUser::getInstance()->id);
		$member->disable = '1';
		$member->save();
		
		return $arrSet;
    }
}

Ich hab den Log Befehl eingebunden. Leider scheint dieser auch nicht zu funktionieren. Er taucht im Log nicht auf :-(

**fiedsch** · 14.10.2018, 15:41

Zitat von dirksche

Ich hab den Log Befehl eingebunden. Leider scheint dieser auch nicht zu funktionieren. Er taucht im Log nicht auf :-(

Dann wird Dein Code nicht aufgerufen. Du musst herausfinden, warum.

Hast Du den Eintrag zum Aufruf des Hooks in der config.php wie im verlinkten Artikel von Hella (siehe #2) gemacht? Name des Hooks natürlich entsprechend angepasst.

**dirksche** · 14.10.2018, 15:53

Es funktioniert !!!!
Allerdings mit dem processFormData Hook.

Code:

<?php

namespace AppBundle;

use Contao\MemberModel; 
use Contao\FrontendUser;

class MyHooks {


    public function memberDeactivate($arrPost, $arrForm, $arrFiles)
    {   
		\System::log("Test",  __METHOD__, TL_GENERAL);
		$member = MemberModel::findById(FrontendUser::getInstance()->id);
		$member->disable = '1';
		$member->save();
    }
}

Mit den storeFormData komme ich auf keinen grünen Zweig.
Kann es evtl. an der Erweiterung Contao Leads liegen? Ich speichere die Formulardaten in einer Datenbank.

Den processFormData Hook kann ich doch auch problemlos verwenden, oder?

**fiedsch** · 14.10.2018, 16:23

Zitat von dirksche

Den processFormData Hook kann ich doch auch problemlos verwenden, oder?

Du musst halt probieren, was passiert, wenn die Formularvalidierung fehlschlägt, das Formular dem Member also erneut vorgelegt wird. Ich nehme an, daß Du in deisem Fall das Member noch nicht deaktivieren willst.