[BETA] Perso Check Mod 0.0.1

[Pionier]

Mod Name: Perso Check Mod
Autor: Pionier
Mod Beschreibung: Personalausweisnummer Check für dein Forum
Funktionsweise: Wenn sich ein neuer User in deinem Forum registriert, muss er erst mit seiner Personalausweisnummer bestätigen, dass er mindestens 18 Jahre alt ist.
Mod Version: 0.0.1
Getestet mit phpBB: 3.0.2
Mod Demo: N/A

Download: Hier

Sonstiges:
Das Script kann zur einer Altersverifikation des Users verwendet werden. Allerdings reicht dieser Check nicht bei Inhalten, die für Jugendliche nicht erreichbar sein dürfen (z.B Pornographie).

Ich bitte euch, den Mod gut zu testen und mir reichlich infos zugeben.
Ganz besonders möchte ich die erfahrenen Modder bitten, den Mod einmal auf Sicherheitslücken und Codeverbesserungen zu überprüfen und ob ich alles richtig (Modx 1.2.1) gepackt habe.
Für Lob und Tadel habe ich immer ein offenes Ohr. Aber auch für Vorschläge und Hilfen.

[4seven]

Hört sich erstmal gut an

[nickvergessen]

Da stimmt so einiges nicht.
Zum einem solltest du mal Tabs verwenden, damit den Code einrücken um die Lesbarkeit zu erhöhen.
dann noch 3 andere Sachen:

Code: Alles auswählen

if ($test1==true and $test2==true and $test3==true and $test4==true and $alterinjahren>=$mindestalter){
header("refresh:0;url=ucp.php?mode=register");
}else{
	}

eine leere else-Abfrage kann man auch weglassen.
header-refresh:0 geht mit der Funktion redirect. Dann sollte man bei Links immer den phpbb_root_path benutzen, die Datei endung mit $phpEx bezeichnen usw.
Also versuch mal

Code: Alles auswählen

redirect("{$phpbb_root_path}ucp.$phpEx", 'mode=register');

das and wird in phpBB auch nciht geswchrieben im Normalfall sondern mit && ausgedrückt.
Und zum Schluss noch

Code: Alles auswählen

$_POST[part1]

Es wäre besser hier

Code: Alles auswählen

request_var('part1', 0)//für Zahlen

zu benutzen.

Guter Lesestoff:

• Coding Guidelines
• phpBB3 MODDB policy
• Ansonsten gibts hier noch n paar Links: http://www.phpbb.com/mods/

[MartectX]

Bitte beachte, dass der im MOD Version Check eingetragene Benutzer auf phpBB.com registriert sein sollte, hole dies also bitte noch nach, Pionier`!

Sprachdatei:

Personalausweis-Nummerncheck

Du musst mindestens 18 Jahre alt sein, um dich registrieren zu können.</br>Trage hier deine Personalausweisnummer ein, um dieses zu[ ]bestätigen.

[Pionier]

eine leere else-Abfrage kann man auch weglassen. ]

Ist nun mit einer Fehlerbehandlung behoben

Code: Alles auswählen

if ($test1==true && $test2==true && $test3==true && $test4==true && $alterinjahren>=$mindestalter){
		header("refresh:0;url=ucp.php?mode=register");
	}else{
		$template->assign_vars(array(    
		'ERROR'		=> TRUE,    
		));
	}

header-refresh:0 geht mit der Funktion redirect. Dann sollte man bei Links immer den phpbb_root_path benutzen, die Datei endung mit $phpEx bezeichnen usw.
Also versuch mal

Code: Alles auswählen

redirect("{$phpbb_root_path}ucp.$phpEx", 'mode=register');

Habe ich bis jetzt noch nicht umsetzen können. Muss dafür noch eine Datei includen?

das and wird in phpBB auch nciht geswchrieben im Normalfall sondern mit && ausgedrückt.
Und zum Schluss noch

Code: Alles auswählen

$_POST[part1]

Es wäre besser hier

Code: Alles auswählen

request_var('part1', 0)//für Zahlen

zu benutzen.

Ist geändert

Bitte beachte, dass der im MOD Version Check eingetragene Benutzer auf phpBB.com registriert sein sollte, hole dies also bitte noch nach, Pionier`!]

Bin ich, war ein Tippfehler in der Datei

