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

2011-04-04 Thread MediaWiki Mail
User Nikerabbit posted a comment on MediaWiki.r85296. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85296#c15686 Comment: You sure didn't give a lot of time to fix this... ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Siebrand posted a comment on MediaWiki.r85296. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85296#c15687 Comment: Isn't it good practice to support this without warnings for at least one release (1.18), have it whine the next release (1.19), and remove it in the 3rd

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

2011-04-04 Thread MediaWiki Mail
User Tim Starling posted a comment on MediaWiki.r79108. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/79108#c15688 Comment: I get pre Running test Interlanguage link, with prefix links... A database error has occurred. Did you forget to run maintenance/update.php after

[Wikitech-l] Actions and Special Pages

2011-04-04 Thread Happy-melon
In the beginning, there was Article.php. And the Article class had methods like Article::view(), Article::delete(), Article::protect(), etc. And the fundamental codeflow of Mediawiki consisted of something along the lines of: $article = new Article( $title ); $action =

Re: [Wikitech-l] Actions and Special Pages

2011-04-04 Thread Tim Starling
On 04/04/2011 11:28 PM, Happy-melon wrote: For either case in the backend I would think we'd want to create an ActionPage base class and an EditActionPage from that, which looks internally rather like a SpecialPage construct, might even subclass it. We had this discussion in January. That's

Re: [Wikitech-l] Actions and Special Pages

2011-04-04 Thread Aryeh Gregor
On Mon, Apr 4, 2011 at 9:28 AM, Happy-melon happy-me...@live.com wrote: If people think it would be better as a special page we'd make http://foo.example.com/w/index.php?title=Baraction=edit a hard redirect to Special:Edit/Bar; that has the significant advantage of being able to be formed as

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

2011-04-04 Thread MediaWiki Mail
User Raymond changed the status of MediaWiki.r85327. Old Status: new New Status: fixme User Raymond also posted a comment on MediaWiki.r85327. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85327#c15689 Comment: Seen on Translatewiki and on my local wiki: Warning:

Re: [Wikitech-l] Focus on sister projects

2011-04-04 Thread Billinghurst
On 2 Apr 2011 at 16:08, Ryan Kaldari wrote: What do you guys think would be more useful: 1. Helping sister projects write up proposals for the Berlin Hackathon If it is going to get selected and have outcome for my favourite wiki, then yes ;-) 2. Creating The Complete Idiot's Guide to

Re: [Wikitech-l] Future: Love for the sister projects!

2011-04-04 Thread Billinghurst
On 3 Apr 2011 at 10:56, Brion Vibber wrote: (Breaking this out from 'Focus on sister projects' thread.) I just want to give an official shout out, as Lead Software Architect for the Wikimedia Foundation, that Wikipedia's sister projects are important and that they need more love. Not only

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

2011-04-04 Thread MediaWiki Mail
User Happy-melon changed the status of MediaWiki.r85302. Old Status: fixme New Status: new User Happy-melon also posted a comment on MediaWiki.r85302. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85302#c15691 Comment: Heaven knows tt:D/tt Should be fixed in r85332 though.

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

2011-04-04 Thread MediaWiki Mail
User ^demon changed the status of MediaWiki.r85211. Old Status: fixme New Status: new User ^demon also posted a comment on MediaWiki.r85211. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85211#c15692 Comment: Merged the first 3. The last two look merged to me, The regexes in

Re: [Wikitech-l] Focus on sister projects

2011-04-04 Thread Ryan Kaldari
Maybe my brain just made a leap here, but is there a plan for implementing a web-based Javascript editor for MediaWiki?? This would be a hugely awesome feature. I've used jsFiddle (http://jsfiddle.net/) which is really slick - syntax color-coding, tabbing that works, a Tidy button(!),

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

2011-04-04 Thread MediaWiki Mail
User ^demon posted a comment on MediaWiki.r85080. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85080#c15693 Comment: Grabbed the ones mentioned in this rev, see r85343 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Tbleher posted a comment on MediaWiki.r83080. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/83080#c15695 Comment: This change breaks the CommentBox extension, which has a special page to add comments to pages. The special page creates a FauxRequest which it submits to

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

