Revert r88722 (allow multiple values for letype): this won't play nice with query...

diff --git a/RELEASE-NOTES-1.19 b/RELEASE-NOTES-1.19
index 3508b39..5b40da3 100644 (file)
--- a/RELEASE-NOTES-1.19
+++ b/RELEASE-NOTES-1.19
@@ -93,7 +93,6 @@ production.
   title inputs
 * (bug 20699) API watchlist should list log-events
 * (bug 29070) Add token to action=watch
-* (bug 29120) ApiQueryLogEvents: let letype accept multiple values
 * (bug 29221) Expose oldrevid in watchlist output
 
 === Languages updated in 1.19 ===

diff --git a/includes/api/ApiQueryLogEvents.php b/includes/api/ApiQueryLogEvents.php
index 8370244..dd43f9b 100644 (file)
--- a/includes/api/ApiQueryLogEvents.php
+++ b/includes/api/ApiQueryLogEvents.php
@@ -111,7 +111,8 @@ class ApiQueryLogEvents extends ApiQueryBase {
                        list( $type, $action ) = explode( '/', $params['action'] );
                        $this->addWhereFld( 'log_type', $type );
                        $this->addWhereFld( 'log_action', $action );
-               } else if ( !is_null( $params['type'] ) ) {
+               }
+               else if ( !is_null( $params['type'] ) ) {
                        $this->addWhereFld( 'log_type', $params['type'] );
                        $index['logging'] = 'type_time';
                }
@@ -370,8 +371,7 @@ class ApiQueryLogEvents extends ApiQueryBase {
                                )
                        ),
                        'type' => array(
-                               ApiBase::PARAM_TYPE => $wgLogTypes,
-                               ApiBase::PARAM_ISMULTI => true,
+                               ApiBase::PARAM_TYPE => $wgLogTypes
                        ),
                        'action' => array(
                                ApiBase::PARAM_TYPE => array_keys( $wgLogActions )