From: Aryeh Gregor <simetrical@users.mediawiki.org>
Date: Mon, 26 May 2008 12:15:21 +0000 (+0000)
Subject: No, something that looks stupid *is* in fact ipso facto stupid -- unless it includes... 
X-Git-Tag: 1.31.0-rc.0~47376
X-Git-Url: http://git.cyclocoop.org/%7B%24admin_url%7Dmembres/cotisations/gestion/rappel_supprimer.php?a=commitdiff_plain;h=9eb97be79347e73b28201d12819c65169ca9dc80;p=lhc%2Fweb%2Fwiklou.git

No, something that looks stupid *is* in fact ipso facto stupid -- unless it includes a comment to explain why it's not.
---

diff --git a/includes/api/ApiDelete.php b/includes/api/ApiDelete.php
index 9ab29ba699..5f1dd476be 100644
--- a/includes/api/ApiDelete.php
+++ b/includes/api/ApiDelete.php
@@ -121,6 +121,8 @@ class ApiDelete extends ApiBase {
 		// Auto-generate a summary, if necessary
 		if(is_null($reason))
 		{
+			# Need to pass a throwaway variable because generateReason expects
+			# a reference
 			$hasHistory = false;
 			$reason = $article->generateReason($hasHistory);
 			if($reason === false)