Page MenuHomePhabricator

Use interwiki prefix when giving list of foreign file duplicates
Open, LowPublic

Description

File links in the dupe list should use the class "extiw" for shared files and no class for local files. Moreover, the title needs to be corrected to "commons:Image:bla.jpg" resp. "Image:bla.jpg".


Version: 1.13.x
Severity: enhancement
URL: http://de.wikipedia.org/wiki/Bild:050730_pascha_032.jpg

Details

Reference
bz14148

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 10:11 PM
bzimport set Reference to bz14148.
bzimport added a subscriber: Unknown Object (MLST).

Bryan.TongMinh wrote:

The problem is that ForeignDBRepo does not provide the interwiki link to the repository it is pointing to. It is even possible that there is no interwiki pointing to the foreign repo and that external links are the only way to link to it.

Would it be a good idea to add a key to $wgForeignFileRepos, like 'interwiki' and add in CommonSettings.php 'interwiki' => 'commons'?

Component: Images -> Foreign file/repo
Assigned: demon

(In reply to comment #2)

Would it be a good idea to add a key to $wgForeignFileRepos, like 'interwiki'
and add in CommonSettings.php 'interwiki' => 'commons'?

What if there's no associated interwiki for said repo? Fail back to the current syntax of using ELs?

Bryan.TongMinh wrote:

Construct a sane default interwiki. Mixing interwikis and external links does not sound a good idea.

(In reply to comment #5)

Construct a sane default interwiki. Mixing interwikis and external links does
not sound a good idea.

Not possible, as there is no connection between a foreign repo and an interwiki link. Nor do I see a compelling reason to change this at present.

Maybe we can slap plainlinks on these so we can ditch the EL icons though.

Mass component change for merge of "File/Repo" and "Images and Files"