Last modified: 2014-09-23 23:07:07 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T20248, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 18248 - Redirects to Images on Shared Repos do not work
Redirects to Images on Shared Repos do not work
Status: NEW
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
1.15.x
All All
: Normal normal with 3 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 20804 23664 32658 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-30 04:21 UTC by OverlordQ
Modified: 2014-09-23 23:07 UTC (History)
16 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Proposed patch (2.00 KB, patch)
2009-09-25 14:08 UTC, Bryan Tong Minh
Details

Description OverlordQ 2009-03-30 04:21:29 UTC
If a page embeds local [[File:A.jpg]] which is then moved to [[File:B.jpg]] pages embedding [[File:A.jpg]] still work.  However if [[File:A.jpg]] is moved to a shared repo (such as commons) and deleted locally, all pages that embed [[File:A.jpg]] will contain broken images.

Ideally, these images should continue to work.
Comment 1 Chad H. 2009-03-30 15:16:28 UTC
Known bug, just hadn't filed it yet.

Assigning to Victor. He was the one who implemented this, and I've banged me head up against walls trying to get it work. Told me a week or so ago he'd take a poke.
Comment 2 Bryan Tong Minh 2009-09-25 13:21:39 UTC
*** Bug 20804 has been marked as a duplicate of this bug. ***
Comment 3 Bryan Tong Minh 2009-09-25 14:08:39 UTC
Created attachment 6582 [details]
Proposed patch

Patch that achieves the functionality.

The problem with this is that not only the local redirect table is followed, but also the foreign one i.e. if Commons file [[File:Redirect to A.jpg]] redirects to [[File:A.jpg]], it will first look to a local file named [[File:A.jpg]]. There are few options:
* First check the current repo and only then pass over control to wfFindFile
* Follow only redirects to non-local files from local files only
* Not care at all, as this behaviour is a reasonable default.
Comment 4 Chad H. 2009-11-28 04:41:55 UTC
Mass component change for merge of "File/Repo" and "Images and Files"
Comment 5 Chad H. 2010-02-19 14:14:55 UTC
*** Bug 22541 has been marked as a duplicate of this bug. ***
Comment 6 Derk-Jan Hartman 2010-02-19 16:28:24 UTC
Grah, I see I duplicated the effort on this :( I kinda assumed that there would not have been a patch yet in bugzilla for this.
Comment 7 Derk-Jan Hartman 2010-02-19 16:30:21 UTC
I'm not sure if this patch would work for ForeignAPIRepo though...
Comment 8 Bryan Tong Minh 2010-10-24 15:35:59 UTC
*** Bug 23664 has been marked as a duplicate of this bug. ***
Comment 9 Logan Rosen 2011-05-22 23:58:44 UTC
Confirming that file redirects to files on Commons on outreach.wikimedia and en.wiki only display the redirect and do not actually go through.

Example: http://outreach.wikimedia.org/wiki/File:Free_licenses.pdf
Sandbox: http://outreach.wikimedia.org/w/index.php?title=Sandbox&oldid=16242 (only links to file)
Comment 10 Chad H. 2011-05-23 00:05:24 UTC
Thanks. I confirmed it way back in comment 1.
Comment 11 Bryan Tong Minh 2011-11-26 18:06:01 UTC
*** Bug 32658 has been marked as a duplicate of this bug. ***
Comment 12 Sumana Harihareswara 2012-01-10 22:16:30 UTC
Comment on attachment 6582 [details]
Proposed patch

This patch no longer cleanly applies to SVN trunk per Rusty Burchfield's automated testing
https://docs.google.com/spreadsheet/ccc?key=0Ah_71HHl7qa7dGtvSms3TGpHQU9NU2Y1VmNzUEUteWc .
Comment 13 Sumana Harihareswara 2012-01-10 22:17:40 UTC
Bryan, sorry, removing the need-review and patch keywords since your patch doesn't cleanly apply to trunk.  Is it quick for you to update and reattach, or commit after the code slush ends?
Comment 14 Bryan Tong Minh 2012-01-11 10:33:39 UTC
(In reply to comment #13)
> Bryan, sorry, removing the need-review and patch keywords since your patch
> doesn't cleanly apply to trunk.  Is it quick for you to update and reattach, or
> commit after the code slush ends?

We first need to establish that the behaviour described in comment #3 is acceptable.
Comment 15 Rd232 2012-04-05 10:07:57 UTC
Is this the same as Bug 28299 - An image redirect from a foreign File Repo overrides local wiki page?
Comment 16 Strainu 2012-04-05 11:34:03 UTC
Not really. That bug talks about how having a (local) redirect affects all versions in the history of the page. This bug describes what happens if you create a local redirect to a file not found on the local Wiki.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links