Last modified: 2012-04-16 09:15:44 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 T29481, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 27481 - Wrong removal of spaces in javascript
Wrong removal of spaces in javascript
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
ResourceLoader (Other open bugs)
unspecified
All All
: Normal major (vote)
: ---
Assigned To: Trevor Parscal
:
Depends on:
Blocks: 27339
  Show dependency treegraph
 
Reported: 2011-02-17 00:33 UTC by zedlik
Modified: 2012-04-16 09:15 UTC (History)
2 users (show)

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


Attachments

Description zedlik 2011-02-17 00:33:38 UTC
Hello,

Resource Loader seems to wrongly remove spaces in javascript code in some cases using debug=false. With debug=true the code remains untouched, so no problem appears.

Test case. The original code is located under [1]. The code compressed by Resource Loader is located under [2]. There is the following line in the first listing:
 InterwikiLinks[i].className += " FA"
after the compressing it appears to be:
 InterwikiLinks[i].className += "FA" (note the dropped space)

Another case. 
 InterwikiLinks[i].title = "Гэта абраны артыкул на іншай мове.";
after the compressing appears to be:
 InterwikiLinks[i].title = "Гэтаабраныартыкулнаіншаймове."; (all the spaces are dropped)

The original code is the valid javascript code, but looks like Resource Loader lost the quote-sign balance and started compressing string literals.

[1] http://be-x-old.wikipedia.org/wiki/MediaWiki:Common.js
[2] http://bits.wikimedia.org/be-x-old.wikipedia.org/load.php?debug=false&lang=be-tarask&modules=site&only=scripts&skin=vector&version=20110216T235746Z
Comment 1 Roan Kattouw 2011-02-17 22:42:00 UTC
It's fixed now. I made a whitespace edit to MediaWiki:Common.js to circumvent caching; this'll take 5 minutes to propagate.

If anyone else encounters a similar issue with site/user JS, making a whitespace edit to the page and waiting 5 minutes should fix it. If it's a core MediaWiki file, please comment on this bug.

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


Navigation
Links