Last modified: 2014-10-16 11:13:22 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 T73200, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71200 - Support redirects in JavaScriptContent
Support redirects in JavaScriptContent
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
ContentHandler (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: gadgets-2.0
  Show dependency treegraph
 
Reported: 2014-09-23 21:59 UTC by Kunal Mehta (Legoktm)
Modified: 2014-10-16 11:13 UTC (History)
5 users (show)

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


Attachments

Description Kunal Mehta (Legoktm) 2014-09-23 21:59:42 UTC
During the Gadgets 2.0 migration, we will be moving a lot of JS pages around. In order not to break existing mw.loader.load calls to MediaWiki:Gadget- pages, we should support redirects for JS pages.
Comment 1 Helder 2014-09-23 22:03:38 UTC
This looks like a dupe of bug 33973 to me.
Comment 2 Gerrit Notification Bot 2014-09-23 22:45:57 UTC
Change 162491 had a related patch set uploaded by Legoktm:
Support redirects in JavaScriptContent

https://gerrit.wikimedia.org/r/162491
Comment 3 Kunal Mehta (Legoktm) 2014-10-05 03:28:24 UTC
Current patch only creates "redirects" behind, but MW doesn't actually recognize these as redirects. This means people without "delete" can't revert moves (if they move their own common.js or something) since the pages aren't recognized as redirects.

Also if a wiki changes their domain, there is no easy way to find "redirects" that need updating without doing a full text search (usually difficult) or just going through logs (not perfect).

My proposal is that we make these real MW redirects.

We can't just turn any mw.loader.load("url...") into a redirect since people might have common.js's with just one script being loaded.

A magic syntax like /* #REDIRECT */mw.loader.load("url..."); is what I'm thinking of. "#REDIRECT" will be hardcoded and English only and must be in that exact format with spaces and whatnot.

Thoughts?
Comment 4 Jackmcbarn 2014-10-05 03:29:33 UTC
+1 to that idea.

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


Navigation
Links