Stop mutually exclusive values in ApiDelete and ApiMove
authorSam Reed <reedy@users.mediawiki.org>
Thu, 25 Mar 2010 21:11:38 +0000 (21:11 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Thu, 25 Mar 2010 21:11:38 +0000 (21:11 +0000)
includes/api/ApiDelete.php
includes/api/ApiMove.php

index 9689c97..aca6b63 100644 (file)
@@ -52,6 +52,10 @@ class ApiDelete extends ApiBase {
                $params = $this->extractRequestParams();
 
                $this->requireOnlyOneParameter( $params, 'title', 'pageid' );
+               
+               if ( isset( $params['watch'] ) && params( $show['unwatch'] ) ) {
+                       $this->dieUsageMsg( array( 'show' ) );
+               }
 
                if ( isset( $params['title'] ) ) {
                        $titleObj = Title::newFromText( $params['title'] );
index 3b2fe99..f7a5f9a 100644 (file)
@@ -39,6 +39,11 @@ class ApiMove extends ApiBase {
        public function execute() {
                global $wgUser;
                $params = $this->extractRequestParams();
+               
+               if ( isset( $params['watch'] ) && params( $show['unwatch'] ) ) {
+                       $this->dieUsageMsg( array( 'show' ) );
+               }
+               
                if ( is_null( $params['reason'] ) ) {
                        $params['reason'] = '';
                }