Page MenuHomePhabricator

Modify Special:MediaSearch to use the MediaQueryBuilder
Closed, DeclinedPublic


ATM Special:MediaSearch uses an api that wraps the search api

When is merged we need to change Special:MediaSearch to use the new backend. ATM the new backend will only be activated if mediasearch=1 is in the request url, and the search is in the File namespace

Acceptance criteria:

  • Special:MediaSearch uses MediaQueryBuilder (possibly via the search api) instead of making multiple calls to the search api using haswbstatement etc

This is not something that can be QAd, except for to make sure nothing has broken

Event Timeline

Cparle created this task.Jul 7 2020, 10:05 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 7 2020, 10:05 AM
Cparle updated the task description. (Show Details)Jul 7 2020, 10:12 AM

Change 610281 had a related patch set uploaded (by Matthias Mullie; owner: Matthias Mullie):
[mediawiki/extensions/WikibaseMediaInfo@master] Pass mediasearch API requests along to plain old search (with mediasearch profile)

I've rewritten the existing API to proxy to the search API because it'd break the frontend rewrite.
Once that is complete, we can transition away from the old API endpoint altogether.

CBogen added a subscriber: CBogen.

Nothing that this can be merged independently of the MediaSearch vue.js port (T251940).

matthiasmullie closed this task as Declined.Sep 8 2020, 1:13 PM

This has taken too long to land & we no longer need this intermediate step because the Vue work has already been merged.
I've worked around the blocker for T258063, which makes this ticket obsolete.

Change 610281 abandoned by Matthias Mullie:
[mediawiki/extensions/WikibaseMediaInfo@master] Pass mediasearch API requests along to plain old search (with mediasearch profile)

Superseded by /625892/