Search engine fails in Special:IndexPage
Closed, ResolvedPublicBUG REPORT
Actions

Description

In Special:IndexPages calls to MediaWiki SearchEngine fails because they do not return SearchResultSet.

Example to reproduce: https://en.wikisource.org/wiki/Special:IndexPages?key=+intitle%3AJaures

Reported first in frwikisource: https://fr.wikisource.org/wiki/Wikisource:Scriptorium/Septembre_2019#%22_le_moteur_de_recherche_ne_fonctionne_pas%22

Details

	Subject	Repo	Branch	Lines +/-
	Do not use instanceof SearchResultSet	mediawiki/extensions/ProofreadPage	master	+3 -3

Customize query in gerrit

Related Objects

Mentioned In: T237835: Unclear that proofreadpage_specialpage_searcherrortext is an error
T237831: Wikisource List of index pages does not work

Event Timeline

Tpt created this task.Sep 12 2019, 8:59 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 12 2019, 8:59 AM

Aklapper added a project: ProofreadPage.Sep 12 2019, 10:08 AM

Hsarrazin added a subscriber: Hsarrazin.Sep 12 2019, 12:49 PM

Tpt added projects: Discovery-Search, MediaWiki-Search.Nov 6 2019, 10:35 AM

Priority: High for Wikisource (breaks a special page)

@EBernhardson This bug seems to be related to the change in MediaWiki search PHP API. May someone from your team investigate it?

Xover added a subscriber: Xover.Nov 9 2019, 10:39 AM

Jan.Kamenicek mentioned this in T237831: Wikisource List of index pages does not work.Nov 9 2019, 9:35 PM

Reedy merged a task: T237831: Wikisource List of index pages does not work.Nov 9 2019, 9:52 PM

Reedy mentioned this in T237835: Unclear that proofreadpage_specialpage_searcherrortext is an error.

Reedy added subscribers: Jan.Kamenicek, Reedy.

Hey @Tpt, does this help?

reedy@mwdebug1002:~$ mwscript eval.php enwikisource
> $indexNamespaceId = ProofreadPage::getIndexNamespaceId();

> $searchEngine = MediaWiki\MediaWikiServices::getInstance()->getSearchEngineFactory()->create();

> $searchEngine->setLimitOffset( 10, 0 );

> $searchEngine->setNamespaces( [ 106 ] );

> $searchEngine->showRedirects = false;

> $status = $searchEngine->searchText( 'foo' );

> var_dump( gettype( $status ) );
string(6) "object"

> var_dump( $status instanceof Status );
bool(true)

> var_dump( $status->isOK() );
bool(true)

> var_dump( gettype( $status->getValue() ) );
string(6) "object"

> var_dump( $status->getValue() instanceof SearchResultSet  );
bool(false)

> var_dump( get_class( $status->getValue() ) );
string(125) "class@anonymous/srv/mediawiki/php-1.35.0-wmf.5/extensions/CirrusSearch/includes/Search/FullTextResultsType.php0x7fd58bb52d4d"

$status->getValue() is returning the anonymous class (which subclasses BaseCirrusSearchResultSet) at https://github.com/wikimedia/mediawiki-extensions-CirrusSearch/blob/master/includes/Search/FullTextResultsType.php#L79

Which is making a soft dependancy on the CirrusSearch extension...

Reedy added a project: Regression.Nov 10 2019, 1:10 AM

This shouldn't be checking SearchResultSet, but rather ISearchResultSet. Cirrus's returned object now extends from BaseSearchResultSet in core that implements ISearchResultSet.

Change 550556 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/ProofreadPage@master] Do not use instanceof SearchResultSet

https://gerrit.wikimedia.org/r/550556

gerritbot added a project: Patch-For-Review.Nov 12 2019, 9:30 PM

Change 550556 merged by jenkins-bot:
[mediawiki/extensions/ProofreadPage@master] Do not use instanceof SearchResultSet

https://gerrit.wikimedia.org/r/550556

EBernhardson assigned this task to dcausse.Nov 12 2019, 10:40 PM

EBernhardson edited projects, added Discovery-Search (Current work); removed Discovery-Search.

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.8; 2019-11-26).Nov 13 2019, 12:01 AM

dcausse moved this task from Incoming to Needs Reporting on the Discovery-Search (Current work) board.Nov 13 2019, 10:51 AM

Gehel closed this task as Resolved.Nov 13 2019, 4:27 PM