Benutzer:Rabe/Spielwiese
Aus Stadtwiki
Test der News-Extension
... scheint ein Problem mit PHP-5.3.2 zu sein, das ist etwas fummelig bei by-reference / by-value calls:
[Wed May 05 18:40:36 2010] [error] [client 91.89.xx.xxx] PHP Warning: Parameter 3 to wfNewsTag() expected to be a reference, value given in /data/www/fuwa.stadtwiki.info/htdocs/includes/parser/Parser.php on line 3243, referer: http://fuwa.stadtwiki.info/index.php5?title=Benutzer:Rabe/Spielwiese&action=edit [Wed May 05 18:40:50 2010] [error] [client 91.89.xx.xxx] PHP Warning: Parameter 3 to wfNewsTag() expected to be a reference, value given in /data/www/fuwa.stadtwiki.info/htdocs/includes/parser/Parser.php on line 3243, referer: http://fuwa.stadtwiki.info/index.php5?title=Benutzer:Rabe/Spielwiese&action=submit
Mein Versuch das symptomatisch zu patchen:
--- extensions/News/News.php.orig 2010-05-05 18:33:44.000000000 +0200
+++ extensions/News/News.php 2010-05-05 18:35:02.000000000 +0200
@@ -48,7 +48,7 @@
wfLoadExtensionMessages( 'NewsExtension' );
}
-function wfNewsTag( $templatetext, $argv, &$parser ) {
+function wfNewsTag( $templatetext, $argv, $parser ) {
global $wgTitle;
$parser->disableCache(); //TODO: use smart cache & purge...?
@@ -57,7 +57,7 @@
return $renderer->renderNews();
}
-function wfNewsFeedTag( $templatetext, $argv, &$parser ) {
+function wfNewsFeedTag( $templatetext, $argv, $parser ) {
global $wgTitle, $wgOut;
$parser->disableCache(); //TODO: use smart cache & purge...?
@@ -74,7 +74,7 @@
return $html;
}
-function wfNewsFeedLinkTag( $linktext, $argv, &$parser ) {
+function wfNewsFeedLinkTag( $linktext, $argv, $parser ) {
return NewsRenderer::renderFeedLink($linktext, $argv, $parser);
}
... ohne jetzt die genauen Deatils verstanden zu haben, aber $parser ist in includes/parser/Parser.php:3243 als "$this" aufgerufen wird, also das Parserobjekt selbst. $this sollte IMHO immer nur eine Referenz sein, d.h. ob die Funktion jetzt "&$parser" oder "$parser" verwendet sollte gleichbedeutend sein. ... vermutlich irre ich mich an dieser Stelle, denn genau obiger Patch hat nicht funktioniert.--Rabe 16:45, 5. Mai 2010 (UTC)
- Nachtrag
- Auch mit der -trunk-Version von News klappt es nicht wie erwartet, hab folgende beide Versionen probiert:
/home/fuwa/News-MW1.15-r48578.tar.gz /home/fuwa/News-trunk-r65527.tar.gz
- --Rabe 16:50, 5. Mai 2010 (UTC)