| Index: trunk/extensions/ReaderFeedback/specialpages/RatedPages_body.php |
| — | — | @@ -17,13 +17,16 @@ |
| 18 | 18 | $this->setHeaders(); |
| 19 | 19 | if( $wgUser->isAllowed( 'feedback' ) ) { |
| 20 | 20 | if( $wgUser->isBlocked() ) { |
| 21 | | - return $wgOut->blockedPage(); |
| | 21 | + $wgOut->blockedPage(); |
| | 22 | + return; |
| 22 | 23 | } |
| 23 | 24 | } else { |
| 24 | | - return $wgOut->permissionRequired( 'feedback' ); |
| | 25 | + $wgOut->permissionRequired( 'feedback' ); |
| | 26 | + return; |
| 25 | 27 | } |
| 26 | 28 | if( wfReadOnly() ) { |
| 27 | | - return $wgOut->readOnlyPage(); |
| | 29 | + $wgOut->readOnlyPage(); |
| | 30 | + return; |
| 28 | 31 | } |
| 29 | 32 | // Purge expired entries on one in every 10 queries |
| 30 | 33 | if( 0 == mt_rand( 0, 10 ) ) { |
| Index: trunk/extensions/ReaderFeedback/specialpages/ReaderFeedback_body.php |
| — | — | @@ -27,13 +27,16 @@ |
| 28 | 28 | $confirm = $wgRequest->wasPosted() && $wgUser->matchEditToken( $wgRequest->getVal( 'wpEditToken' ) ); |
| 29 | 29 | if( $wgUser->isAllowed( 'feedback' ) ) { |
| 30 | 30 | if( $wgUser->isBlocked( !$confirm ) ) { |
| 31 | | - return $wgOut->blockedPage(); |
| | 31 | + $wgOut->blockedPage(); |
| | 32 | + return; |
| 32 | 33 | } |
| 33 | 34 | } else { |
| 34 | | - return $wgOut->permissionRequired( 'feedback' ); |
| | 35 | + $wgOut->permissionRequired( 'feedback' ); |
| | 36 | + return; |
| 35 | 37 | } |
| 36 | 38 | if( wfReadOnly() ) { |
| 37 | | - return $wgOut->readOnlyPage(); |
| | 39 | + $wgOut->readOnlyPage(); |
| | 40 | + return; |
| 38 | 41 | } |
| 39 | 42 | $this->setHeaders(); |
| 40 | 43 | # Our target page |