2011-04-04 Thread MediaWiki Mail
User ^demon changed the status of MediaWiki.r85151. Old Status: fixme New Status: new User ^demon also posted a comment on MediaWiki.r85151. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85151#c15696 Comment: Yay for followups :D

[Wikitech-l] writing phpunit tests for extensions

2011-04-04 Thread Michael Dale
I had a bit of a documentation challenge approaching the problem of writing phpunit test for extensions, mostly because many of the extensions do this very differently and the manual did not have any recommendations. It appears many extension have custom bootstraping code ( somewhat hacky path

Re: [Wikitech-l] HipHop

2011-04-04 Thread Inez Korczynski
Hi Tim, I have no problem running foo.php and test.php that you sent in hphpi, but also it compiles and run without any problems with hphp. What command exactly do you use to compile and then execute? Inez On Sun, Apr 3, 2011 at 5:38 PM, Tim Starling tstarl...@wikimedia.orgwrote: On

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

2011-04-04 Thread MediaWiki Mail
User ^demon changed the status of MediaWiki.r85358. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85358#c0 ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wikimedia.org

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r84929. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/84929#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r84627. Old Status: new New Status: resolved Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/84627#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r85144. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/85144#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r84637. Old Status: new New Status: resolved Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/84637#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r85276. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/85276#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r85328. Old Status: new New Status: deferred Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/85328#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Catrope changed the status of MediaWiki.r85048. Old Status: new New Status: ok Full URL: https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/85048#c0 ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Nikerabbit posted a comment on MediaWiki.r85361. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85361#c15699 Comment: Did you add or remove BOM in trunk/extensions/TimedMediaHandler/TimedMediaHandler.hooks.php? ___

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

2011-04-04 Thread MediaWiki Mail
User Mdale posted a comment on MediaWiki.r85361. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85361#c15700 Comment: not intentionally... What is BOM? ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wikimedia.org

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

2011-04-04 Thread MediaWiki Mail
User Nikerabbit posted a comment on MediaWiki.r85361. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85361#c15701 Comment: [[wikipedia:Byte order mark]] - and PHP doesn't like it. ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Platonides changed the status of MediaWiki.r85302. Old Status: new New Status: fixme User Platonides also posted a comment on MediaWiki.r85302. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85302#c15702 Comment: parserTests.php creates a new OutputPage.

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

2011-04-04 Thread MediaWiki Mail
User Platonides changed the status of MediaWiki.r85317. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85317#c0 ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wikimedia.org

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

2011-04-04 Thread MediaWiki Mail
User ^demon posted a comment on MediaWiki.r85373. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85373#c15703 Comment: Did you check it first? If so, you should bump the version in the README. ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Reedy changed the status of MediaWiki.r85376. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85376#c0 ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wikimedia.org

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

2011-04-04 Thread MediaWiki Mail
User Reedy changed the status of MediaWiki.r39635. Old Status: ok New Status: fixme User Reedy also posted a comment on MediaWiki.r39635. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/39635#c15704 Comment: Line 251 now in trunk has an undefined $name usage

Re: [Wikitech-l] writing phpunit tests for extensions

2011-04-04 Thread Platonides
Michael Dale wrote: Eventually it would be ideal to be able to 'just test your extension' from the core bootstraper (ie dynamically generate our suite.xml and namespace the registration of extension tests) ... but for now at least not having to wait for all the core tests as you write you

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

2011-04-04 Thread MediaWiki Mail
User Brion VIBBER posted a comment on MediaWiki.r85177. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85177#c15707 Comment: Passing a string doesn't work either -- the timezone parameter needs to be a DateTimeZone object. ___

Re: [Wikitech-l] writing phpunit tests for extensions

2011-04-04 Thread Michael Dale
On 04/04/2011 02:20 PM, Platonides wrote: Michael Dale wrote: Eventually it would be ideal to be able to 'just test your extension' from the core bootstraper (ie dynamically generate our suite.xml and namespace the registration of extension tests) ... but for now at least not having to wait

Re: [Wikitech-l] Future: Love for the sister projects!

2011-04-04 Thread Platonides
I think the source of the problems is the decentralizated process that wikimedia javascript have followed. Medium-size wikis will have 2-3 tech people. Small wikis will be lucky to have 1 js-savy sysop. Even wikis with no local expert may have just forked the Monobook.js from its mother project

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

2011-04-04 Thread MediaWiki Mail
User Reedy changed the status of MediaWiki.r85385. Old Status: new New Status: fixme User Reedy also posted a comment on MediaWiki.r85385. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85385#c15709 Comment: empty? :P Also, should have more spaces on your if statement ;)

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

2011-04-04 Thread MediaWiki Mail
User Brion VIBBER posted a comment on MediaWiki.r85385. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85385#c15710 Comment: Oh poop, I still haven't retrained my muscle memory for our space style. ;) The empty() should be fine for arrays, but if it makes you feel better change

Re: [Wikitech-l] Future: Love for the sister projects!

2011-04-04 Thread Alex Brollo
Very interesting! :-) I'm a beginner in js stuff, but I realized how much js can be useful in a number of ways (presently I am a very simple gadgets maker t help users engaged into proofreading into wikisource). I'm a little worried about additional server load coming from AJAX, and I suggest

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

