Re: [Wikitech-l] thumbnail generation for Extension:OggHandler

2011-07-11 Thread Tim Starling
On 08/07/11 21:18, Dmitriy Sintsov wrote: > Hi! > > What's the proper way of thumbnail generation for Ogg media handler, so > it will work like at commons? This is documented in the OggHandler README file: In short

[MediaWiki-CodeReview] [MediaWiki r91815]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91815. Old Status: new New Status: fixme User "Brion VIBBER" also posted a comment on MediaWiki.r91815. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91815#c19471 Commit summary: if we scale the $(body), interface is becoming

[MediaWiki-CodeReview] [MediaWiki r89853]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Reedy" posted a comment on MediaWiki.r89853. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89853#c19470 Commit summary: Added jquery.qunit.completenessTest.js (A jQuery/QUnit test coverage utility) * Added to /resources * Conditionally loaded (condition being that the url p

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

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91833. Old Status: new New Status: resolved Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91833#c0 Commit summary: Escape sidebar ID's. Bug 18688 ___ MediaWiki-CodeReview mailing li

[MediaWiki-CodeReview] [MediaWiki r91845]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Reedy" posted a comment on MediaWiki.r91845. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91845#c19469 Commit summary: Update jQuery from 1.6.1 to 1.6.2 (maintenance/bugfix release) * Release notes: http://blog.jquery.com/2011/06/30/jquery-162-released/ * Source: http://co

[MediaWiki-CodeReview] [MediaWiki r79856]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Reedy" changed the status of MediaWiki.r79856. Old Status: fixme New Status: resolved User "Reedy" also posted a comment on MediaWiki.r79856. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/79856#c19468 Commit summary: Bug 26563: Add bytes changed per revision for stub and

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

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91834. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91834#c0 Commit summary: Removing @broken comment. Not anymore, they don't as of r91388. - (Follows-up 91387) ___

[MediaWiki-CodeReview] [MediaWiki r91845]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91845. Old Status: new New Status: ok User "Brion VIBBER" also posted a comment on MediaWiki.r91845. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91845#c19467 Commit summary: Update jQuery from 1.6.1 to 1.6.2 (maintenance/bu

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

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91860. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91860#c0 Commit summary: Per SPQRobin, fix for r91817: escape the link so that in renders properly on NS_FILE and NS_CATEGORY __

[MediaWiki-CodeReview] [MediaWiki r91871]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" posted a comment on MediaWiki.r91871. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91871#c19466 Commit summary: Add action-* class to body. Bug 4438. Based on patch by Subfader Comment: Doesn't validate input; while the output will be HTML-escaped, it's a b

[MediaWiki-CodeReview] [MediaWiki r91927]: New comment added

2011-07-11 Thread MediaWiki Mail
User "DieBuche" posted a comment on MediaWiki.r91927. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91927#c19465 Commit summary: Revert r91870, r91873 -- seems to be trying to remove the 'view' action on diff views; diffs are a subset of regular page views, like old page views,

[MediaWiki-CodeReview] [MediaWiki r91869]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91869. Old Status: fixme New Status: resolved User "Brion VIBBER" also posted a comment on MediaWiki.r91869. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91869#c19464 Commit summary: use tab name, not tab index for anchors o

[MediaWiki-CodeReview] [MediaWiki r91869]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91869. Old Status: new New Status: fixme User "Brion VIBBER" also posted a comment on MediaWiki.r91869. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91869#c19463 Commit summary: use tab name, not tab index for anchors on Spe

[MediaWiki-CodeReview] [MediaWiki r91776]: New comment added

2011-07-11 Thread MediaWiki Mail
User "NeilK" posted a comment on MediaWiki.r91776. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91776#c19462 Commit summary: start special page for viewing archived content. Comment: Also, in general, try to move configuration into its own block or method. It's much better if

[MediaWiki-CodeReview] [MediaWiki r91776]: New comment added

2011-07-11 Thread MediaWiki Mail
User "NeilK" posted a comment on MediaWiki.r91776. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91776#c19461 Commit summary: start special page for viewing archived content. Comment: FYI "Archive" is an overloaded term in MediaWiki, since old deleted pages are also called 'ar

