Page MenuHomePhabricator

Arch/Performance review of ExtraLanguageLink for WMF deploy
Closed, DeclinedPublic

Details

Reference
bz64903

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:17 AM
bzimport set Reference to bz64903.
bzimport added a subscriber: Unknown Object (MLST).

I see you can find pages using this extension using Special:PagesWithProp or API action=query&list=pageswithprop. But there doesn't seem to be any way to search for pages using this extension for particular links short of reading and parsing the property for every page using the extension. Should there be such a way? I'd think yes.

For comparison, real langlinks can be queried using API action=query&list=langbacklinks and interwiki links using API action=query&list=iwbacklinks.

Gerrit change 111390 avoids this problem because the non-language langlinks added via that patch do get recorded in the langlinks table.

(In reply to Brad Jorsch from comment #1)

I see you can find pages using this extension using Special:PagesWithProp or
API action=query&list=pageswithprop. But there doesn't seem to be any way to
search for pages using this extension for particular links short of reading
and parsing the property for every page using the extension. Should there be
such a way? I'd think yes.

I can add an API module to search for uses of {{extralanguagelink}}, but I can't see any way of making it very efficient. Can API modules be marked expensive, or limited to a small number of results (10 or 20) at a time?

Gerrit change #111390 avoids this problem because the non-language langlinks
added via that patch do get recorded in the langlinks table.

It's not always appropriate to include extra language links in the langlinks table; hence this extension.

(In reply to This, that and the other from comment #2)

It's not always appropriate to include extra language links in the langlinks
table; hence this extension.

It seems odd to me that the links would be appropriate to display in the langlinks area of the web UI but not to include in the langlinks table.

Not storing them in langlinks might also have trouble with code (e.g. mobile) that fetches the langlinks via the API and constructs its own UI to display them.

I would welcome review/merge of Gerrit change 111390; if that patch proves insufficient for resolving the two dependencies of bug 64786 then we can pick this up again.

No longer needed. See parent task.