stylize.php on API code
[lhc/web/wiklou.git] / includes / api / ApiDelete.php
index ebfaf3e..bffbfc8 100644 (file)
@@ -22,9 +22,9 @@
  * http://www.gnu.org/copyleft/gpl.html
  */
 
-if (!defined('MEDIAWIKI')) {
+if ( !defined( 'MEDIAWIKI' ) ) {
        // Eclipse helper - will be ignored in production
-       require_once ("ApiBase.php");
+       require_once ( "ApiBase.php" );
 }
 
 
@@ -36,8 +36,8 @@ if (!defined('MEDIAWIKI')) {
  */
 class ApiDelete extends ApiBase {
 
-       public function __construct($main, $action) {
-               parent :: __construct($main, $action);
+       public function __construct( $main, $action ) {
+               parent :: __construct( $main, $action );
        }
 
        /**
@@ -51,59 +51,59 @@ class ApiDelete extends ApiBase {
                global $wgUser;
                $params = $this->extractRequestParams();
 
-               $this->requireOnlyOneParameter($params, 'title', 'pageid');
-               if(!isset($params['token']))
-                       $this->dieUsageMsg(array('missingparam', 'token'));
+               $this->requireOnlyOneParameter( $params, 'title', 'pageid' );
+               if ( !isset( $params['token'] ) )
+                       $this->dieUsageMsg( array( 'missingparam', 'token' ) );
 
-               if(isset($params['title']))
+               if ( isset( $params['title'] ) )
                {
-                       $titleObj = Title::newFromText($params['title']);
-                       if(!$titleObj)
-                               $this->dieUsageMsg(array('invalidtitle', $params['title']));
+                       $titleObj = Title::newFromText( $params['title'] );
+                       if ( !$titleObj )
+                               $this->dieUsageMsg( array( 'invalidtitle', $params['title'] ) );
                }
-               else if(isset($params['pageid']))
+               else if ( isset( $params['pageid'] ) )
                {
-                       $titleObj = Title::newFromID($params['pageid']);
-                       if(!$titleObj)
-                               $this->dieUsageMsg(array('nosuchpageid', $params['pageid']));
+                       $titleObj = Title::newFromID( $params['pageid'] );
+                       if ( !$titleObj )
+                               $this->dieUsageMsg( array( 'nosuchpageid', $params['pageid'] ) );
                }
-               if(!$titleObj->exists())
-                       $this->dieUsageMsg(array('notanarticle'));
+               if ( !$titleObj->exists() )
+                       $this->dieUsageMsg( array( 'notanarticle' ) );
 
-               $reason = (isset($params['reason']) ? $params['reason'] : null);
-               if ($titleObj->getNamespace() == NS_FILE) {
-                       $retval = self::deleteFile($params['token'], $titleObj, $params['oldimage'], $reason, false);
-                       if(count($retval))
+               $reason = ( isset( $params['reason'] ) ? $params['reason'] : null );
+               if ( $titleObj->getNamespace() == NS_FILE ) {
+                       $retval = self::deleteFile( $params['token'], $titleObj, $params['oldimage'], $reason, false );
+                       if ( count( $retval ) )
                                // We don't care about multiple errors, just report one of them
-                               $this->dieUsageMsg(reset($retval));
+                               $this->dieUsageMsg( reset( $retval ) );
                } else {
-                       $articleObj = new Article($titleObj);
-                       $retval = self::delete($articleObj, $params['token'], $reason);
+                       $articleObj = new Article( $titleObj );
+                       $retval = self::delete( $articleObj, $params['token'], $reason );
                        
-                       if(count($retval))
+                       if ( count( $retval ) )
                                // We don't care about multiple errors, just report one of them
-                               $this->dieUsageMsg(reset($retval));
+                               $this->dieUsageMsg( reset( $retval ) );
                        
-                       if($params['watch'] || $wgUser->getOption('watchdeletion'))
+                       if ( $params['watch'] || $wgUser->getOption( 'watchdeletion' ) )
                                $articleObj->doWatch();
-                       else if($params['unwatch'])
+                       else if ( $params['unwatch'] )
                                $articleObj->doUnwatch();
                }
 
-               $r = array('title' => $titleObj->getPrefixedText(), 'reason' => $reason);
-               $this->getResult()->addValue(null, $this->getModuleName(), $r);
+               $r = array( 'title' => $titleObj->getPrefixedText(), 'reason' => $reason );
+               $this->getResult()->addValue( null, $this->getModuleName(), $r );
        }
 
-       private static function getPermissionsError(&$title, $token) {
+       private static function getPermissionsError( &$title, $token ) {
                global $wgUser;
                
                // Check permissions
-               $errors = $title->getUserPermissionsErrors('delete', $wgUser);
-               if (count($errors) > 0) return $errors;
+               $errors = $title->getUserPermissionsErrors( 'delete', $wgUser );
+               if ( count( $errors ) > 0 ) return $errors;
                
                // Check token
-               if(!$wgUser->matchEditToken($token))
-                       return array(array('sessionfailure'));
+               if ( !$wgUser->matchEditToken( $token ) )
+                       return array( array( 'sessionfailure' ) );
                return array();
        }
 
@@ -115,62 +115,62 @@ class ApiDelete extends ApiBase {
         * @param string $reason - Reason for the deletion. Autogenerated if NULL
         * @return Title::getUserPermissionsErrors()-like array
         */
-       public static function delete(&$article, $token, &$reason = null)
+       public static function delete( &$article, $token, &$reason = null )
        {
                global $wgUser;
-               if($article->isBigDeletion() && !$wgUser->isAllowed('bigdelete')) {
+               if ( $article->isBigDeletion() && !$wgUser->isAllowed( 'bigdelete' ) ) {
                        global $wgDeleteRevisionsLimit;
-                       return array(array('delete-toobig', $wgDeleteRevisionsLimit));
+                       return array( array( 'delete-toobig', $wgDeleteRevisionsLimit ) );
                }
                $title = $article->getTitle();
-               $errors = self::getPermissionsError($title, $token);
-               if (count($errors)) return $errors;
+               $errors = self::getPermissionsError( $title, $token );
+               if ( count( $errors ) ) return $errors;
 
                // Auto-generate a summary, if necessary
-               if(is_null($reason))
+               if ( is_null( $reason ) )
                {
                        # Need to pass a throwaway variable because generateReason expects
                        # a reference
                        $hasHistory = false;
-                       $reason = $article->generateReason($hasHistory);
-                       if($reason === false)
-                               return array(array('cannotdelete'));
+                       $reason = $article->generateReason( $hasHistory );
+                       if ( $reason === false )
+                               return array( array( 'cannotdelete' ) );
                }
 
                $error = '';
-               if (!wfRunHooks('ArticleDelete', array(&$article, &$wgUser, &$reason, $error)))
-                       $this->dieUsageMsg(array('hookaborted', $error));
+               if ( !wfRunHooks( 'ArticleDelete', array( &$article, &$wgUser, &$reason, $error ) ) )
+                       $this->dieUsageMsg( array( 'hookaborted', $error ) );
 
                // Luckily, Article.php provides a reusable delete function that does the hard work for us
-               if($article->doDeleteArticle($reason)) {
-                       wfRunHooks('ArticleDeleteComplete', array(&$article, &$wgUser, $reason, $article->getId()));
+               if ( $article->doDeleteArticle( $reason ) ) {
+                       wfRunHooks( 'ArticleDeleteComplete', array( &$article, &$wgUser, $reason, $article->getId() ) );
                        return array();
                }
-               return array(array('cannotdelete', $article->mTitle->getPrefixedText()));
+               return array( array( 'cannotdelete', $article->mTitle->getPrefixedText() ) );
        }
 
-       public static function deleteFile($token, &$title, $oldimage, &$reason = null, $suppress = false)
+       public static function deleteFile( $token, &$title, $oldimage, &$reason = null, $suppress = false )
        {
-               $errors = self::getPermissionsError($title, $token);
-               if (count($errors)) return $errors;
+               $errors = self::getPermissionsError( $title, $token );
+               if ( count( $errors ) ) return $errors;
 
-               if( $oldimage && !FileDeleteForm::isValidOldSpec($oldimage) )
-                       return array(array('invalidoldimage'));
+               if ( $oldimage && !FileDeleteForm::isValidOldSpec( $oldimage ) )
+                       return array( array( 'invalidoldimage' ) );
 
                $file = wfFindFile( $title, array( 'ignoreRedirect' => true ) );
                $oldfile = false;
                
-               if( $oldimage )
+               if ( $oldimage )
                        $oldfile = RepoGroup::singleton()->getLocalRepo()->newFromArchiveName( $title, $oldimage );
                        
-               if( !FileDeleteForm::haveDeletableFile($file, $oldfile, $oldimage) )
-                       return self::delete(new Article($title), $token, $reason);
-               if (is_null($reason)) # Log and RC don't like null reasons
+               if ( !FileDeleteForm::haveDeletableFile( $file, $oldfile, $oldimage ) )
+                       return self::delete( new Article( $title ), $token, $reason );
+               if ( is_null( $reason ) ) # Log and RC don't like null reasons
                        $reason = '';
                $status = FileDeleteForm::doDelete( $title, $file, $oldimage, $reason, $suppress );
                                
-               if( !$status->isGood() )
-                       return array(array('cannotdelete', $title->getPrefixedText()));
+               if ( !$status->isGood() )
+                       return array( array( 'cannotdelete', $title->getPrefixedText() ) );
                        
                return array();
        }