Frage zu Patch-Anleitung

[schmidtedv]

huhu, ich hatte mal vor längerer Zeit vor, eine Sicherheitsabfrage für den Import von Benutzern einzubauen, wie hier beschrieben http://www.phpbb.com/phpBB/viewtopic.ph ... 92#2343792 allerdings konnte mir der Author bisher keine "normale" Anleitung zukommen lassen und die Patch-Geschichte verstehe ich nicht...kann mir einer erklären, wie ich manuell mit den + und - Zeilen vorgehen müßte um das gleiche Ergebnis zu erzielen? Meine Datei ist ein bisserl (eigenes Projekt) anders aufgebaut...aber ich denke, ich sllte es hinkriegen, wenn ich das nur richtig interpretieren kann, was der Author da schreibt...

[oxpus]

Das ist schnell erklärt:
+ = hinzufügen
- = löschen
! = ändern
und das jeweils ohne die führenden Zeichen versteht sich.

Und die Zeilen mit @@ sind die Zeilenangaben, wo das zu finden wäre (je nach Version der eigenen Datei unterschiedlich).

[schmidtedv]

hm, so hatte ich's mir gedacht,ging nur beim ersten mal schief...dann geh ich das nochmal durch :-) thx

EDIT: Jetzt weiß ich wo das Hauptproblem lag....er hat was falsch "übersetzt", d.h. in seinem code sollte

Code: Alles auswählen

$errors[] = "Insert failed for $username ($email).";

eigentlich

Code: Alles auswählen

$errors[] = "Insert successfull for $username ($email).";

sein. Bzw. bei mir hab ich's nun so gelöst:

Code: Alles auswählen

if ($user_check) {
                if (!$user->insert_user()) {
                    $errors[] = "<font color=green>$username</font> (<font color=green>$email</font>)";
                }
            } else {
                $errors[] = "<font color=red>$username</font> (<font color=red>$email</font>) - falsche oder schon vorhandene Angaben (Benutzername, E-Mail) in der Liste";
            }

Code: Alles auswählen

} else {
        $e = '';
        if (count( $errors )) {
            for ($i = 0; $i < count($errors); $i++) {
                $e .= "<div align='left'><span class='gensmall'>{$errors[$i]}</span></div>";
            }
        }
    	  
        $template->assign_vars(array('MESSAGE_TITLE' => $lang['Add_new_import_complete'],
                'MESSAGE_TEXT' => ($e ? $e : $lang['Add_new_import_complete_explain']))
            );
    }

Scheint zu funktionieren, wobei das bei mehr als 100 Benutzern eine recht lange Liste würde...aber immernoch übersichtlicher als nur mit Komma getrennt, naja, Geschmackssache