Social Media
Foren
QA Register
|
QA Register
Das Plugin Register habe ich noch nicht komplett getestet. Mir ist aber aufgefallen, dass immer fest mit der Role-ID 4 gearbeitet wird. Das können wir leider nicht so machen, da man in der Rollenverwaltung die Rollen verschieden tauschen kann. Schön wäre es, wenn man beim Anlegen der Registrieren-Seite bereits die Rolle auswählen kann, die der User bekommen soll.
Was mir schon gut gefallen hat, ist dass geprüft wird, ob die Abhängigkeiten (plugindata) bereits vorhanden sind. Das hab ich bei meinen Plugins so noch nicht gemacht. Was nicht so toll ist, ist das exit, mir dem die komplette Ausführung der Seite beendet wird. So wird z.B. der Footer der Seite gar nicht mehr geladen. Hier sollten wir uns etwas anderes ausdenken. Siehe activate.php.
Die Übersetzungen werden schon teilweise genutzt, allerdings stehen an vielen Stellen noch feste Texte drin. Da wäre es schön, wenn man diese noch durch die Tokens ersetzen könnte. In der activate.php muss man die Language-Datei per Include laden. Hier sollten wir und auch nochmal was überlegen.
Meine Änderungen im Code:
- trunk und branch hinzugefügt
Was mir schon gut gefallen hat, ist dass geprüft wird, ob die Abhängigkeiten (plugindata) bereits vorhanden sind. Das hab ich bei meinen Plugins so noch nicht gemacht. Was nicht so toll ist, ist das exit, mir dem die komplette Ausführung der Seite beendet wird. So wird z.B. der Footer der Seite gar nicht mehr geladen. Hier sollten wir uns etwas anderes ausdenken. Siehe activate.php.
Die Übersetzungen werden schon teilweise genutzt, allerdings stehen an vielen Stellen noch feste Texte drin. Da wäre es schön, wenn man diese noch durch die Tokens ersetzen könnte. In der activate.php muss man die Language-Datei per Include laden. Hier sollten wir und auch nochmal was überlegen.
Meine Änderungen im Code:
- trunk und branch hinzugefügt
Das Plugin Register habe ich noch nicht komplett getestet. Mir ist aber aufgefallen, dass immer fest mit der Role-ID 4 gearbeitet wird. Das können wir leider nicht so machen, da man in der Rollenverwaltung die Rollen verschieden tauschen kann. Schön wäre es, wenn man beim Anlegen der Registrieren-Seite bereits die Rolle auswählen kann, die der User bekommen soll.
Stimmt!
Werde eine Einstellung hinzufügen, mit der man das auwählen kann.
![;)](wcf/images/smilies/wink.png)
Habe dies auch bei meinem Plugin "vbulletinimportuser" so gemacht, nur dass da die Einstellungen in ner Datei gespeichert werden.
![;)](wcf/images/smilies/wink.png)
Was mir schon gut gefallen hat, ist dass geprüft wird, ob die Abhängigkeiten (plugindata) bereits vorhanden sind. Das hab ich bei meinen Plugins so noch nicht gemacht. Was nicht so toll ist, ist das exit, mir dem die komplette Ausführung der Seite beendet wird. So wird z.B. der Footer der Seite gar nicht mehr geladen. Hier sollten wir uns etwas anderes ausdenken. Siehe activate.php.
Danke!
![:)](wcf/images/smilies/smile.png)
@exit:
Stimmt! Diese Variante finde ich auch überhaupt nicht schön, aber ich habe keine andere Möglichkeit gefunden...
Die Übersetzungen werden schon teilweise genutzt, allerdings stehen an vielen Stellen noch feste Texte drin. Da wäre es schön, wenn man diese noch durch die Tokens ersetzen könnte. In der activate.php muss man die Language-Datei per Include laden. Hier sollten wir und auch nochmal was überlegen.
Meine Änderungen im Code:
- trunk und branch hinzugefügt
Hmmm...
Das liegt aber voralle daran, dass ContentLion z.Z. noch einen Bug darin hat, s. anderen Thread.
Aber in den Template-Dateien kann man das ändern. Derzeit habe ich als Sprache auch nur deutsch angegeben, wollte sowas dann als nächstes Update rausbringen. Ich finde, dass User- & Registrierungssystem ist eines der wichtigsten Sachen jetzt, womit wir CL vorwärts bringen können. Deßhalb wolte ich die User nicht zu lange warten lassen.
![;)](wcf/images/smilies/wink.png)
EDIT:
Wie mit Include?
|
einfach das Array reinladen und an Hand der die Übersetzungen holen
http://www.php.net/manual/de/function.include.php
![^^](wcf/images/smilies/squint.png)
http://www.php.net/manual/de/function.include.php
|
So,
mal weiter getestet.
- Wenn z.B. die Passwörter nicht übereinstimmen am besten den Benutzernamen und die E-Mail gefüllt lassen, damit man es nicht nochmal eingeben muss
- Ist das E-Mail Format falsch, die Passwörter aber gleich, wird trotzdem angegeben, dass die Passwörter ungleich sind.
- Beim Ändern-Button beim Registrieren ist bei mir kein Text zu sehen
- Wäre gut, wenn man auch eingeben könnte, wohin der User nach der Registrierung geleitet wird. Und das am besten mit dem PHP-Header Location und nicht mit JavaScript ;-) (Schau dir mal die Weiterleitung im Login oder Logout Plugin an)
mal weiter getestet.
- Wenn z.B. die Passwörter nicht übereinstimmen am besten den Benutzernamen und die E-Mail gefüllt lassen, damit man es nicht nochmal eingeben muss
- Ist das E-Mail Format falsch, die Passwörter aber gleich, wird trotzdem angegeben, dass die Passwörter ungleich sind.
- Beim Ändern-Button beim Registrieren ist bei mir kein Text zu sehen
- Wäre gut, wenn man auch eingeben könnte, wohin der User nach der Registrierung geleitet wird. Und das am besten mit dem PHP-Header Location und nicht mit JavaScript ;-) (Schau dir mal die Weiterleitung im Login oder Logout Plugin an)
Weiß schon.![^^](wcf/images/smilies/squint.png)
Aber ich will erst nach 5 Sek. weiterleiten.![;)](wcf/images/smilies/wink.png)
@Übersetzung:
Aber die
gibt ja den Error zurück.![^^](wcf/images/smilies/squint.png)
@Formular:
Stimmt... Werde das mal versuchen, zu verbessern...![;)](wcf/images/smilies/wink.png)
@Ist das E-Mail Format falsch, die Passwörter aber gleich, wird trotzdem angegeben, dass die Passwörter ungleich sind.: Hä?
![^^](wcf/images/smilies/squint.png)
Aber ich will erst nach 5 Sek. weiterleiten.
![;)](wcf/images/smilies/wink.png)
@Übersetzung:
Aber die
![]() |
PHP-Quelltext |
1 |
DataBase::Current()->ReadRows("SELECT * FROM `...` ...");
|
![^^](wcf/images/smilies/squint.png)
@Formular:
Stimmt... Werde das mal versuchen, zu verbessern...
![;)](wcf/images/smilies/wink.png)
@Ist das E-Mail Format falsch, die Passwörter aber gleich, wird trotzdem angegeben, dass die Passwörter ungleich sind.: Hä?
![?(](wcf/images/smilies/confused.png)
![^^](wcf/images/smilies/squint.png)
|
Hmm, hast ja nen Link drin. Dann lassen wir das mal so (aber nicht wundern, wenn ich mein eigenes Registrieren-Plugin rausbringe
).
Hmm, irgendwie kann ich jetzt auch ne falsche Mail-Adresse angeben. Dann hat sich der erste Fehler erledigt. Solltest die Mail aber validieren, weil "test" ist keine Mail-Adresse ;-) Lässt sich mit Regex machen.
![:)](wcf/images/smilies/smile.png)
Hmm, irgendwie kann ich jetzt auch ne falsche Mail-Adresse angeben. Dann hat sich der erste Fehler erledigt. Solltest die Mail aber validieren, weil "test" ist keine Mail-Adresse ;-) Lässt sich mit Regex machen.
Hmm, hast ja nen Link drin. Dann lassen wir das mal so (aber nicht wundern, wenn ich mein eigenes Registrieren-Plugin rausbringe).
Ok, dann behebe ich es halt.
![^^](wcf/images/smilies/squint.png)
Hmm, irgendwie kann ich jetzt auch ne falsche Mail-Adresse angeben. Dann hat sich der erste Fehler erledigt. Solltest die Mail aber validieren, weil "test" ist keine Mail-Adresse ;-) Lässt sich mit Regex machen.
Richtig! Das ist nämlich nicht Umfang dieses Plugins.
![;)](wcf/images/smilies/wink.png)
In nem anderem Thread habe ich bereits geschrieben, wie man so etwas ganz einfach realisieren kann.
![;)](wcf/images/smilies/wink.png)
Grund ist die Anpassbarkeit. Deßwegen wirft das Plugin auch so unendlich viele Events.
![^^](wcf/images/smilies/squint.png)
Es ist ganz einfach, so eine Überprüfung als Plugin zu bauen und vllt. noch bei SpamAssis (oder wie das heißt), nachzuschauen, ob die Mail-Adresse einem Spam-Bot gehört.
![;)](wcf/images/smilies/wink.png)
Du hast ja selbst gesagt, dass wir viele kleine Plugins haben wollen, damit wir einzelne einfach "austauschen" können.
![:D](wcf/images/smilies/biggrin.png)
Dieses Konzept habe ich bei diesem Plugin angewandt, hätte ich diese Überprüfung gleich mit realisiert, könnte man diese später nicht so einfach ersetzen.
![;)](wcf/images/smilies/wink.png)
Hier noch ein Plugin, welches die E-Mail validiert / überprüft: ![;)](wcf/images/smilies/wink.png)
registercheckmail.zip
Geht sogar auch ohne Regex, da PHP für so etwas eine extra Methode "filter_var($email, FILTER_VALIDATE_EMAIL)" bereitstellt.![;)](wcf/images/smilies/wink.png)
--> http://php.net/manual/de/filter.examples.validation.php
![;)](wcf/images/smilies/wink.png)
registercheckmail.zip
Geht sogar auch ohne Regex, da PHP für so etwas eine extra Methode "filter_var($email, FILTER_VALIDATE_EMAIL)" bereitstellt.
![;)](wcf/images/smilies/wink.png)
--> http://php.net/manual/de/filter.examples.validation.php
|
Sehe ich nicht so. Für die Validität der Daten muss an jeder Stelle gesorgt werden. Das macht keinen Sinn, nur für die Validierung der E-Mail Adresse ein weiteres Plugin rauszubringen.
Wenn überhaupt die andere Richtung. Du machst eine Klasse E-Mail, in der auch eine Überprüfung statt findet. Die register nutzt diese dann und gibt ne Meldung aus, wenn die falsch ist.
Wenn überhaupt die andere Richtung. Du machst eine Klasse E-Mail, in der auch eine Überprüfung statt findet. Die register nutzt diese dann und gibt ne Meldung aus, wenn die falsch ist.
Ein Captcha-Bild folgt übrigens noch. ![;)](wcf/images/smilies/wink.png)
Ich werde ein Plugin erstellen, welches unter anderem eine Einstellung erstellt, mit der man sein Captcha-Modul auswählen kann.![;)](wcf/images/smilies/wink.png)
Das Plugin liefer ich gleich mit, bzw. trage es in die Voraussetzungen ein, da es die Datenbank-Struktur für die Captcha-Module erstellt.![;)](wcf/images/smilies/wink.png)
Hier mal nen Screenshot von nem Captcha-Script, welches ich mit nem kleinem Tut erstellt habe:
![](index.php?page=Attachment&attachmentID=120&thumbnail=1&embedded=1)
Natürlich kann man im Backend dann auch Größe, Schriftart, Hintergrund usw. einstellen.
![;)](wcf/images/smilies/wink.png)
Ich werde ein Plugin erstellen, welches unter anderem eine Einstellung erstellt, mit der man sein Captcha-Modul auswählen kann.
![;)](wcf/images/smilies/wink.png)
Das Plugin liefer ich gleich mit, bzw. trage es in die Voraussetzungen ein, da es die Datenbank-Struktur für die Captcha-Module erstellt.
![;)](wcf/images/smilies/wink.png)
Hier mal nen Screenshot von nem Captcha-Script, welches ich mit nem kleinem Tut erstellt habe:
Natürlich kann man im Backend dann auch Größe, Schriftart, Hintergrund usw. einstellen.
![;)](wcf/images/smilies/wink.png)
So, habe das Plugin nochmal komplett überarbeitet, vorallem die Struktur des Codes, damit dieser besser lesbar ist.
Bitte nochmal testen!![;)](wcf/images/smilies/wink.png)
Diesmal habe ich EXTRA FÜR STEFAN
eine HTTP-Weiterleitung eingebaut. ![:D](wcf/images/smilies/biggrin.png)
Die Seite, auf die Weitergeleitet wird, wird dynamisch per Plugin "PluginData" geladen und kann so auch beeinflusst werden. (Plugin_PluginData_Data::Current()->setData("plugin_register_redirection", "<Dein Alias>")).
Ebenfalls vorhanden ist eine Fallback-Variante, falls jemand keine HTTP-Weiterleitungen mag.
Hier kann man mittels
den Wert auf "0" stellen und es wird die normale Seite mit HTML-Weiterleitung angezeigt.
Bitte nochmal testen!
![;)](wcf/images/smilies/wink.png)
Diesmal habe ich EXTRA FÜR STEFAN
![^^](wcf/images/smilies/squint.png)
![:D](wcf/images/smilies/biggrin.png)
Die Seite, auf die Weitergeleitet wird, wird dynamisch per Plugin "PluginData" geladen und kann so auch beeinflusst werden. (Plugin_PluginData_Data::Current()->setData("plugin_register_redirection", "<Dein Alias>")).
Ebenfalls vorhanden ist eine Fallback-Variante, falls jemand keine HTTP-Weiterleitungen mag.
Hier kann man mittels
![]() |
PHP-Quelltext |
1 |
Plugin_PluginData_Data::Current()->setData("plugin_register_http_redirection", 0);
|
den Wert auf "0" stellen und es wird die normale Seite mit HTML-Weiterleitung angezeigt.
![;)](wcf/images/smilies/wink.png)
![](icon/similarThreadsM.png)
Ähnliche Themen
-
QR Code Generator »
-
QR-Code Generator in neuer Version (27. Januar 2013, 11:55)
-
Projekt und Plugin Vorstellung »
-
User-System wir kommen! :) (1. Februar 2013, 16:35)