Seite 2 von 2
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 12.07.2020 18:09
von chris1278
Ich denke wenn du dir den Code Block hier genau anschaust sieht man wo diese Lastvisit variable herkommt:
Code: Alles auswählen
public function send_topic()
{
if ($this->user->data['is_registered'] && $this->user->data['user_lastvisit'] == 0)
{
$this->create_welcome_topic($this->user->data['user_id']);
$sql_ary = array(
'user_lastvisit' => $this->user->data['session_last_visit'],
);
$sql = 'UPDATE ' . USERS_TABLE . '
SET ' . $this->db->sql_build_array('UPDATE', $sql_ary) . '
WHERE user_id = ' . $this->user->data['user_id'];
$result = $this->db->sql_query($sql);
$this->db->sql_freeresult($result);
}
}
Sie in der dazu gehöhrigen sql abfrage folgene zeile:
Code: Alles auswählen
'user_lastvisit' => $this->user->data['session_last_visit'],
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 12.07.2020 20:04
von Mike-on-Tour
Wo user_lastvisit
herkommt, ist klar, das wird aus der Users Tabelle der Datenbank entnommen, die Frage ist aber, wo kommt is_registered
her und wie ist das zum Zeitpunkt des Funktionsaufrufes definiert, denn die Abfrage in Zeile 58 ist eine UND-Verknüpfung. Dass bei einem neu registrierten Nutzer beim ersten Login user_lastvisit
gleich 0 ist, steht fest, aber um die if-Bedingung zu erfüllen, muss is_registered
ja auch noch true
sein, ansonsten wird die Funktion nicht durchlaufen.
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 04:28
von seb81
Hallo zusammen,
ich habe mir diese Erweiterung heruntergeladen, wie beschrieben geändert und in Betrieb genommen. Im Großen und Ganzen funktioniert sie prima. Es wäre zwar schön, wenn ich einen Benutzer zuweisen könnte, der als Verfasser des Begrüßungsbeitrags fungiert. Der Beitrag wird immer im Namen des Benutzers verfasst, der sich neu angemeldet hat (und der Name ist immer in
kleinbuchstaben).
Mein Hauptproblem ist jedoch, dass ich nur einen sehr kurzen Begrüßungstext eingeben kann. Derzeit habe ich nur "xxxx".
Code: Alles auswählen
[size=110][b]Herzlich Willkommen "%user"!
Wir wünschen dir viel Spaß bei uns.
Wenn du etwas über dich schreiben möchtest, kannst du das gerne hier tun.[/size]
Dieser Beitrag wurde automatisch von "%robot" für dich erstellt.[/b
Wenn ich 2 oder 3 Buchstaben mehr eingebe, erhalte ich folgende Fehlermeldung:
Code: Alles auswählen
SQL ERROR [ mysqli ]
Data too long for column 'config_value' at row 1 [1406]
SQL
UPDATE phpbb_config SET config_value = '[size=110][b]Herzlich Willkommen "%user"!\n\nWir wünschen dir viel Spaß bei uns. \n\nWenn du etwas über dich schreiben möchtest, kannst du das gerne hier tun.[/size]\nDieser Beitrag wurde automatisch von "%robot" für dich erstellt.[/b] Mehr Mehr Mehr' WHERE config_name = 'welcomerobot_detail'
BACKTRACE
FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()
FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 1031
CALL: trigger_error()
FILE: [ROOT]/phpbb/db/driver/mysqli.php
LINE: 202
CALL: phpbb\db\driver\driver->sql_error()
FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 353
CALL: phpbb\db\driver\mysqli->sql_query()
FILE: [ROOT]/phpbb/config/db.php
LINE: 163
CALL: phpbb\db\driver\factory->sql_query()
FILE: [ROOT]/phpbb/config/db.php
LINE: 137
CALL: phpbb\config\db->set_atomic()
FILE: [ROOT]/includes/functions_compatibility.php
LINE: 299
CALL: phpbb\config\db->set()
FILE: [ROOT]/ext/alifaraji/welcomerobot/acp/welcomerobot_module.php
LINE: 84
CALL: set_config()
FILE: [ROOT]/includes/functions_module.php
LINE: 676
CALL: alifaraji\welcomerobot\acp\welcomerobot_module->main()
FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()
Wo könnte der Fehler liegen? Die Erweiterung ist großartig und genau das, wonach ich gesucht habe. Allerdings wäre es toll, wenn ich ein paar Zeilen Begrüßungstext mehr eingeben könnte. (Ich hatte schon überlegt, einen BBCODE für die Darstellung von einem Bild einzufügen. Und einfach einen Text als Bild anzeigen zu lassen. Aber toll ist das nicht ...
Liebe Grüße, Sebastian
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 10:10
von chris1278
Das liegt wahrscheinlich daan das die Tabelle config_valu nur eine bestimmte anzahl an zeichen zuläasst. Wenn man in deine Tabelle mehr zeichen eintrageb will als zulässig passiert dies so.
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 14:35
von seb81
Hallo Chris,
Ja so weit dachte ich mir das auch. Kann man da bei der Ext was ändern, ohne das ganze Ding gleich neu bauen zu müssen? Sonst muss das halt genügen, oder ich mach das mit dem BBCode <Img> und füge ein Bild ein.
Ich freu mich auf jeden Fall das du sie aufgearbeitet hast. Ich find das super. Neue Mitglieder bekommen jetzt automatisch eine PN (da kann man den User der sie "verschickt" auswählen.) Und halt einen kleinen Begrüßungsbeitrag. Das ist auf jeden Fall schon super !!!
Grüße, Sebastian
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 15:39
von chris1278
Keine PN sondern Beitrag.
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 15:46
von LukeWCS
seb81 hat geschrieben: 04.05.2024 14:35
Kann man da bei der Ext was ändern, ohne das ganze Ding gleich neu bauen zu müssen?
Der Autor hat - aus mir unverständlichen Gründen - zur Speicherung der Beitragsmaske die Tabelle
config
gewählt. Hier können pro Wert jedoch nur 256 Bytes gespeichert werden, was natürlich zur Bildung eines Beitrags ruckzuck viel zu wenig ist und für derlei Beitragsmasken ist die besagte Tabelle auch gar nicht gedacht, sondern für Einstellungen (Schalterzustände) und kleinere Texte, die keinesfalls 256 Bytes übersteigen würden. Hierzu hätte also die Tabelle
config_text
gewählt werden müssen, wo Werte bis zu 16MB aufnehmen können. Das heisst hier müsste eine neue Migration hinzugefügt werden, die dieses Problem behebt.
Da müsste aber noch deutlich mehr gemacht werden, Chris hat die Ext damals nur soweit fit gemacht, damit sie unter 3.3. wieder aktiviert werden kann. Ein flüchtiger Blick in die Codebase hat gezeigt, dass da noch einiges mehr zu machen wäre.
Im jetzigen Zustand kann ich nur davon abraten, diese unter 3.3 in einem LB zu nutzen, alleine durch den Ext Check Bericht, der klar aussagt, das veraltete Techniken und Funktionen genutzt wurden, die von EPV (Analysetool) auch als deprecated (veraltet) eingestuft sind. Davon abgesehen ist die Ext auch bezüglich Richtlinien in keinem guten Zustand, was nicht weiter verwundert, da die Codebase der Ext mittlerweile 9 Jahre alt ist, somit noch zur 3.1 Ära gehört und 2015 als ABD eingestuft und das Thema dazu geschlossen wurde. Die Ext ist auch laut Autor eine portierte Mod, was auch erklärt, warum da noch Techniken zu finden sind, die zur 3.0 Ära gehören.
Das heisst die Ext bräuchte einen neuen und aktiven Coder, der diese auf Vordermann bringt und dann auch mit Updates versorgt. Mit "mal eben" ist hier nicht viel gewonnen. Daher gilt nach wie vor was Chris bereits schrieb:
chris1278 hat geschrieben: 09.07.2020 06:51
Danach sollte das korrekt in Deutsch angezeigt werden. Mehr kann ich leider nicht tun.
Aber bedenkt, die Extension an sich wird so nicht mehr weiterentwickelt. Das heißt, Einsatz ist auf eigene Gefahr.
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 16:15
von seb81
chris1278 hat geschrieben: 04.05.2024 15:39
Keine PN sondern Beitrag.
Doch, doch, das ist aber eine andere Erweiterung von hier. Diese erstellt automatisch eine private Nachricht mit einem freundlichen Begrüßungstext, ein paar netten Worten und einem Verweis auf den Beitrag, den die Ext "Automatisches Begrüßungsthema für Neu registrierte Benutzer" erstellt (das Forum/Kategorie, in der er steht)
LukeWCS hat geschrieben: 04.05.2024 15:46
Ah, okay, vielen Dank. Das habe ich jetzt verstanden. Schade, dass es keine "neue" Version gibt oder eine andere Erweiterung, die dasselbe leistet. Ich finde diese Erweiterung eigentlich super. Meine Hoffnung ist es, damit neu angemeldete Mitglieder zu einem "ersten -eigenem" Posting zu motivieren. Wenn das einmal getan ist (weil sie vielleicht auch sehen, dass von anderen Mitgliedern bereits Willkommensgrüße bekommen), fällt meiner Meinung nach die Hemmschwelle ein wenig und sie schreiben eher etwas. Ich meine, ich habe jetzt seit dem Boardstart vor 10 Wochen mit 35 Mitgliedern keinen Rekord aufgestellt, aber ich bin damit sehr zufrieden. Allerdings sind 20 Mitglieder "Nullposter", was ich eigentlich schade finde.
Also lasse ich sie erst einmal aktiv und warte ab ob sie etwas bei/nach Neuanmeldungen verändert (was Nullposter angeht). Installieren konnte man sie ja ohne Probleme. Und den Text den ich vorher hatte, habe ich jetzt noch einmal halbiert in dem ich ein Bild eingefügt habe (in dem Text steht). An der Sache ist vieles blöde, das weiß ich. Aber versuchen möchte ich es wenigstens.
Vielen Dank für eure liebe Hilfe,
bis bald und alles Gute. Sebastian
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 04.05.2024 16:33
von LukeWCS
Bitte keine unnötigen Vollzitate.
seb81 hat geschrieben: 04.05.2024 16:15
Ich meine, ich habe jetzt seit dem Boardstart vor 10 Wochen mit 35 Mitgliedern keinen Rekord aufgestellt, aber ich bin damit sehr zufrieden. Allerdings sind 20 Mitglieder "Nullposter", was ich eigentlich schade finde.
Wenn es dir speziell darum geht, könnte man das auch alternativ regeln. Nach meiner Erfahrung sind Nullposter zu 99% Schläfer-Accounts von Bots, mit denen zu einem späteren Zeitpunkt Spam abgesetzt, oder die für andere illegale Aktionen missbraucht werden.
Ich hab da eine ganz simple Regel auf die ich jeden Benutzer beim ersten Login per Willkommens-PN hinweise: jeder Account der nicht innerhalb 1 Woche mindestens einen freigegebenen Beitrag aufweist, wird automatisch gelöscht. Menschen lesen das und richten sich danach. Bots lesen das natürlich nicht und werden somit gelöscht. Für Registrierte ist das Thema mit den Regeln ebenfalls sichtbar, aber nur KRBs sehen im Index einen direkten Link darauf. Sobald ein Mitglied die KRB Gruppe verlassen hat, sieht er diesen Link nicht mehr, da nicht mehr relevant.
Re: Automatisches Begrüßungsthema für Neu registrierte Benutzer.
Verfasst: 05.05.2024 13:46
von LukeWCS
Thema geschlossen, weiter geht es mit dem Fork von chris1278:
viewtopic.php?t=247219