Seite 1 von 1
Doppelte Zeile bei Update auf 2.0.21
Verfasst: 17.09.2006 13:46
von 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...
Verfasst: 17.09.2006 15:30
von 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);
Verfasst: 17.09.2006 16:14
von Valerie Raghnall
bei mir is aber darunter nochmal
sprich die zeile ist doppelt
Verfasst: 17.09.2006 16:18
von Miriam
Doppelt hält besser.... Wenn es wirklich
dieselbe Codezeile ist, lösche diese....
Such Dir eine aus.....
Verfasst: 17.09.2006 16:27
von 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.
Verfasst: 17.09.2006 16:40
von 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*
Verfasst: 17.09.2006 16:50
von 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.
Verfasst: 17.09.2006 16:57
von Miriam
Du kannst Die Zeile auch auskommentieren mit // davor.
Verfasst: 19.09.2006 01:07
von easygo
Valerie Raghnall hat geschrieben: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.
Verfasst: 19.09.2006 19:44
von 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!