[MediaWiki-CodeReview] [MediaWiki r91873]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91873. Old Status: fixme New Status: reverted User "Brion VIBBER" also posted a comment on MediaWiki.r91873. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91873#c19460 Commit summary: r91870 : Got tricked into believing it wa

[MediaWiki-CodeReview] [MediaWiki r91870]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91870. Old Status: fixme New Status: reverted User "Brion VIBBER" also posted a comment on MediaWiki.r91870. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91870#c19459 Commit summary: wgAction incorrectly asumes 'view' instea

[MediaWiki-CodeReview] [MediaWiki r91873]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91873. Old Status: new New Status: fixme User "Brion VIBBER" also posted a comment on MediaWiki.r91873. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91873#c19458 Commit summary: r91870 : Got tricked into believing it was act

[MediaWiki-CodeReview] [MediaWiki r91870]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91870. Old Status: new New Status: fixme User "Brion VIBBER" also posted a comment on MediaWiki.r91870. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91870#c19457 Commit summary: wgAction incorrectly asumes 'view' instead of

[MediaWiki-CodeReview] [MediaWiki r91885]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" posted a comment on MediaWiki.r91885. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91885#c19456 Commit summary: (follow-up r90256) Unit tests. Comment: Hmm... this seems to sound like it'll *drop* metadata from a paged tiff file proxied from another site v

[MediaWiki-CodeReview] [MediaWiki r79856]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" posted a comment on MediaWiki.r79856. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/79856#c19455 Commit summary: Bug 26563: Add bytes changed per revision for stub and full article dumps Comment: Added to shell bugs: bug 29819. _

[MediaWiki-CodeReview] [MediaWiki r91842]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r91842. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91842#c19454 Commit summary: * Changed dynamic calls to Linker methods into static ones * Also preferably pass a Language object instead of a Skin one or boolean indicating whethe

[MediaWiki-CodeReview] [MediaWiki r79856]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r79856. Old Status: ok New Status: fixme User "Brion VIBBER" also posted a comment on MediaWiki.r79856. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/79856#c19453 Commit summary: Bug 26563: Add bytes changed per revision for s

[MediaWiki-CodeReview] [MediaWiki r50526]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Tfinc" posted a comment on MediaWiki.r50526. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/50526#c19451 Commit summary: fixing spacing Comment: Fixed in r51000 ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wi

[MediaWiki-CodeReview] [MediaWiki r83280]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r83280. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/83280#c19452 Commit summary: Follow-up r83183, r83202: * Update SpecialCheckUser.php to new location of IP functions * Spin out the 'hide-other-field-if-select-box-not-on-other

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r83280. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/83280#c0 Commit summary: Follow-up r83183, r83202: * Update SpecialCheckUser.php to new location of IP functions * Spin out the 'hide-other-fi

[MediaWiki-CodeReview] [MediaWiki r50526]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r50526. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/50526#c19450 Commit summary: fixing spacing Comment: This rev has been marked "todo" for the past 2 years. Quick check came up with almost [http://www.mediawiki.org/wiki/Special

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r83778. Old Status: new New Status: resolved Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/83778#c0 Commit summary: (bug 14706) Added support for the Imagick PHP extension. Based on patch by Leslie Hoare. Scaler type is "imext

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Patrick Reilly
On Mon, Jul 11, 2011 at 1:36 PM, Brion Vibber wrote: > On Mon, Jul 11, 2011 at 12:36 PM, Tomasz Finc wrote: > >> Firefox is tough as the current version has the exact same UA on >> mobile phones AND tablets. And since we don't redirect tablets we >> haven't switched it over yet. >> >> Anyone know

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r83791. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/83791#c0 Commit summary: (bug 2581, bug 6834) Added links to thumbnail in several resolutions to the file description page. The sizes are set

[MediaWiki-CodeReview] [MediaWiki r90793]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r90793. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90793#c19449 Commit summary: *sigh*...added bit left off of r90789 Comment: Tagged 1.18 because r90789 is tagged 1.18. ___ MediaWiki-

