Doppelte Zeile bei Update auf 2.0.21

[Valerie Raghnall]

Hi,

mir ist da grade was beim Update auf 2.0.21 aufgefallen... und zwar ergeben sich bei folgender Anweisung 2 gleiche Zeilen:

Code: Alles auswählen

#-----[ OPEN ]--------------------------------------------- 
# 
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);

... damit hab ich dann 2mal die letzte Zeile... ist das Absicht oder ein Fehler?

Ich hab zwar danach gesucht, aber zu dem Thema hier nichts gefunden, sollte ich eine Lösung dafür übersehen haben, tut's mir sorry...

[Miriam]

Wo hast Du 2x die gleiche Zeile???
Nach dem Ersetzen steht da nur noch:

Code: Alles auswählen

         $message .= preg_replace($html_entities_match, $html_entities_replace, $part) . clean_html($tag);
      }

      $message = addslashes($message);
      $message = str_replace('"', '\"', $message);

[Valerie Raghnall]

bei mir is aber darunter nochmal

Code: Alles auswählen

$message = str_replace('"', '\"', $message);

sprich die zeile ist doppelt

[Miriam]

Doppelt hält besser.... Wenn es wirklich dieselbe Codezeile ist, lösche diese....
Such Dir eine aus.....

[Valerie Raghnall]

wenn es nicht die selbe wäre, hätt ich ja nu kaum geschrieben, dass es die selbe 2x gibt, oder?

meine frage ist einfach ob das so gedacht ist oder nicht. wenn es nämlich so gedacht sein sollte - aus welchem grund immer, ich bin ja nun kein programmierer - dann wär's ziemlich bescheuert die zeile einfach zu löschen.

[Miriam]

Du kannst sie auch gern drin lassen, dann macht das Script eben zweimal dasselbe. Und das ist oftmals nicht das gleiche.
Im ersten str_replace() würde aus " \" und dann würde \\" draus werden. (Also: " --> \" --> \\")

Doppelte Maskierung würde imho nix bringen... ausser Ärger.

P.S.:

und zwar ergeben sich bei folgender Anweisung 2 gleiche Zeilen:

Leider habe ich nicht herausgelesen, dass es zweimal dieselbe Zeile ist. *sry*

[Valerie Raghnall]

dann versuch ich es einfach mal mit löschen, mit dem zeug drin hats keine probleme gegeben beim quoten bisher...

naja und gleich = gleich... also das selbe und 2 gleiche wäre dann 2x das selbe... aber deutsch ist eben nicht deutsch wie mir scheint.

[Miriam]

Du kannst Die Zeile auch auskommentieren mit // davor.

[easygo]

wenn es nicht die selbe wäre, hätt ich ja nu kaum geschrieben, dass es die selbe 2x gibt, oder?

meine frage ist einfach ob das so gedacht ist oder nicht. wenn es nämlich so gedacht sein sollte - aus welchem grund immer, ich bin ja nun kein programmierer - dann wär's ziemlich bescheuert die zeile einfach zu löschen.

Nö, ist so nicht gedacht. Das Standard phpBB 2.0.20 kam ohne die Zeile aus / -->

Code: Alles auswählen

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

		$message = addslashes($message);
	}
	else
	{
		$message = preg_replace($html_entities_match, $html_entities_replace, $message);
	}

und davon wird nun mal ausgegangen in den Code Changes.

[TheGardner]

bei michens gabs die Probleme beim Zitieren!

[ quote = "Mich Selbst" ]

wurde als

[ quote = \"MichSelbst\" ]

ausgeschildert! Damit ging das Zitieren völlig gegen den Baum - zumindest Guck-technisch! Also wech mitta doppelten Zeile und es geht dann wieder wie in vorherigen Versionen!