Sprachdatei:

Personalausweis-Nummerncheck

Du musst mindestens 18 Jahre alt sein, um dich registrieren zu können.</br>Trage hier deine Personalausweisnummer ein, um dieses zu[ ]bestätigen.

Ist geändert.

Alle Hinweise behoben, und hochgeladen. Abgesehen von der „redirect-Funktion“

[MartectX]

Habe ich bis jetzt noch nicht umsetzen können. Muss dafür noch eine Datei includen?

Nein, das macht alles common.php für Dich.

[djchrisnet]

Sehe ich das richtig, dass man nach dem check zur registrierung vergeleitet wird, aber wenn ich die registrierung direkt aufrufe (z.B. weil ich selbst phpBB3 nutze und die datei + Get kenne und kein bock habe, meinen perso zu holen) es nicht gecheckt wird?

Die Grundidee sicher cool, aber so leider leicht zu umgehen.

Ich denke aber, dass man den check selbst gut nutzen kann um den usern die möglichkeit zu geben sich zu verifizieren, sprich ihr im profil angegebenes alter mit dem perso prüfen zu lassen.

mal ne frage, lässt sich darüber auch part 1 selbst checken, eventuell ob die richtige postleitzahl angegeben wurde?

[Pionier]

Sehe ich das richtig, dass man nach dem check zur registrierung vergeleitet wird, aber wenn ich die registrierung direkt aufrufe (z.B. weil ich selbst phpBB3 nutze und die datei + Get kenne und kein bock habe, meinen perso zu holen) es nicht gecheckt wird?

Die Grundidee sicher cool, aber so leider leicht zu umgehen. ?

Ich bin ja auch erst am Anfang. Ich hoffe das es noch eine Möglichkeit gibt, mit der ich dann einen Aufruf der ucp.php?mode=register abfangen kann.
Aber wer sich so einen Zugang zum Forum verschaffen will/kann, der nutz auch einen Perso-Generator um sich älter zu machen

Ich denke aber, dass man den check selbst gut nutzen kann um den usern die möglichkeit zu geben sich zu verifizieren, sprich ihr im profil angegebenes alter mit dem perso prüfen zu lassen.

mal ne frage, lässt sich darüber auch part 1 selbst checken, eventuell ob die richtige postleitzahl angegeben wurde?

Ich glaube nicht Denn der erste Block ist die Behördenkennzahl und eine fortlaufende Nummer

Mehr Infos zur Perso-Nummer

@ MartectX
Wenn ich die „redirect-Funktion“ direkt einbaue wird die Seite nicht neu geladen

Code: Alles auswählen

	if ($test1==true && $test2==true && $test3==true && $test4==true && $alterinjahren>=$mindestalter){
		redirect("{$phpbb_root_path}ucp.$phpEx", 'mode=register');
	}else{
		$template->assign_vars(array(    
		'ERROR'		=> TRUE,    
		));
	}

[nickvergessen]

Nun, mit Hilfe dieses Links ( http://www.pruefziffernberechnung.de/P/ ... s-DE.shtml ) und etwas Intelligenz, kann sich eigentlich jeder so eine Nummer ausdenken um doch rein zukommen, auch wenn er nicht alt genug ist.

@Pionier: Wie siehts mit den Coding Guidelines bezüglich Tabs und Datei-Struktur aus?

[djchrisnet]

Nun, mit Hilfe dieses Links ( http://www.pruefziffernberechnung.de/P/ ... s-DE.shtml ) und etwas Intelligenz, kann sich eigentlich jeder so eine Nummer ausdenken um doch rein zukommen, auch wenn er nicht alt genug ist.

Wozu so umständlich? Google sucht dir sicher hunderte gültige Nummern raus!



Aber mal zurück zu der Behördenkennzahl ôo, dazu gibts sicher ne tabelle oder? ich meine, es gibt Geodaten der ganzen Welt im netz, den ich glaube weniger, dass Google dir die passende Kreisverwaltung zu einer Perso-Nr. gibt. dies ließe doch zusätzliche sicherheit bringen oder?

Achja, ich habe den Check in eine function() gebaut, dann kann man den während der registrierung nutzen, anstatt vorher. Die function hab ich hier hochgeladen: http://www.mods.djchrisnet.de/function.validate_age.txt