X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=blobdiff_plain;f=includes%2Fapi%2FApiQueryBase.php;h=25053345af615a651fc06bd9c785eb4792448368;hp=c92f037c9cc2e2b04ce82e5748d218b98dcbbc2a;hb=e5ef0fd0c6607dd34f6dee69d716b159662a0a34;hpb=04390e58fe1302824a0b586a21f6b09f8f64780d diff --git a/includes/api/ApiQueryBase.php b/includes/api/ApiQueryBase.php index c92f037c9c..25053345af 100644 --- a/includes/api/ApiQueryBase.php +++ b/includes/api/ApiQueryBase.php @@ -157,16 +157,14 @@ abstract class ApiQueryBase extends ApiBase { */ protected function addTables( $tables, $alias = null ) { if ( is_array( $tables ) ) { - if ( !is_null( $alias ) ) { + if ( $alias !== null ) { ApiBase::dieDebug( __METHOD__, 'Multiple table aliases not supported' ); } $this->tables = array_merge( $this->tables, $tables ); + } elseif ( $alias !== null ) { + $this->tables[$alias] = $tables; } else { - if ( !is_null( $alias ) ) { - $this->tables[$alias] = $tables; - } else { - $this->tables[] = $tables; - } + $this->tables[] = $tables; } } @@ -425,35 +423,6 @@ abstract class ApiQueryBase extends ApiBase { return Hooks::run( 'ApiQueryBaseProcessRow', [ $this, $row, &$data, &$hookData ] ); } - /** - * @deprecated since 1.33, use LinkFilter::getQueryConditions() instead - * @param string|null $query - * @param string|null $protocol - * @return null|string - */ - public function prepareUrlQuerySearchString( $query = null, $protocol = null ) { - wfDeprecated( __METHOD__, '1.33' ); - $db = $this->getDB(); - if ( $query !== null && $query !== '' ) { - if ( is_null( $protocol ) ) { - $protocol = 'http://'; - } - - $likeQuery = LinkFilter::makeLikeArray( $query, $protocol ); - if ( !$likeQuery ) { - $this->dieWithError( 'apierror-badquery' ); - } - - $likeQuery = LinkFilter::keepOneWildcard( $likeQuery ); - - return 'el_index ' . $db->buildLike( $likeQuery ); - } elseif ( !is_null( $protocol ) ) { - return 'el_index ' . $db->buildLike( "$protocol", $db->anyString() ); - } - - return null; - } - /** * Filters hidden users (where the user doesn't have the right to view them) * Also adds relevant block information @@ -510,7 +479,7 @@ abstract class ApiQueryBase extends ApiBase { * @param string $prefix Module prefix */ public static function addTitleInfo( &$arr, $title, $prefix = '' ) { - $arr[$prefix . 'ns'] = intval( $title->getNamespace() ); + $arr[$prefix . 'ns'] = (int)$title->getNamespace(); $arr[$prefix . 'title'] = $title->getPrefixedText(); } @@ -524,7 +493,7 @@ abstract class ApiQueryBase extends ApiBase { $result = $this->getResult(); ApiResult::setIndexedTagName( $data, $this->getModulePrefix() ); - return $result->addValue( [ 'query', 'pages', intval( $pageId ) ], + return $result->addValue( [ 'query', 'pages', (int)$pageId ], $this->getModuleName(), $data ); } @@ -532,7 +501,7 @@ abstract class ApiQueryBase extends ApiBase { /** * Same as addPageSubItems(), but one element of $data at a time * @param int $pageId Page ID - * @param array $item Data array à la ApiResult + * @param mixed $item Data à la ApiResult * @param string|null $elemname XML element name. If null, getModuleName() * is used * @return bool Whether the element fit in the result