2011-04-04 Thread MediaWiki Mail
User Reedy changed the status of MediaWiki.r85385. Old Status: fixme New Status: new Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85385#c0 ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wikimedia.org

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

2011-04-04 Thread MediaWiki Mail
User Tim Starling posted a comment on MediaWiki.r79108. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/79108#c15711 Comment: I'm not sure. It's stopped happening now. ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Happy-melon posted a comment on MediaWiki.r85402. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85402#c15712 Comment: This is half of r79542. Is it worth merging the rest of that rev? ___ MediaWiki-CodeReview mailing list

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

2011-04-04 Thread MediaWiki Mail
User Dantman posted a comment on MediaWiki.r85403. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85403#c15715 Comment: You can also drop code$wgUser-getSkin( $wgTitle ); // set skin title/code it's not needed. ___

Re: [Wikitech-l] Future: Love for the sister projects!

2011-04-04 Thread Casey Brown
On Mon, Apr 4, 2011 at 6:22 PM, Platonides platoni...@gmail.com wrote: Some steps to take: * Finally create global gadgets. * Create a central wiki for javascript sharing [1] It seems like this always comes up for a number of different things -- global pages for interwiki transclusion and

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

2011-04-04 Thread MediaWiki Mail
User Happy-melon changed the status of MediaWiki.r85302. Old Status: fixme New Status: new User Happy-melon also posted a comment on MediaWiki.r85302. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85302#c15714 Comment: Fixed in r85403

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

2011-04-04 Thread MediaWiki Mail
User Reedy posted a comment on MediaWiki.r85402. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85402#c15713 Comment: Making it private and the whitespace? Probably not ___ MediaWiki-CodeReview mailing list

Re: [Wikitech-l] Some changes to $wgOut, $wgUser, Skin, and SpecialPage code patterns

2011-04-04 Thread Daniel Friesen
On 11-04-04 02:40 PM, Platonides wrote: I like it. Specially the Linker change. It really looks the way to have it. I'm considering making the Parser get it's linker via $po-getLinker(); (either ParserOutput or ParserOptions, I need another look) The linker would be an input parameter, so it

Re: [Wikitech-l] Some changes to $wgOut, $wgUser, Skin, and SpecialPage code patterns

2011-04-04 Thread Aaron Schulz
I second the idea of the static Linker class. It's far better than the subclass system. Skin modification of links should focus on CSS anyway, rather than trying to overload link generating code. Daniel Friesen-4 wrote: On 11-04-04 02:40 PM, Platonides wrote: I like it. Specially the Linker

[Wikitech-l] Linker, more on Skins, OutputPage, and context code patterns changes

2011-04-04 Thread Daniel Friesen
Continuing with my changes to $wgOut, $wgUser, Skin, and SpecialPage patterns. The Linker is now static, $sk-link will still work, however you should not be requiring a Skin anymore to use linker methods. Please use Linker::link directly. The only exception is the method to create an

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

