Drop the `redirect=no` thing for non-redirects, per Brion. Seems like a waste of...

diff --git a/includes/ProtectionForm.php b/includes/ProtectionForm.php
index 8ec5fd4..f349311 100644 (file)
--- a/includes/ProtectionForm.php
+++ b/includes/ProtectionForm.php
@@ -76,11 +76,12 @@ class ProtectionForm {
        }
        
        function execute() {
-               global $wgRequest;
+               global $wgRequest, $wgOut;
                if( $wgRequest->wasPosted() ) {
                        if( $this->save() ) {
-                               global $wgOut;
-                               $wgOut->redirect( $this->mTitle->getFullUrl( 'redirect=no' ) );
+                               $article = new Article( $this->mTitle );
+                               $q = $article->isRedirect() ? 'redirect=no' : '';
+                               $wgOut->redirect( $this->mTitle->getFullUrl( $q ) );
                        }
                } else {
                        $this->show();