No, something that looks stupid *is* in fact ipso facto stupid -- unless it includes...

diff --git a/includes/api/ApiDelete.php b/includes/api/ApiDelete.php
index 9ab29ba..5f1dd47 100644 (file)
--- 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)