forcemerge 612267 612227
tag 612267 + patch
thanks

> Some sources on the net say, that this extension does not work in Mediawiki 
> 1.15 and PHP 5.3.

While we will probably want to upload a new version of this package with
a newer ConfirmEdit, this bug affects stable, and so I propose this
minimal patch to fix it.

SR

-- 
Stefano Rivera
  http://tumbleweed.org.za/
  H: +27 21 465 6908 C: +27 72 419 8559  UCT: x3127
Description: PHP 5.3 compatibility fixes for ConfirmEdit.
Origin:
 http://www.mediawiki.org/wiki/Special:Code/MediaWiki/53461
 http://www.mediawiki.org/wiki/Special:Code/MediaWiki/55449
Bug-Debian: http://bugs.debian.org/612267
Bug-Wikimedia: https://bugzilla.wikimedia.org/show_bug.cgi?id=20383
Last-Update: 2011-02-14
--- a/dist/mediawiki-extensions-confirmedit/usr/share/mediawiki-extensions/confirmedit/ConfirmEdit_body.php
+++ b/dist/mediawiki-extensions-confirmedit/usr/share/mediawiki-extensions/confirmedit/ConfirmEdit_body.php
@@ -15,15 +15,15 @@
 		return $wgCaptcha;
 	}
 
-	static function confirmEdit( &$editPage, $newtext, $section ) {
+	static function confirmEdit( $editPage, $newtext, $section ) {
 		return self::getInstance()->confirmEdit( $editPage, $newtext, $section );
 	}
 
-	static function confirmEditMerged( &$editPage, $newtext ) {
+	static function confirmEditMerged( $editPage, $newtext ) {
 		return self::getInstance()->confirmEditMerged( $editPage, $newtext );
 	}
 	
-	static function confirmEditAPI( &$editPage, $newtext, &$resultArr ) {
+	static function confirmEditAPI( $editPage, $newtext, &$resultArr ) {
 		return self::getInstance()->confirmEditAPI( $editPage, $newtext, $resultArr );
 	}
 
@@ -458,7 +458,7 @@
 	 * Backend function for confirmEdit() and confirmEditAPI()
 	 * @return bool false if the CAPTCHA is rejected, true otherwise
 	 */
-	private function doConfirmEdit( &$editPage, $newtext, $section, $merged = false ) {
+	private function doConfirmEdit( $editPage, $newtext, $section, $merged = false ) {
 		if( $this->shouldCheck( $editPage, $newtext, $section, $merged ) ) {
 			if( $this->passCaptcha() ) {
 				return true;
@@ -479,7 +479,7 @@
 	 * @param bool $merged
 	 * @return bool true to continue saving, false to abort and show a captcha form
 	 */
-	function confirmEdit( &$editPage, $newtext, $section, $merged = false ) {
+	function confirmEdit( $editPage, $newtext, $section, $merged = false ) {
 		if( defined('MW_API') ) {
 			# API mode
 			# The CAPTCHA was already checked and approved 
@@ -497,12 +497,12 @@
 	 * @param EditPage $editPage
 	 * @param string $newtext
 	 */
-	function confirmEditMerged( &$editPage, $newtext ) {
+	function confirmEditMerged( $editPage, $newtext ) {
 		return $this->confirmEdit( $editPage, $newtext, false, true );
 	}
 	
 	
-	function confirmEditAPI( &$editPage, $newtext, &$resultArr) {
+	function confirmEditAPI( $editPage, $newtext, &$resultArr) {
 		if( !$this->doConfirmEdit( $editPage, $newtext, false, false ) ) {
 			$this->addCaptchaAPI($resultArr);
 			return false;

Reply via email to