[MediaWiki-CodeReview] [MediaWiki r90866]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r90866. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90866#c19448 Commit summary: Tweaked r90766 messages. Still awkward. Also, this still needs JS to disable the check if "all" is selected, to be less confusing. Comment: Tagged 1

[MediaWiki-CodeReview] [MediaWiki r90531]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r90531. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90531#c19447 Commit summary: Follow up with the problem noted in r90530. You can't pass the result of a function to reset(), since it expects a reference. Those random errors sho

[MediaWiki-CodeReview] [MediaWiki r89817]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r89817. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89817#c19446 Commit summary: Fix for logic error in r89529 causing a notice. Comment: Tagging 1.18 because r89529 is tagged 1.18. ___

[MediaWiki-CodeReview] [MediaWiki r80612]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r80612. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/80612#c19445 Commit summary: Partial revert r78450: doQuery() and query() are not the same. You can't just swap one for the other without checking what might be using the result

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91732. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91732#c0 Commit summary: Create a new RequestContext to use its OutputPage and Skin members instead of messing with global ones

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91861. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91861#c0 Commit summary: Use getPageTitle() to set the page title instead of doing it manually __

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91820. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91820#c0 Commit summary: Disable feed mode when including the page. I know that there is already r87705 to address this, but this one is need

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91859. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91859#c0 Commit summary: Removed print and render actions from the check: * print was removed in 86373 * render does not reach this point sinc

Re: [Wikitech-l] ResourceLoader JavaScript validation on trunk (bug 28626)

