Mitgliedergruppe hinzufügen

**ele** · 14.03.2018, 14:58

Hallo zusammen,
ich habe einen Mitgliederbereich mit verschriedenen Gruppen.
Nun möchte ich den Mitglieder, die schon angemeldet sind, neue Mitgliedergruppen zufügen.
ich habe hier schon im Forum nachgesehen, aber nur etwas für Contao 4.4 gefunden und eine alte Anleitung aus 2012.

Wie kann ich das lösen? Evtl. über ein Anmeldeformular?
Oder geht das evtl. über den Catalog Manager?
Der ist auf der Seite installiert.
Danke euch.
LG ele

**eS_IT** · 15.03.2018, 06:47

Hallo ele,

ich bin nicht sicher, ob ich Dein Problem richtig verstehe. Die Mitgliedergruppen werden einfach im Backend zugewiesen. Du Klickst in der linken Seitenleiste auf Mitglieder, wählst das entsprechende Mitglied aus der Liste und setze einen Haken bei der gewünschten Mitgliedergruppe. Hier kannst Du es sehen: Member.png

**ele** · 15.03.2018, 08:40

Hall Patrick,
danke für deine Hilfe.

Es geht nicht ums Backend.
Den Kunden werden durch unterschiedliche Anmeldeformulare Mitgliedergruppen zugeordnet.
Nun gibt es aber Boni, die per Mitgliedergruppen zugeordnet werden.
Daher muss ich einem bestehenden Mitglied z.B. bei Anmeldung eine neue Mitgliedergruppe zuordnen können.
Gruß ele

**fiedsch** · 15.03.2018, 08:50

