Seite 1 von 1

phpBB2-2.0.21 Update, da stimmt was nicht

Verfasst: 10.06.2006 15:10
von michi-w.
Hallo,

ich habe mir das aktuelle Update von phpbb.com geholt.

Nun ist mir hier wohl ein Fehler aufgefallen:

Code: Alles auswählen

includes/functions_post.php

#
#-----[ FIND ]---------------------------------------------
# Line 59
			$message .= htmlspecialchars($part) . clean_html($tag);
		}

		$message = addslashes($message);

#
#-----[ REPLACE WITH ]---------------------------------------------
#
			$message .= preg_replace($html_entities_match, $html_entities_replace, $part) . clean_html($tag);
		}

		$message = addslashes($message);
		$message = str_replace('"', '\"', $message);
Müsste das nicht so sein?

Code: Alles auswählen

#
#-----[ FIND ]---------------------------------------------
# Line 59
			$message .= htmlspecialchars($part) . clean_html($tag);
		}

		$message = addslashes($message);
		$message = str_replace('"', '\"', $message);
Gruß
michi-w.

Verfasst: 10.06.2006 15:14
von Balint
Hallo!

Also die Originalversion von 2.0.20 sagt an der betreffenden Stelle:

Code: Alles auswählen

		foreach ($message_split as $part)
		{
			$tag = array(array_shift($matches[0]), array_shift($matches[1]), array_shift($matches[2]));
			$message .= htmlspecialchars($part) . clean_html($tag);
		}

		$message = addslashes($message);
	}
	else
	{
		$message = preg_replace($html_entities_match, $html_entities_replace, $message);
	}
Bei der von Dir veränderten FIND-Anweisung könnte man diesen Teil nicht finden, weil die letzte Zeile nicht in 2.0.20 enthalten ist.


Viele Grüße,
Bálint

Verfasst: 10.06.2006 15:38
von michi-w.
So sieht mein Original aus:

Code: Alles auswählen

	$message .= htmlspecialchars($part) . clean_html($tag);
		}

		$message = addslashes($message);
		$message = str_replace('"', '\"', $message);
	}
	else
Wie kommt das?

Verfasst: 10.06.2006 15:57
von Balint
Vielleicht hattest du einen der Updates von hier:

http://www.phpbb.de/viewtopic.php?t=121550&start=10

eingespielt? Und mit 2.0.21 wurde es "offiziell" behoben?


Viele Grüße,
Bálint