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

This commit is contained in:
Roan Kattouw 2011-06-02 16:13:42 +00:00
parent b83905dae3
commit ec83e2da6e
2 changed files with 3 additions and 4 deletions

View file

@ -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 ===

View file

@ -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 )