Ich würde den postLogin-Hook (https://docs.contao.org/books/api/ex...postLogin.html) verwenden und dort Deine Gruppenzuodnungslogik implementieren.

**ele** · 15.03.2018, 10:26

Da waren sie wieder - die minimalen Kenntnissse ...

Danke Andreas für den Hinweis.
Benutze ich das wie in easy_insertag?
Würde mich über Anregungen freuen.
Die Zeit drängt, wie immer in solchen Momenten.

Das setzten der neuen Mitgliedergruppe gilt auch nicht für jeden, der sich anmeldet.
Kann man das über ein vorgegebenes Formular einfügen?
LG ele

**fiedsch** · 15.03.2018, 12:01

Zitat von ele

Benutze ich das wie in easy_insertag?

Wenn Du die Registrierung des Hooks meinst: ja.

Zitat von ele

Das setzten der neuen Mitgliedergruppe gilt auch nicht für jeden, der sich anmeldet.
Kann man das über ein vorgegebenes Formular einfügen?

Eher nicht. In dem Hook bekommst Du als einzigen Parameter den angemeldeten User, weißt also nicht, wie die Anmeldung erfolgt ist. Es hört sich so an, als hättest Du verschiedene Loginformulare. Was soll denn genau passieren?

**ele** · 15.03.2018, 13:28

Auf der Seite ist der CatalogManager installiert.
Es gibt Mitglieder Gold, Silber usw.
Je nach Mitglieder werden unterschiedliche Inhalte angezeigt.

Nun habe ich einen Test mit Auswertung mit dem Catalog Manager erstellt.
Den soll aber nicht jeder sehen, ist aber auch nicht den bestehenden Gruppen zuzuordnen.
Dafür möchte ich eine neue Gruppe anlegen. Einigen bestehenden Mitgliedern soll diesen Test aber auch zugeordnet werden.

Es wäre auch möglich, dass der Kunden einen Link zu einer versteckten Seite erhält.
Dort könnte man die Mitgliederdaten ändern, Ausgabe nur diese Mitgliedergruppe.
Ist das möglichund wie kann ich das abprüfen?

Ich habe von xtra diese Funktion gefunden:

Code:

// Check ob aktueller User in Gruppe mit id $gid Mitglied ist.
if($this->User->isMemberOf($gid))
{ 
} else {
// Tu was anderes.
}

In diesem Bereich in member_default.html5 müsste das doch passieren, oder?

Code:

<?= $this->fields ?>

Da werden ja sicher alle Werte ausgegeben.
Wie splitte ich das?

Code:

if ($memberGroups[0] != 3) {}
{

**fiedsch** · 15.03.2018, 14:17

Ungetestet, aber in etwa das sollte funktionieren

PHP-Code:


// Wenn $this->Member eine Instanz von FrontendUser ist:

$groups = \StringUtil::deserialize($this->Member->groups);

$groups[] = 42; // die 42 durch die ID Deiner neuen Gruppe ersetzen

$this->Member->groups =serialize($groups);

$this->Member->save();

**ele** · 15.03.2018, 16:04

Sehe ich das richtig, dass du das Kennzeichen nur überschreibst?
Nicht ganz

Das Kennzeichen wird nicht gesetzt.

Diesen Ansatz habe ich gefunden:

PHP-Code:


$userId = $arrData["member_id"];

$groupId = "3";



$groups = $this->Database->prepare('SELECT groups FROM tl_member WHERE id=?')->execute($userId)->next()->groups;



#$groups = unserialize($groups);

$groups = deserialize($groups);

array_push($groups, $groupId);

$groups = serialize($groups);



$this->Database->prepare('UPDATE tl_member %s WHERE id=?')->set(array('groups' => $groups))->execute($userId);

**fiedsch** · 15.03.2018, 16:18

Zitat von ele

Sehe ich das richtig, dass du das Kennzeichen nur überschreibst?
Nicht ganz

Das Kennzeichen wird nicht gesetzt.

Ich verstehe ehrlich gesagt nicht, was Du meinst.

**ele** · 15.03.2018, 16:35

Ich dachte, dass er mit deinem Code die Mitgliedergruppe aktiviert.

Er geht durch die Seite und leitet auch weiter. Die Mitgliedergruppe wird aber nicht gesetzt.
Oder muss ich noch etwas anpassen.
Evtl. wie das Codebeispiel, das groups ja ein Array sein müsste

**fiedsch** · 15.03.2018, 16:41

Zitat von ele

Ich dachte, dass er mit deinem Code die Mitgliedergruppe aktiviert.

Ja, das ist die Absicht.

Er geht durch die Seite und leitet auch weiter. Die Mitgliedergruppe wird aber nicht gesetzt.
Oder muss ich noch etwas anpassen.
Evtl. wie das Codebeispiel, das groups ja ein Array sein müsste

Mit

PHP-Code:


$groups[] = 42;

wird an den Array angehängt, das ist identisch zur Vrwendung von array_push().

Wenn es nicht funktioniert, könnte es natürlich auch daran liegen, daß Du in $this->Member keinen User hast. Lass Dir doch mal (z.B. mit print_r()) ausgeben, was Du hast. Oder zeig mal Deinen gesamten Code.

**ele** · 15.03.2018, 17:36

Ich kann nicht mehr klar denken.
Ich danke dir Andreas für deine Hilfe.

PHP-Code:


<?= 

// print_r($groups);



// Wenn $this->Member eine Instanz von FrontendUser ist:

$groups = \StringUtil::deserialize($this->Member->groups);

$groups[] = 11; // die 11 durch die ID Deiner neuen Gruppe ersetzen

$this->Member->groups =serialize($groups);

$this->Member->save();

 ?>

**fiedsch** · 15.03.2018, 18:56

Wenn Du print_r($groups) aufrufst, bevor Du $groups etwas zuweist, wird nichts ausgegeben. Zudem solltest Du mal print_($this->Member) anschauen um zu sehen, ob Du überhaupt ein Member hast.

Außerdem sehe ich gerade, daß der öffnende PHP-Tag falsch ist. Du musst <?php verwenden.

**ele** · 16.03.2018, 09:27

Ein Mitglied müsste er haben, da er die gefilterten Daten vom Catalog Manger anzeigt.
Ich erhalte aber die Fehlermeldung:

Fatal error: Uncaught exception Error with message Call to undefined method Contao\StringUtil::deserialize() thrown in templates/tp/member_person.html5 on line 4

#0 system/modules/core/library/Contao/BaseTemplate.php(88): include()
#1 system/modules/core/library/Contao/Template.php(277): Contao\BaseTemplate->parse()
#2 system/modules/core/classes/FrontendTemplate.php(46): Contao\Template->parse()
#3 system/modules/core/modules/Module.php(311): Contao\FrontendTemplate->parse()
#4 system/modules/core/modules/ModulePersonalData.php(60): Contao\Module->generate()
#5 system/modules/core/elements/ContentModule.php(59): Contao\ModulePersonalData->generate()
#6 system/modules/core/library/Contao/Controller.php(484): Contao\ContentModule->generate()
#7 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao \ContentModel), 'main')
#8 system/modules/core/modules/Module.php(287): Contao\ModuleArticle->compile()
#9 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate()
#10 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(false)
#11 system/modules/core/library/Contao/Controller.php(277): Contao\Controller::getArticle(Object(Contao\Articl eModel), false, false, 'main')
#12 system/modules/core/pages/PageRegular.php(133): Contao\Controller::getFrontendModule('0', 'main')
#13 system/modules/core/controllers/FrontendIndex.php(285): Contao\PageRegular->generate(Object(Contao\PageModel), true)
#14 index.php(20): Contao\FrontendIndex->run()
#15 {main}

Folgenden Code verwende ich.

PHP-Code:


<?php 

// Wenn $this->Member eine Instanz von FrontendUser ist:

$groups = \StringUtil::deserialize($this->Member->groups);

$groups[] = 11; // die 11 durch die ID Deiner neuen Gruppe ersetzen

$this->Member->groups =serialize($groups);

$this->Member->save();

print_r($this->Member);

?>

**fiedsch** · 16.03.2018, 10:49

\StringUtil::deserialize() ist Contao 4 (https://github.com/contao/core-bundl...Util.php#L1037).

In Contao 3 (was Du verwendest und ich überlesen hatte :-( ist es einafch nur deserialize() (aus https://github.com/contao/core/blob/...tions.php#L304).

**ele** · 16.03.2018, 10:56

So meintest du sicher so:

PHP-Code:




<?php 

// Wenn $this->Member eine Instanz von FrontendUser ist:

$groups = deserialize($this->Member->groups);

$groups[] = 11; // die 11 durch die ID Deiner neuen Gruppe ersetzen

$this->Member->groups =serialize($groups);

$this->Member->save();

print_r($this->Member);

?>

Zu früh gefreut:

PHP-Code:


Warning: Creating default object from empty value in templates/tp/member_person.html5 on line 6



#0 templates/tp/member_person.html5(6): __error(2, 'Creating defaul...', '/www/htdocs/w01...', 6, Array)

#1 system/modules/core/library/Contao/BaseTemplate.php(88): include('/www/htdocs/w01...')

#2 system/modules/core/library/Contao/Template.php(277): Contao\BaseTemplate->parse()

#3 system/modules/core/classes/FrontendTemplate.php(46): Contao\Template->parse()

#4 system/modules/core/modules/Module.php(311): Contao\FrontendTemplate->parse()

#5 system/modules/core/modules/ModulePersonalData.php(60): Contao\Module->generate()

#6 system/modules/core/elements/ContentModule.php(59): Contao\ModulePersonalData->generate()

#7 system/modules/core/library/Contao/Controller.php(484): Contao\ContentModule->generate()

#8 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao\ContentModel), 'main')

#9 system/modules/core/modules/Module.php(287): Contao\ModuleArticle->compile()

#10 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate()

#11 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(false)

#12 system/modules/core/library/Contao/Controller.php(277): Contao\Controller::getArticle(Object(Contao\ArticleModel), false, false, 'main')

#13 system/modules/core/pages/PageRegular.php(133): Contao\Controller::getFrontendModule('0', 'main')

#14 system/modules/core/controllers/FrontendIndex.php(285): Contao\PageRegular->generate(Object(Contao\PageModel), true)

#15 index.php(20): Contao\FrontendIndex->run()

#16 {main}





Fatal error: Uncaught exception Error with message Call to a member function save() on null thrown in templates/tp/member_person.html5 on line 7



#0 system/modules/core/library/Contao/BaseTemplate.php(88): include()

#1 system/modules/core/library/Contao/Template.php(277): Contao\BaseTemplate->parse()

#2 system/modules/core/classes/FrontendTemplate.php(46): Contao\Template->parse()

#3 system/modules/core/modules/Module.php(311): Contao\FrontendTemplate->parse()

#4 system/modules/core/modules/ModulePersonalData.php(60): Contao\Module->generate()

#5 system/modules/core/elements/ContentModule.php(59): Contao\ModulePersonalData->generate()

#6 system/modules/core/library/Contao/Controller.php(484): Contao\ContentModule->generate()

#7 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao\ContentModel), 'main')

#8 system/modules/core/modules/Module.php(287): Contao\ModuleArticle->compile()

#9 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate()

#10 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(false)

#11 system/modules/core/library/Contao/Controller.php(277): Contao\Controller::getArticle(Object(Contao\ArticleModel), false, false, 'main')

#12 system/modules/core/pages/PageRegular.php(133): Contao\Controller::getFrontendModule('0', 'main')

#13 system/modules/core/controllers/FrontendIndex.php(285): Contao\PageRegular->generate(Object(Contao\PageModel), true)

#14 index.php(20): Contao\FrontendIndex->run()

#15 {main}

**fiedsch** · 16.03.2018, 12:56

Code:

Call to a member function save() on null

heißt, in $this->Member steht nichts (null) und nicht wie erwartet das angemeldete Mitglied.

Du könntest FrontendUser::getInstance() verwenden:

PHP-Code:


<?php  

$user = FrontendUser::getInstance();

$groups = deserialize($user->groups); 

$groups[] = 11;

$this->user->groups = serialize($groups); 

$this->user->save(); 

print_r(['gruppen'=>$this->user->groups]);

?>

**ele** · 16.03.2018, 15:11

Erst einmal danke für dein Durchhaltevermögen.

Ich hatte die Funktion auch schon gefunden, aber nicht herausgefunden wie man speichert, denn hier passiert ja der Fehler.
Ich denke, da fehlt ein \
also:

PHP-Code:


$user = \FrontendUser::getInstance();

Trotzdem wird der gleiche Fehler angezeigt.

Mit

PHP-Code:


var_dump($user->getData());

habe ich mir alle Felder anzeigen lassen.

Das steht in groups

PHP-Code:


 ["groups"]=> array(1) { [0]=> string(1) "8" }

Vielleicht sagt dir das was:

PHP-Code:


 <?php 



// Mitgliedsgruppen auslesen.

$memberGroups = \FrontendUser::getInstance()->groups ?: array();



// Überpr+fe Mitgliedsgruppe 3

if (in_array('3', $memberGroups)): ?>

User ist in Gruppe drei

<?php endif; ?>

von hier: https://community.contao.org/de/show...tgliedergruppe

werden Arrays anders gespeichert?

**fiedsch** · 16.03.2018, 15:40

Zitat von ele

werden Arrays anders gespeichert?

Hier würde ich sagen: ja, serialisiert. Daher ja das deserialize und serialize.
Wenn Du aber

HTML-Code:

["groups"]=> array(1) { [0]=> string(1) "8" }

bekommst wird anscheinend bereits automatisch deserialisiert. Abschließend stellt sich noch die Frage, ob auch automatisch serialisiert wird, wenn Du vor dem Speichern dem groups array einen weiteren Wert hinzufügst (weil "8" auch bereits ein String ist würde ich in Deinem Fall "11" und nicht 11 an groups anhängen).

Trotzdem wird der gleiche Fehler angezeigt.

Ich hatte in #18 natürlich auch Unsinn geschrieben, nachdem ich erst eine FrontendUser Instanz in $user gespeichert habe und dann auf $this->user weitergearbeitet habe. Gemeint hatte ich "natürlich" das hier:

<?php
$user = \FrontendUser::getInstance();
$groups = $user->groups;
$groups[] = 11;
$user->groups = $groups;
$user->save();
print_r(['gruppen'=>$user->groups]);
?>

**ele** · 16.03.2018, 16:47

Hatte mich schon gefreut:

PHP-Code:


Array ( [gruppen] => Array ( [0] => 8 [1] => 11 ) )

aber, es wird nicht abgespeichert

**fiedsch** · 16.03.2018, 17:11

Und

PHP-Code:


$user->groups = serialize($groups);

anstelle

PHP-Code:


$user->groups = $groups;

hast Du auch ausprobiert?

**ele** · 16.03.2018, 17:31

Geht leider auch nicht. Da gibt er print_r noch nicht mal aus

**fiedsch** · 16.03.2018, 17:41

Dann bin ich jetzt auch ratlos. Vielleicht zeigst Du noch mal den aktuellen Stand Deines Codes.

**ele** · 16.03.2018, 17:44

gerne

PHP-Code:


<?php

$user = \FrontendUser::getInstance();

$groups = $user->groups;

$groups[] = 11;

$user->groups = serialize($groups);  

$user->save();

print_r(['gruppen'=>$user->groups]);

?>

Habe den Cache gelöscht und noch mal aufgerufen, print_r zeigt er an

PHP-Code:


Array ( [gruppen] => a:2:{i:0;s:1:"8";i:1;i:11;} )

Aber speichert nicht

Die 8 ist in Hochkommas, die 11 nicht und S:irgenetwas: fehlt doch auch

**fiedsch** · 16.03.2018, 21:14

Die 8 ist in Hochkommas, die 11 nicht und S:irgenetwas: fehlt doch auch

Die 8 ist ein String ("8") und die 11 nicht, weil Du

PHP-Code:


$groups[] = 11;

und nicht

PHP-Code:


$groups[] = "11";

geschrieben hast.

**ele** · 16.03.2018, 21:28

PHP-Code:


Array ( [gruppen] => a:2:{i:0;s:1:"8";i:1;s:2:"11";} )

aber er sichert immer noch nicht

**fiedsch** · 16.03.2018, 22:17

Hmmm, keine Idee mehr. Ich probiere das mal morgen in Ruhe in einer Contao 3 Installation aus und schreibe Dir, was dabei raus gekommen ist.

**fiedsch** · 17.03.2018, 05:52

Das Speichern der Gruppen hat nicht funktioniert, weil FrontendUser::save() die "Gruppen zurücksetzt". Was der Sinn dahinter ist kann vielleicht jemand anderes erklären.

Was Du machen kannst ist das hier:

PHP-Code:


<?php

$user = \FrontendUser::getInstance();

if ($user->id) {

  // wir haben einen angemeldeten User. 

  $memberModel = \MemberModel::findById($user->id);

  $groups = deserialize($memberModel->groups);

  $groups[] = "11";

  $groups = array_unique($groups);

  $memberModel->groups = serialize($groups);

  $memberModel->save();

}

?>

Eine Alternative und wahrscheinlich der elegantere Weg (weil Du dann nicht extra ein Model verwendest):

PHP-Code:


<?php

$user = \FrontendUser::getInstance();

if ($user->id) {

  $groups = $user->groups;

  $groups[] = "2";

  $user->allGroups = array_unique($groups);

  $user->save();

}

?>