2011-07-11 Thread Brion Vibber
On Wed, Jul 6, 2011 at 3:04 PM, Brion Vibber wrote: > Only the parser is being used right now, in two places: > - on the JavaScriptMinifier test cases to confirm that results are valid JS > (should be extended to a fuzz tester, probably) > - on each individual file loaded via ResourceLoaderFileMo

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91864. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91864#c0 Commit summary: Call Linker methods statically and removed the $mSkin member ___ MediaWi

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91862. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91862#c0 Commit summary: Override UsersPager::getTitle() in ActiveUsersPager ___ MediaWiki-CodeRe

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91744. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91744#c0 Commit summary: Fix for r78786: make Special:Wantedpages includable again ___ MediaWiki-

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91804. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91804#c0 Commit summary: Added Request::setLang() and RequestContext::setSkin(); the latter will clone the object and set the context on it.

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91842. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91842#c0 Commit summary: * Changed dynamic calls to Linker methods into static ones * Also preferably pass a Language object instead of a Skin

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Brion Vibber
On Mon, Jul 11, 2011 at 12:36 PM, Tomasz Finc wrote: > Firefox is tough as the current version has the exact same UA on > mobile phones AND tablets. And since we don't redirect tablets we > haven't switched it over yet. > > Anyone know why they did that? > Mozilla generally recommends using CSS

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Brion Vibber
On Mon, Jul 11, 2011 at 12:56 PM, Jay Ashworth wrote: > - Original Message - > > From: "Brion Vibber" > > > * default redirection to the appropriate view based on device > > Please remember that some people with high-function browsers *want* the > low-function results... and some people

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Stephen Bain
On Tue, Jul 12, 2011 at 5:59 AM, Stephen Bain wrote: > > TestSwarm 'doesn't need my help' when I load it with Fennec 5 or Opera > Mobile 11.1. Though Opera does allow you to set the UA to mobile or desktop, and it will let me run tests when set to the desktop version. (Wikipedia doesn't give me

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Stephen Bain
On Sat, Jul 9, 2011 at 7:30 PM, Ashar Voultoiz wrote: > > * Android : has an emulator. On my computer it is painfully slow and not > usable for anything. I can connect to TestSwarm with the stock Android browser on 2.3 (though it's identifying it as 2.2) but it won't give me any tests to run. Te

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Jay Ashworth
- Original Message - > From: "Brion Vibber" > * default redirection to the appropriate view based on device Please remember that some people with high-function browsers *want* the low-function results... and some people with *what your code thinks are low-function browsers* want the "st

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Tomasz Finc
Firefox is tough as the current version has the exact same UA on mobile phones AND tablets. And since we don't redirect tablets we haven't switched it over yet. Anyone know why they did that? --tomasz On Mon, Jul 11, 2011 at 12:06 PM, Steven Walling wrote: > On Mon, Jul 11, 2011 at 11:40 AM,

Re: [Wikitech-l] Postmortem for the MediaWiki 1.17 release

2011-07-11 Thread Ashar Voultoiz
On 07/07/11 03:42, Rob Lanphier wrote: > we're probably too reliant on Tim to not only drive but execute > many steps. > Additionally, we will probably experiment with other team members > (e.g. Chad) performing at least alpha or beta releases. This is actually a great way to train new people. L

Re: [Wikitech-l] Postmortem for the MediaWiki 1.17 release

2011-07-11 Thread Ashar Voultoiz
On 07/07/11 03:42, Rob Lanphier wrote: > http://live.gnome.org/ThreePointOne > having clear deadlines for proposing new features, deadlines for > features being done or pulled, and other date-risk mitigation > strategies. Having a roadmap like Gnome is the way I am advocating. Another way I co

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Steven Walling
On Mon, Jul 11, 2011 at 11:40 AM, Brion Vibber wrote: > default redirection to the appropriate view based on device I can tell you right now that Firefox for Android has never redirected to the mobile site for me. If you want more device/version details let me know... Steven __

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

2011-07-11 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r91896. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91896#c0 Commit summary: Follow-up r91895: Call parent destructor as well, just to be safe. ___ MediaW

Re: [Wikitech-l] testing mobile browsers?

2011-07-11 Thread Brion Vibber
On Sat, Jul 9, 2011 at 2:30 AM, Ashar Voultoiz wrote: > * Safari, Opera & Mozilla for mobile : they are probably mostly the same > as the desktop version. I have not found emulators for them. > * Android : has an emulator. On my computer it is painfully slow and not > usable for anything. > * Bla

[MediaWiki-CodeReview] [MediaWiki r91853]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Krinkle" posted a comment on MediaWiki.r91853. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91853#c19444 Commit summary: Use more strict if-statement to decide whether to reload current page or rebuild url. - Extracting the current url without the hash-tag - Building the

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r88054. Old Status: new New Status: resolved Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/88054#c0 Commit summary: (bug 23002) Imagelinks table not updated after imagemove. The actual bug was inconsistent behaviour between im

[MediaWiki-CodeReview] [MediaWiki r91853]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Krinkle" posted a comment on MediaWiki.r91853. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91853#c19443 Commit summary: Use more strict if-statement to decide whether to reload current page or rebuild url. - Extracting the current url without the hash-tag - Building the

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91785. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91785#c0 Commit summary: Follow-up r88054: register the file if a hook changed the target file. _

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

2011-07-11 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r91857. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91857#c0 Commit summary: follow-up to r91853 - revising comments to explain the new logic, also guarding against javascript error in case of data.

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r88071. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/88071#c0 Commit summary: Follow-up r88054: Update messages.inc ___ MediaWiki-CodeReview mailing l

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

2011-07-11 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r91891. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91891#c0 Commit summary: * (bug 29804) Fix byte-length-limited fields to check the length of to-be-added char instead of hardcoding a possible len

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

2011-07-11 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r91894. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91894#c0 Commit summary: Followup r91844: fix jquery.byteLimit test cases on IE6, IE7 str[i] doesn't work on these older browsers; use str.charAt(

[MediaWiki-CodeReview] [MediaWiki r91844]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" changed the status of MediaWiki.r91844. Old Status: new New Status: resolved User "Brion VIBBER" also posted a comment on MediaWiki.r91844. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91844#c19442 Commit summary: Add test suite for jquery.byteLimit: - Test

[MediaWiki-CodeReview] [MediaWiki r91875]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Nikerabbit" posted a comment on MediaWiki.r91875. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91875#c19441 Commit summary: * (bug 16699) {{#language:}} accepts second parameter to specify the language in which the language name is wanted. Coverage depends on the cldr ex

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

2011-07-11 Thread MediaWiki Mail
User "Siebrand" changed the status of MediaWiki.r91830. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91830#c0 Commit summary: Tweak extension credits, add description message Add extension to Translatewiki, ping r87714 to inform the author __

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r88646. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/88646#c0 Commit summary: * Simplify message existence checks by using wfMessage() instead of wfMsg() with wfEmptyMsg() * Fixed one check in S

[MediaWiki-CodeReview] [MediaWiki r91875]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r91875. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91875#c19440 Commit summary: * (bug 16699) {{#language:}} accepts second parameter to specify the language in which the language name is wanted. Coverage depends on the cldr exte

[MediaWiki-CodeReview] [MediaWiki r91844]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Brion VIBBER" posted a comment on MediaWiki.r91844. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91844#c19439 Commit summary: Add test suite for jquery.byteLimit: - Tests to verify that the byteLength does not exceed the byteLimit when inserting more characters - Tests to

[MediaWiki-CodeReview] [MediaWiki r91875]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Nikerabbit" posted a comment on MediaWiki.r91875. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91875#c19438 Commit summary: * (bug 16699) {{#language:}} accepts second parameter to specify the language in which the language name is wanted. Coverage depends on the cldr ex

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r88705. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/88705#c0 Commit summary: Rewriting mw.loader test suite. The previous one didn't work properly in IE6-IE8. _

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r89001. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89001#c0 Commit summary: Fix issue on the live site causing RL requests for ext.uploadWizard to consistently take more than 5 seconds: the ca

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91800. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91800#c0 Commit summary: Call doMarkPatrolled() on the RecentChange object instead of calling RecentChange::markPatrolled() that will create

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91677. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91677#c0 Commit summary: * Fixed comment * Wrap arround RequestContext::msg() instead of doing that all that stuff once more ___

[MediaWiki-CodeReview] [MediaWiki r91677]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r91677. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91677#c19437 Commit summary: * Fixed comment * Wrap arround RequestContext::msg() instead of doing that all that stuff once more Comment: Ah, my SVN copy was outdated ;) __

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91551. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91551#c0 Commit summary: Override Pager::getTitle() in core subclasses of ReverseChronologicalPager that don't have it _

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

2011-07-11 Thread MediaWiki Mail
User "Reedy" changed the status of MediaWiki.r91889. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91889#c0 Commit summary: Fix MIME types of images added in r91525 ___ MediaWiki-CodeReview mailing list

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91461. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91461#c0 Commit summary: Override IndexPager::getTitle() in AlphabeticPager instance of core to not rely on $wgTitle ___

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r89005. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89005#c0 Commit summary: Refactor MessageBlobStore::updateModule() to remove the multi-language update behavior. Instead of updating all lang

[MediaWiki-CodeReview] [MediaWiki r91875]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Siebrand" posted a comment on MediaWiki.r91875. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91875#c19436 Commit summary: * (bug 16699) {{#language:}} accepts second parameter to specify the language in which the language name is wanted. Coverage depends on the cldr exte

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91074. Old Status: new New Status: resolved Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91074#c0 Commit summary: * Modified some methods of SpecialWatchlist to be non-static so that they can use $this->getTitle() instead of

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

2011-07-11 Thread MediaWiki Mail
User "Reedy" changed the status of MediaWiki.r91887. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91887#c0 Commit summary: Jarry1250 = Harry Burt, per Reedy on r91886 ___ MediaWiki-CodeReview mailing li

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

2011-07-11 Thread MediaWiki Mail
User "Catrope" changed the status of MediaWiki.r91525. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/91525#c0 Commit summary: (part of bug 24692) Make RTL versions of the arrows at the top of UploadWizard _

[MediaWiki-CodeReview] [MediaWiki r91886]: New comment added

2011-07-11 Thread MediaWiki Mail
User "SPQRobin" posted a comment on MediaWiki.r91886. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91886#c19435 Commit summary: (bug 29797) Error: "Tried to load block with invalid type" when subpages are disabled for user pages. Patch by Jarry1250 Comment: Ah, didn't know th

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

2011-07-11 Thread MediaWiki Mail
User "Catrope" changed the status of MediaWiki.r91030. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/91030#c0 Commit summary: InstantCommons files also have -1 as page id; handle this case when dealing with filename unique

[MediaWiki-CodeReview] [MediaWiki r91854]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Raymond" posted a comment on MediaWiki.r91854. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91854#c19434 Commit summary: SocialProfile: add a special page, Special:GenerateTopUsersReport, to generate the report of the users who earned the most points during the past week

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

2011-07-11 Thread MediaWiki Mail
User "Catrope" changed the status of MediaWiki.r90687. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/90687#c0 Commit summary: move comment ___ MediaWiki-CodeReview mailing list m

[MediaWiki-CodeReview] [MediaWiki r91886]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Reedy" posted a comment on MediaWiki.r91886. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91886#c19433 Commit summary: (bug 29797) Error: "Tried to load block with invalid type" when subpages are disabled for user pages. Patch by Jarry1250 Comment:

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

2011-07-11 Thread MediaWiki Mail
User "Catrope" changed the status of MediaWiki.r91807. Old Status: new New Status: deferred Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/91807#c0 Commit summary: Localisation updates for core and extension messages from translatewiki.net (2011-07-09 19

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r88727. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/88727#c0 Commit summary: Simplify message existence checks by using wfMessage() instead of wfEmptyMsg() _

[MediaWiki-CodeReview] [MediaWiki r91784]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r91784. Old Status: new New Status: ok User "Aaron Schulz" also posted a comment on MediaWiki.r91784. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91784#c19432 Commit summary: Pass the User object to RecentChange::doMarkPatro

[MediaWiki-CodeReview] [MediaWiki r90256]: New comment added, and revision status changed

2011-07-11 Thread MediaWiki Mail
User "Bawolff" changed the status of MediaWiki.r90256. Old Status: fixme New Status: new User "Bawolff" also posted a comment on MediaWiki.r90256. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90256#c19431 Commit summary: (follow-up r86567) per CR rename the class JpegOrTiffHan

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

2011-07-11 Thread MediaWiki Mail
User "Aaron Schulz" changed the status of MediaWiki.r88815. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/88815#c0 Commit summary: * Check existence in correct language * Simplify a bit existence check _

Re: [Wikitech-l] thumbnail generation for Extension:OggHandler

2011-07-11 Thread Michael Dale
Thanks for this thread, Please do commit fixes for 1.17, If not obvious already I have really only been targeting trunk. While the extension has been around since before 1.16, it would be very complicated to restore the custom resource loader it was using before there was a resource loader in core.

[MediaWiki-CodeReview] [MediaWiki r91854]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Jack Phoenix" posted a comment on MediaWiki.r91854. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91854#c19430 Commit summary: SocialProfile: add a special page, Special:GenerateTopUsersReport, to generate the report of the users who earned the most points during the past

[MediaWiki-CodeReview] [MediaWiki r89165]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Bawolff" posted a comment on MediaWiki.r89165. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89165#c19429 Commit summary: Added entry for alias file Comment: Do you have [[Manual:$wgDevelopmentWarnings]] on? ___ MediaWiki-CodeR

[MediaWiki-CodeReview] [MediaWiki r89165]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Yuvipanda" posted a comment on MediaWiki.r89165. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89165#c19428 Commit summary: Added entry for alias file Comment: http://xkcd.com/583/ ___ MediaWiki-CodeReview mailing list mediawik

[MediaWiki-CodeReview] [MediaWiki r91882]: New comment added

2011-07-11 Thread MediaWiki Mail
User "Yuvipanda" posted a comment on MediaWiki.r91882. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91882#c19427 Commit summary: Added Global statistics support Comment: Dammit, whining doesn't seem to get me anywhere. Will fix. ___

[MediaWiki-CodeReview] [MediaWiki r91627]: New comment added

2011-07-11 Thread MediaWiki Mail
User "^demon" posted a comment on MediaWiki.r91627. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91627#c19426 Commit summary: Initial import of GSoC project. Comment: This is the review of your proposed schema: * I see you used my old mwreleases schema for tracking MediaWiki c

  1   2   >