Last modified: 2014-11-14 21:56:12 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 T71678, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69678 - Track down popular gadgets that refer to files in skins/common/ and fix them not to
Track down popular gadgets that refer to files in skins/common/ and fix them ...
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
wmf-deployment
All All
: Normal normal (vote)
: ---
Assigned To: Bartosz Dziewoński
:
Depends on:
Blocks: 69277
  Show dependency treegraph
 
Reported: 2014-08-17 22:22 UTC by Bartosz Dziewoński
Modified: 2014-11-14 21:56 UTC (History)
4 users (show)

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


Attachments
mwgrep Arr_r, for contribsrange (2.45 KB, text/plain)
2014-08-17 22:30 UTC, Bartosz Dziewoński
Details
mwgrep ajax.js (2.14 KB, application/octet-stream)
2014-09-03 13:49 UTC, Bartosz Dziewoński
Details
mwgrep wikibits.js (8.65 KB, application/octet-stream)
2014-09-03 13:50 UTC, Bartosz Dziewoński
Details

Description Bartosz Dziewoński 2014-08-17 22:22:12 UTC
Track down popular gadgets that refer to files in skins/common/ and fix them not to.

Probably the best and simplest way is to upload offending files to Commons and update the URLs to point there.
Comment 1 Bartosz Dziewoński 2014-08-17 22:30:21 UTC
Created attachment 16223 [details]
mwgrep Arr_r, for contribsrange

First one for warmup: contribsrange.
Comment 3 Bartosz Dziewoński 2014-08-18 19:44:30 UTC
I don't like that because it decreases performance for users (by sidestepping RL request concatenation and this requiring more requests to load everything) and makes it impossible to express dependencies on such modules (because the actual loading is doubly-asynchronous). You can work on that if you want, I won't.
Comment 4 Helder 2014-08-18 19:52:04 UTC
Me neither, but it is either doing that or having to fix the same things hundreds of times over and over again (or worse: keeping forks broken forever). I assume that is why this was recommended back in 2011...
Comment 5 Bartosz Dziewoński 2014-08-18 21:28:55 UTC
(In reply to Bartosz Dziewoński from comment #1)
> First one for warmup: contribsrange.

Done, example edit: https://en.wikipedia.org/w/index.php?diff=621826869
Comment 6 Bartosz Dziewoński 2014-08-30 23:48:51 UTC
From b1dea1d957833d1965999c350a05d365ba56adba:

* ajax.js: mwgrep points to outdated HotCat versions, but they only have the
  file name in core comments.
* wikibits.js: mwgrep points to HotCat for the same reason, and to a bunch of
  scripts that generate popup windows with <script src="…/wikibits.js"> in them.
  At a very quick glance, these popups don't even rely on wikibits in any way…

(I'll have the queries redone without the 100 results limit and attach them.)
Comment 7 Bartosz Dziewoński 2014-09-03 13:49:30 UTC
Created attachment 16350 [details]
mwgrep ajax.js
Comment 8 Bartosz Dziewoński 2014-09-03 13:50:19 UTC
Created attachment 16351 [details]
mwgrep wikibits.js
Comment 9 Bartosz Dziewoński 2014-11-10 15:40:31 UTC
The above stuff seems mostly gone.

Let's look at magnify-clip.png and bullet.gif a bit more and then close this.
Comment 10 Bartosz Dziewoński 2014-11-10 20:24:33 UTC
magnify-clip.png is used almost exclusively by copies of the Navigation Popups gadget on 50 wikis or so. Conveniently, Popups does this well, using the same HTML structure as core, so this means that nothing will break if we just remove the image file.

I fixed the two other uses found in the MediaWiki namespace across all Wikimedia wikis:
https://www.wikidata.org/w/index.php?title=MediaWiki:Gadget-CommonsMedia.js&diff=172833809&oldid=139274853
https://no.wikipedia.org/w/index.php?title=MediaWiki:Gadget-youtube-player.js&diff=13508747&oldid=5472232
Comment 12 Bartosz Dziewoński 2014-11-14 21:56:12 UTC
Replaced bullet.gif on a couple dozen wikis today. There are still a few matches, but it seems like they are all false positives or dead code. (In particular, there's a few copies of a gadget that does literally `url(bullet.gif)`, which has never worked in the first place.)

I'm calling this done.

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


Navigation
Links