2011-04-04 Thread MediaWiki Mail
User Tim Starling posted a comment on MediaWiki.r84782. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/84782#c15716 Comment: pre total processing time: 188 days 4520h 57m 51.00s gained time: 2 days 62h 32m 52.859375s /pre Too many hours!

Re: [Wikitech-l] Some changes to $wgOut, $wgUser, Skin, and SpecialPage code patterns

2011-04-04 Thread MZMcBride
Brion Vibber wrote: I'd be kind of inclined to separate out the Skin and Linker worlds a bit further; really, the HTML formatting of links and such doesn't depend on the user (except in some edge cases like stub links which could in principle be done differently). The surrounding *skin*

Re: [Wikitech-l] Future: Love for the sister projects!

2011-04-04 Thread Casey Brown
On Mon, Apr 4, 2011 at 9:05 PM, K. Peachey p858sn...@gmail.com wrote: I don't see why we need to spilt it off to another list No reason not to do it here on wikitech-l. Did you realize that that list already exists? There's also no reason not to use both of them, with the ambassadors list

Re: [Wikitech-l] Actions and Special Pages

2011-04-04 Thread MZMcBride
Aryeh Gregor wrote: On Mon, Apr 4, 2011 at 9:28 AM, Happy-melon happy-me...@live.com wrote: If people think it would be better as a special page we'd make http://foo.example.com/w/index.php?title=Baraction=edit a hard redirect to Special:Edit/Bar; that has the significant advantage of being

Re: [Wikitech-l] Actions and Special Pages

2011-04-04 Thread Daniel Friesen
On 11-04-04 08:12 PM, MZMcBride wrote: Aryeh Gregor wrote: On Mon, Apr 4, 2011 at 9:28 AM, Happy-melonhappy-me...@live.com wrote: If people think it would be better as a special page we'd make http://foo.example.com/w/index.php?title=Baraction=edit a hard redirect to Special:Edit/Bar; that

Re: [Wikitech-l] HipHop

2011-04-04 Thread Tim Starling
On 03/30/2011 09:22 AM, Magnus Manske wrote: Plus, free C++ MediaWiki parser ;-) Seriously, there should be a way to turn the entire package into a (huge) library; maybe transpile it and then replace the C++ code for index.php with a manually written library interface? HipHop has a library

Re: [Wikitech-l] Actions and Special Pages

2011-04-04 Thread MZMcBride
Daniel Friesen wrote: What IS so evil about actions? Besides the implementation details which are just that... something we can make better. In fact something we could potentially make work better than SpecialPages would in this context. Personally, I like tacking on ?action=edit and

[Wikitech-l] GSOC and networking

2011-04-04 Thread Michael Costello
Hello, I was encouraged (hello, Sumana) to post a Google Summer of Code idea and solicit comments. So here goes. Perhaps entirely outside of the scope of GSOC, I think it would be a fun project to make wikipedia.org and wikimedia.org accessible over IPv6. Also, while creating an account to

Re: [Wikitech-l] GSOC and networking

2011-04-04 Thread K. Peachey
On Tue, Apr 5, 2011 at 1:07 PM, Michael Costello mc3...@columbia.edu wrote: Perhaps entirely outside of the scope of GSOC, I think it would be a fun project to make wikipedia.org and wikimedia.org accessible over IPv6. I noticed that my login was insecure, and that there was no SSL option.  

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

2011-04-04 Thread MediaWiki Mail
User Kaldari posted a comment on MediaWiki.r85417. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85417#c15717 Comment: This is a less clunky fix for bug 27424. The other rev (r85415) was overkill. ___ MediaWiki-CodeReview mailing

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

2011-04-04 Thread MediaWiki Mail
User MaxSem changed the status of MediaWiki.r85387. Old Status: new New Status: ok Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85387#c0 ___ MediaWiki-CodeReview mailing list mediawiki-coderev...@lists.wikimedia.org

[Wikitech-l] HipHop: need developers

2011-04-04 Thread Tim Starling
I need to get back to work on reviews and deployments and other such strategic things. I hope the work I've done on HipHop support is enough to get the project started. Initial benchmarks are showing a 5x speedup for article parsing, which is better than we had hoped for. So there are big