[MediaWiki-CodeReview] [MediaWiki r87497]: Revision status changed

2011-06-07 Thread MediaWiki Mail
User Brion VIBBER changed the status of MediaWiki.r87497.

Old Status: new
New Status: resolved

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/87497#c0
Commit summary:

Per bug 28738 comment 4, pack ResourceLoader URLs by encoding 
foo.bar|foo.baz|bar.baz|bar.quux as foo.bar,baz|bar.baz,quux

* Expand these URLs in ResourceLoaderContext
* Build and emit these URLs in OutputPage::makeResourceLoaderLink() and in 
mw.loader
* Throw an exception in ResourceLoader::register() for module names that 
contain pipe characters or commas. Commas need to be forbidden for this packing 
feature to work. Pipes were already forbidden but weren't checked for

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r87497]: Revision status changed

2011-05-14 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r87497.

Old Status: fixme
New Status: new

Full URL: 
https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/87497#c0
Commit summary:

Per bug 28738 comment 4, pack ResourceLoader URLs by encoding 
foo.bar|foo.baz|bar.baz|bar.quux as foo.bar,baz|bar.baz,quux

* Expand these URLs in ResourceLoaderContext
* Build and emit these URLs in OutputPage::makeResourceLoaderLink() and in 
mw.loader
* Throw an exception in ResourceLoader::register() for module names that 
contain pipe characters or commas. Commas need to be forbidden for this packing 
feature to work. Pipes were already forbidden but weren't checked for

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview