PHPbb Entwickler haben wohl langeweile (die Patchfile)

phpBB2.de User Anmeldungsdatum: 10.08.2002 Beiträge: 112 Wohnort: Münster

Hi,
möchte das mal so in den Raum stellen denn ich habe heute und gestern mein Forum mittels der Patchfile, die ich von Hand durchgearbeitet habe und alle Änderungen manuell gemacht habe, komisches festgestellt.

Dort werden mehr leerzeilen und sinnloses Zeug verändert als anderes. In der Patchfile 2.02 --> 2.03, die ich mir soebend anschaue finde ich folgendes :

Code:

Index: login.php
===================================================================
RCS file: /cvsroot/phpbb/phpBB2/login.php,v
retrieving revision 1.47.2.2
diff -u -r1.47.2.2 login.php
--- login.php 27 Jun 2002 21:26:46 -0000 1.47.2.2
+++ login.php 28 Sep 2002 12:55:25 -0000
@@ -6,7 +6,7 @@
* copyright : (C) 2001 The phpBB Group
* email : support@phpbb.com
*
- * $Id: login.php,v 1.47.2.2 2002/06/27 21:26:46 dougk_ff7 Exp $
+ * $Id: login.php,v 1.47.2.3 2002/09/20 11:40:38 psotfx Exp $
*
*
***************************************************************************/
@@ -46,7 +46,7 @@
{
   //
   // This appears to work for IIS5 CGI under Win2K. Uses getenv
- // since this doesn't exist for ISAPI mode and therefore the
+ // since this doesn't exist for ISAPI mode and therefore the
   // normal Location redirector is used in preference
   //
   if( ( isset($HTTP_POST_VARS['login']) || isset($HTTP_GET_VARS['login']) ) && !$userdata['session_logged_in'] )
@@ -54,7 +54,7 @@
      $username = isset($HTTP_POST_VARS['username']) ? $HTTP_POST_VARS['username'] : '';
      $password = isset($HTTP_POST_VARS['password']) ? $HTTP_POST_VARS['password'] : '';

-    $sql = "SELECT user_id, username, user_password, user_active, user_level
+    $sql = "SELECT user_id, username, user_password, user_active, user_level
         FROM " . USERS_TABLE . "
         WHERE username = '" . str_replace("\'", "''", $username) . "'";
      if ( !($result = $db->sql_query($sql)) )
@@ -99,7 +99,7 @@
            {
               $redirect = ( !empty($HTTP_POST_VARS['redirect']) ) ? $HTTP_POST_VARS['redirect'] : '';
               $redirect = str_replace("?", "&", $redirect);
-
+
               $template->assign_vars(array(
                  'META' => '<meta http-equiv="refresh" content="3;url=' . append_sid("login.$phpEx?redirect=$redirect") . '">')
               );
@@ -175,7 +175,7 @@
      {
         $forward_to = $HTTP_SERVER_VARS['QUERY_STRING'];

-       if( preg_match("/^redirect=(.*)$/si", $forward_to, $forward_matches) )
+       if( preg_match("/^redirect=([a-z0-9\.#\/\?&=\+\-_]+)/si", $forward_to, $forward_matches) )
         {
            $forward_to = ( !empty($forward_matches[3]) ) ? $forward_matches[3] : $forward_matches[1];
            $forward_match = explode('&', $forward_to);
@@ -216,11 +216,11 @@
      $template->assign_vars(array(
         'USERNAME' => $username,

-       'L_ENTER_PASSWORD' => $lang['Enter_password'],
+       'L_ENTER_PASSWORD' => $lang['Enter_password'],
         'L_SEND_PASSWORD' => $lang['Forgotten_password'],

-       'U_SEND_PASSWORD' => append_sid("profile.$phpEx?mode=sendpassword"),
-
+       'U_SEND_PASSWORD' => append_sid("profile.$phpEx?mode=sendpassword"),
+
         'S_HIDDEN_FIELDS' => $s_hidden_fields)
      );

So das is nen Kleiner Ausschnitt wo die Login.php verändert werden soll.
5 "Steps" werden dort gemacht (fange an beim 2. Step da der Erste ja nur Verwaltungstechnische Gründe hat (Versionnummer))

Step 1 : Ein Komentar wird verändert obwohl er total identisch ist Question

Step 2 : Eine SQL Zeile wird geändert obwohl sie total identisch ist. Question

Step 3 : Eine leerzeile wird gelöscht und dannach wieder eingefügt Shocked

Step 4 : Krass mal ne Zeile die neu anders aussieht als Alt (preg_match) Exclamation

Step 5 :Templateausgaben werden geändert...kann aber keine veränderungen in den Zeilen zwischen alt und neu feststellen.

Also ich muß ja sagen finds irgendwie komisch das 4 von 5 Änderungen vollkommen überflüssig sind oder muß man etwas was Updaten was zwischen den Zeilen steht Question

Sagt mal was die Fachkundigen davon Halten?

phpBB2.de User Anmeldungsdatum: 02.08.2002 Beiträge: 1172 Wohnort: Bayern

Ich schätze mal, dass da vlt des CSV Proggy was falsch aufgezeichnet hat.

Und deshalb die Änderung keine ist.

Fabro