[Wikitech-l] MediaWiki Workshop: Preparing extensions for MediaWiki 1.19 - 1/13/2012 on #wikimedia-dev

2012-01-06 Thread Gregory Varnum
Greetings!

Utilizing the model set by Wikimedia staff IRC office hours, MediaWiki.org's 
WikiProject Extensions is presenting our first MediaWiki Workshops for 
developers (volunteer and staff).  Preparing extensions for MediaWiki 1.19 will 
be held on 13 January, 2012 at 19:00 UTC in IRC (#wikimedia-dev).

This IRC workshop will be an opportunity to find out about changes in MediaWiki 
1.19 that may require revisions to extensions or skins. Also an opportunity to 
ask MediaWiki developers questions regarding extension development.

Everyone is invited to attend.  Developers interested in serving as 
extensions or MediaWiki 1.19 experts are encouraged to signup as 
participants at: 
http://www.mediawiki.org/wiki/Project:WikiProject_Extensions/MediaWiki_Workshops#MediaWiki_1.19_Experts

If there's continued interest / demand, MediaWiki Workshops will typically last 
one hour, and happen no more than twice a month to present trainings, hold 
discussions and collaborate on community or WikiProject Extensions projects. 
Facilitators host the session to introduce any presenters, determine the order 
of questions, and generally helps to keep things going. Time of day will vary 
in order to offer people in different parts of the world the opportunity to 
participate. Future topics will likely include MediaWiki.org documentation, 
ResourceLouder orientation and workshops similar to our inaugural chat to 
prepare for each MediaWiki release.

More information: 
http://www.mediawiki.org/wiki/Project:WikiProject_Extensions/MediaWiki_Workshops

MediaWiki.org's WikiProject SysAdmins will also likely host similar MediaWiki 
Workshops to help third-party wiki system administrators.  Stay tuned for more 
information, and chime in at: 
http://www.mediawiki.org/wiki/Project:WikiProject_SysAdmins/Ideas

Look forward to seeing folks next Friday!  Please feel free to forward this 
along to any interested folks.

-greg aka varnent

PS. Sorry if this is a duplicate - was having listserv problems earlier.


---
Gregory Varnum
Lead Administrator, WikiQueer
Lead, Aequalitas Project
@GregVarnum
fb.com/GregVarnum
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108200 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108200

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108200:

Follow-up r107816: Update forgotten FuzzyBot reference.

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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108178 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108178

Old status:  deferred
 New status: ok

Commit summary for MediaWiki.r108178:

Removed DTUtils::loadMessages() method - no longer needed, since versions of 
MediaWiki less than 1.16 are no longer being supported

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


[Wikitech-l] Pages-logging dump - more information?

2012-01-06 Thread Katja Mueller
Dear list,

do you provide more information about the pages-logging dump somewhere?
While parsing it we came across some questions that we are trying to
clarify:

* Why are there only ~40 million logs (at over 450 million revisions)?
Which logs does the pages-logging dump / the public logging table
(not) contain? (We double-checked the number of logs on the database,
using our Toolserver-Account (Logging table).)

* Can you give us more information about the TextElement that is defined
for logitem in the XML Schema?
Definition: element name=text type=mw:TextType/

In pages-logging it sometimes occurs as: 
text deleted=deleted /

How is this being used?


Kind regards,
Katja Mueller



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108195 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108195

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108195:

Correction in the test comment, followup r107824

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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108198 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108198

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108198:

Make spelling of MasterCard consistent.
Spotted by EugeneZelenko 
https://translatewiki.net/wiki/Thread:Support/MediaWiki:Donate_interface-card-name-mc/en

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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108199 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108199

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108199:

Consistency tweak: no space between word and ... in English language
Spotted by OsamaK 
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Mobile-frontend-wml-continue/en

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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit posted a comment on MediaWiki.r108201.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108201#c29039

Commit summary for MediaWiki.r108201:

replace hard tabs with spaces

Nikerabbit's comment:

The ruling practice is to use tabs for indentation. 

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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108184 to fixme and commented 
it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108184#c29040

Old Status: new
 New Status: fixme

Commit summary for MediaWiki.r108184:

ResourceLoader: Add an experimental option to move the main module loading 
queue (the bottom queue) from the bottom of the body up into the head , 
while still being loaded asynchronously. This makes them load earlier, which 
should make the page load faster. This is the product of a long discussion on 
bug 27488

* Added a blocking state to mw.loader . When loading scripts while the 
document is not ready, the loader will use document.write() if blocking is 
true, and append to the body or the head if blocking is false. If the 
document is ready, the loader will always append to the body
* Enable blocking mode while loading the top queue, and disable it after. This 
ensures that modules in the top queue are still loaded in a blocking way as 
they were before
* If $wgResourceLoaderExperimentalAsyncLoading is true, the bottom queue is 
also loaded in the head, but with blocking mode disabled. Otherwise, it's 
loaded at the bottom of the body as before
* scripts-only and messages-only requests need special treatment:
** in the top queue, they can continue to use script src=... tags because 
they are blocking
** if the bottom queue is at the bottom of the body (experimental async 
loading disabled), they can continue to use script src=... tags as before
** if the bottom queue is in the head (experimental async loading enabled), 
they cannot use script src=... tags, because those would block. Instead, 
call mw.loader.load() on the load.php URL

Nikerabbit's comment:

Looks like this is breaking some scripts even when the new global is not set to 
true.

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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108212 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108212

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108212:

svn:eol-style native

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


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

2012-01-06 Thread MediaWiki Mail
Santhosh.thottingal posted a comment on MediaWiki.r108217.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108217#c29042

Commit summary for MediaWiki.r108217:

Updated ChangeLog

Santhosh.thottingal's comment:

Translatable pages can now be discourages, = discouraged ?

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


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

2012-01-06 Thread MediaWiki Mail
Santhosh.thottingal changed the status of MediaWiki.r108219 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108219

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108219:

Short URL looks like a page tool

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


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

2012-01-06 Thread MediaWiki Mail
Catrope changed the status of MediaWiki.r108203 to fixme and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29043

Old Status: new
 New Status: fixme

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Catrope's comment:

This is broken, causing codemw.loader::execute Exception thrown by 
ext.vector.simpleSearch: mw.jqueryMsg is undefined/code on TWN. Will look 
into fixing it.


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


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

2012-01-06 Thread MediaWiki Mail
Nikerabbit posted a comment on MediaWiki.r108217.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108217#c29044

Commit summary for MediaWiki.r108217:

Updated ChangeLog

Nikerabbit's comment:

Yes, I did that in a hurry.

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle posted a comment on MediaWiki.r108203.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29045

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Krinkle's comment:

I don't see how this is any different than r107556, please revert. The 
codemediawiki/code module can't have dependencies, and adding it to another 
random module isn't helping the obvious race condition.

And branching Message into a separate module is not a solution in the short 
term because of backwards compatibility.

As I've asked before, what's wrong with modules needing the parser simply 
adding mediawiki.jqueryMsg as a dependency and using it ? Or if you want it 
to be available by default, create a mediawiki.jqueryMsg.init module that 
will extend/overwrite the message prototype and use the parser by default (so 
that you don;t have to do codemw.jqueryMsg.getMessageFunction( )/code). 
Either way, this isn't working.

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


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

2012-01-06 Thread MediaWiki Mail
Catrope posted a comment on MediaWiki.r108203.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29046

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Catrope's comment:

No need for hasty reactions, I'm already fixing this.

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle posted a comment on MediaWiki.r108203.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29047

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Krinkle's comment:

Also, it appears this revision is introducing a regression in the unit tests.

pre
6. mediawiki: mw.message  mw.messages

Message.escaped returns the escaped message
Expected:   
Hello lt;bgt;awesomelt;/bgt; world
Result: 
Hello lt;Bgt;awesomelt;/Bgt; world
Diff:   
 Hello lt;bgt;awesomelt;/bgt; lt;Bgt;awesomelt;/Bgt;  world 
/pre

It looks like the message is parsed as HTML before it is escaped, this causes 
tagnames to become uppercase. Although this is not a big problem, there should 
be no need to parse it before escaping.

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle posted a comment on MediaWiki.r108203.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29048

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Krinkle's comment:

Alrighty, red blur on the swarm freaked me out..

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle posted a comment on MediaWiki.r108215.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108215#c29049

Commit summary for MediaWiki.r108215:

Prevent #firstHeading overriding the language specific h1 height.
Ref Bug 30809

Krinkle's comment:

If at all possible, we should consider looking for another way. I'm not marking 
fixme as I don't if it is inevitable in this case (tagging todo instead), but 
there should be no need for code!important/code in core MediaWiki. Doing so 
makes future development harder than it needs to be.

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


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

2012-01-06 Thread MediaWiki Mail
Catrope posted a comment on MediaWiki.r108203.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29050

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Catrope's comment:

Hrmph, the Message prototype is private, so jQueryMsg can't extend it. Maybe I 
should make it public?

Since you're commenting here and responding quite quickly, any chance you could 
get on IRC?

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


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

2012-01-06 Thread MediaWiki Mail
SVG changed the status of MediaWiki.r106556 to deferred
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106556

Old status:  fixme
 New status: deferred

Commit summary for MediaWiki.r106556:

Replacing  by  in a few of most used extensions and renaming Specialphp to 
..._body.php

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


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

2012-01-06 Thread MediaWiki Mail
SVG changed the status of MediaWiki.r108229 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108229

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108229:

Moretypeos

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


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

2012-01-06 Thread MediaWiki Mail
Siebrand posted a comment on MediaWiki.r108203.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29051

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Siebrand's comment:

Lame comment to watch rev: yay!

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


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

2012-01-06 Thread MediaWiki Mail
Catrope changed the status of MediaWiki.r108203 to resolved and commented 
it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108203#c29052

Old Status: fixme
 New Status: resolved

Commit summary for MediaWiki.r108203:

Use mw.jqueryMsg parser for message parsing to support PLURAL and GENDER
Follow up r107556 and based on the discussions on wikitech-l about this.
mediawiki.jqueryMsg is now loaded always. mw.msg uses the parser if required.
Add qunit test cases.

Catrope's comment:

I don't know where this came from, but it's gone now. Perhaps that's because 
jqueryMsg notices the messages doesn't contain nowiki{{/nowiki and hands it 
off to the normal parser in that case (which it probably shouldn't do for HTML 
output? This conflation of HTML and non-HTML is a mess, as I remarked earlier).

Marking resolved, tests pass as of r108231.

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


Re: [Wikitech-l] Rolling towards 1.19... scheduling a code freeze

2012-01-06 Thread Sumana Harihareswara
On 01/02/2012 08:49 AM, Chad wrote:
 On Sun, Jan 1, 2012 at 12:49 PM, Brion Vibber bvib...@wikimedia.org wrote:
 I'd like us to plan for a code freeze on trunk -- at least a feature 
 refactoring freeze -- starting within a few days to give us all a chance to
 tidy up, catch up with code review, and prepare for deployments.

 
 +1
 

 I believe we've got some general plans to hit deployment in February; if we
 hit code slush this week or so that gives some time for everybody to catch
 up on review, do more thorough testing and -- perhaps most importantly of
 all -- make sure we have good documentation on what's changed and what
 needs to be tested and tried out by real humans!

 
 Sounds good. Want to say Friday's the date to be feature complete to
 give people the rest of this week?
 
 -Chad

Questions about today's freeze (from Niklas, Roan,  me in IRC):

on what -- all of trunk? core? wmf extensions?
So does that mean you have to stop adding features when Friday starts,
or ends? i.e. is Friday the first day of the frozen state, or the last
day of the non-frozen state?
Are we enforcing this in any way within SVN, or are we just agreeing to
quickly revert any new features or refactoring commits that come in
between now and [date]?

-- 
Sumana Harihareswara
Volunteer Development Coordinator
Wikimedia Foundation

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


Re: [Wikitech-l] Rolling towards 1.19... scheduling a code freeze

2012-01-06 Thread Chad
On Fri, Jan 6, 2012 at 9:38 AM, Sumana Harihareswara
suma...@wikimedia.org wrote:
 Questions about today's freeze (from Niklas, Roan,  me in IRC):

 on what -- all of trunk? core? wmf extensions?

Definitely core. Probably should apply to wmf-deployed extensions
too. I think non-deployed extensions can continue doing what they're
doing.

 So does that mean you have to stop adding features when Friday starts,
 or ends? i.e. is Friday the first day of the frozen state, or the last
 day of the non-frozen state?

Let's give until midnight UTC, end of Friday.

 Are we enforcing this in any way within SVN, or are we just agreeing to
 quickly revert any new features or refactoring commits that come in
 between now and [date]?


No changes to access -- there's still legit reasons to commit things such
as followups, regression fixes and the like.

Remember everybody: we're calling this a slush and not an actual
freeze. The idea is to at least try and get trunk feature-complete so we
can make the push in CR.

-Chad

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


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

2012-01-06 Thread MediaWiki Mail
Freakolowsky changed the status of MediaWiki.r102428 to new
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102428

Old status:  fixme
 New status: new

Commit summary for MediaWiki.r102428:

* Small bugfix, has already been fixed in 1.18 and trunk (installer issue: 
hardcoded NLS_NUMERIC_CHARACTERS)

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


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

2012-01-06 Thread MediaWiki Mail
Freakolowsky changed the status of MediaWiki.r106807 to new
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106807

Old status:  fixme
 New status: new

Commit summary for MediaWiki.r106807:

* backport of r106806
* TESTED install

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


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

2012-01-06 Thread MediaWiki Mail
Catrope posted a comment on MediaWiki.r108184.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108184#c29053

Commit summary for MediaWiki.r108184:

ResourceLoader: Add an experimental option to move the main module loading 
queue (the bottom queue) from the bottom of the body up into the head , 
while still being loaded asynchronously. This makes them load earlier, which 
should make the page load faster. This is the product of a long discussion on 
bug 27488

* Added a blocking state to mw.loader . When loading scripts while the 
document is not ready, the loader will use document.write() if blocking is 
true, and append to the body or the head if blocking is false. If the 
document is ready, the loader will always append to the body
* Enable blocking mode while loading the top queue, and disable it after. This 
ensures that modules in the top queue are still loaded in a blocking way as 
they were before
* If $wgResourceLoaderExperimentalAsyncLoading is true, the bottom queue is 
also loaded in the head, but with blocking mode disabled. Otherwise, it's 
loaded at the bottom of the body as before
* scripts-only and messages-only requests need special treatment:
** in the top queue, they can continue to use script src=... tags because 
they are blocking
** if the bottom queue is at the bottom of the body (experimental async 
loading disabled), they can continue to use script src=... tags as before
** if the bottom queue is in the head (experimental async loading enabled), 
they cannot use script src=... tags, because those would block. Instead, 
call mw.loader.load() on the load.php URL

Catrope's comment:

This was fixed in r108220 and r108222.

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


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

2012-01-06 Thread MediaWiki Mail
Siebrand changed the status of MediaWiki.r108231 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108231

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108231:

Fix broken oldParser call in r108230

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle changed the status of MediaWiki.r108236 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108236

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108236:

Followup r108231, remove useless _this

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


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

2012-01-06 Thread MediaWiki Mail
Reedy posted a comment on MediaWiki.r108207.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108207#c29054

Commit summary for MediaWiki.r108207:

revert back to tmhOAuth and add necessary SSL cert file now needed by tmhOAuth

Reedy's comment:

The missing cert was the problem?

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle changed the status of MediaWiki.r108231 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108231

Old status:  ok
 New status: resolved

Commit summary for MediaWiki.r108231:

Fix broken oldParser call in r108230

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r108197 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108197

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108197:

Fix grammar in r108154.
Spotted by kgh 
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Perfcached/en

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


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

2012-01-06 Thread MediaWiki Mail
Hashar changed the status of MediaWiki.r108241 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108241

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108241:

Add a few @since entries

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r96225 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/96225

Old status:  new
 New status: ok

Commit summary for MediaWiki.r96225:

Do not use replaceafter, per r52735 CR

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


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

2012-01-06 Thread MediaWiki Mail
Hashar changed the status of MediaWiki.r108141 to new and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108141#c29055

Old Status: fixme
 New Status: new

Commit summary for MediaWiki.r108141:

rewrite getXCFMetaData() to get ride of im identify

By reading the file header and unpacking the data, we can avoid
shelling out to imagemagick 'identify'. Save up some CPU cycles :D

An XCF is made of a canvas of a given width / height, the various
layers are applied to it which must fit in the canvas. So we just
use the canvas size :-)

I do not think we have any usage for channels count, so I have just
skip that part. I am not sure it makes any sense when the picture
can be made of several layers each using different channels count.

Bits per color is always 8 per definition. Grayscale is 0 - 255
and indexed palette is 256 colors at most.

XCF spec:
http://svn.gnome.org/viewvc/gimp/trunk/devel-docs/xcf.txt?view=markup

pack() / unpack() is familiar to perl monkeys


Fully reimplements r107351

Hashar's comment:

r108248

I just suppressed warnings since I am not willing to handle exceptions thrown 
by wfUnpack() :-D



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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r106807 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106807

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r106807:

* backport of r106806
* TESTED install

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r102428 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102428

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r102428:

* Small bugfix, has already been fixed in 1.18 and trunk (installer issue: 
hardcoded NLS_NUMERIC_CHARACTERS)

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r99668 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/99668

Old status:  new
 New status: ok

Commit summary for MediaWiki.r99668:

(bug 31662) - 'CIDR calculator now has results reversed', regression from r88730

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r102719 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102719

Old status:  fixme
 New status: ok

Commit summary for MediaWiki.r102719:

Add test to catch the problem fixed in r102595.

Confirmed the test works by removing Test from one of the test file names.

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r101298 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101298

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101298:

Fixed 1.17 compatability issue

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r101299 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101299

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101299:

Really added the script into version control

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r101302 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101302

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101302:

Another 1.17 compatibility fix

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r108227 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108227

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108227:

Typofix

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r107450 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107450

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107450:

Random eol w/s

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108248 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108248

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r108248:

XCF: suppress warning on unpack + early exit on error

follow r108141

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r107310 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107310

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107310:

moar w/s

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


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

2012-01-06 Thread MediaWiki Mail
^demon posted a comment on MediaWiki.r108141.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108141#c29056

Commit summary for MediaWiki.r108141:

rewrite getXCFMetaData() to get ride of im identify

By reading the file header and unpacking the data, we can avoid
shelling out to imagemagick 'identify'. Save up some CPU cycles :D

An XCF is made of a canvas of a given width / height, the various
layers are applied to it which must fit in the canvas. So we just
use the canvas size :-)

I do not think we have any usage for channels count, so I have just
skip that part. I am not sure it makes any sense when the picture
can be made of several layers each using different channels count.

Bits per color is always 8 per definition. Grayscale is 0 - 255
and indexed palette is 256 colors at most.

XCF spec:
http://svn.gnome.org/viewvc/gimp/trunk/devel-docs/xcf.txt?view=markup

pack() / unpack() is familiar to perl monkeys


Fully reimplements r107351

^demon's comment:

Handled the exception in r108253. If someone wants to add $length for sanity 
they can.

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


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

2012-01-06 Thread MediaWiki Mail
Platonides changed the status of MediaWiki.r108253 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108253

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108253:

Follow up r108248, r108141: use wfUnpack() so we don't duplicate logic. Could 
also pass $length for sanity, but someone else can do that

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


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

2012-01-06 Thread MediaWiki Mail
^demon posted a comment on MediaWiki.r108234.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108234#c29057

Commit summary for MediaWiki.r108234:

* RN for r106807 and r102428 (sry 4 delay)

^demon's comment:

Shouldn't it be in the changes since 1.17.1 section?

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r101336 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101336

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101336:

* Actually made use of getStableRevId() function in some places.
* Made RevisionReviewForm::doSubmit() sanity check the rev order when rejecting 
edits and also made it use timestamp to compare chronology rather than rev_id. 
WikiPage is used instead of Article.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r102619 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102619

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102619:

* Created FlaggedRevsSetup class with a defineSourcePaths() function to handle 
setting the file paths of code. This is cleaner and avoids global pollution. 
Unit tests added for this function.
* Moved all setting of API pages (conditional or not) to 
FlaggedRevsSetup::setAPIModules().
* Moved most efFlaggedRevs* functions to FlaggedRevsSetup class.
* Moved unconditional setting of $wgHooks to 
FlaggedRevsSetup::setUnconditionalHooks().
* Moved '$wgGroupPermissions['bot']['autoreview'] = true;' line down a bit.
* Moved '$wgExtensionFunctions[] = 'efLoadFlaggedRevs';' line down near 
function.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r102621 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102621

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102621:

* Moved setting of $wgAjaxExportList to 
FlaggedRevsUISetup::defineAjaxFunctions().
* Moved setting of $wgSpecialPageUpdates to 
FlaggedRevsUISetup::defineSpecialPages(). We don't want to have ab 
UnreviewedPages query every time updateSpecialPages is run if aren't even using 
it (and it's also more expensive in the case where $wgFlaggedRevsProtection is 
on).
* Moved FlaggedRevs::ready() function to FlaggedRevsSetup to avoid including 
the FlaggedRevs class on each request.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r102627 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102627

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102627:

* Moved MW_HTML_FOR_DUMP check to FlaggedRevsUISetup::defineHookHandlers(). We 
only want to disable the FR page view UI changes, not any backend and parser 
hook handlers. FR parser variables or includable special pages should still 
work, and DB related hooks should still trigger.
* Moved parser hooks to FlaggedRevsSetup::setConditionalHooks. These are not UI 
hooks.
* Reorganized hook definition order in defineHookHandlers() to group them a bit 
better.
* Added some array type hinting.
* A few whitespace and doc tweaks.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r102760 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102760

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102760:

* Removed getArticleInstance() and updated callers to use getTitleInstance() 
directly.
* Moved $wgJobClasses code and setUnconditionalHooks() call.
* Moved b/c $wgLogActions code to defineLogActionHandlers().
* Minor comment tweaks.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r104215 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/104215

Old status:  new
 New status: ok

Commit summary for MediaWiki.r104215:

Fixed doFlaggedRevsRevTimestamp() in updater, broken in r99755 and also had the 
wrong directory

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r106625 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106625

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r106625:

* Added proper bypassValidationKey() function to review submission class and 
removed API hack.
* Improved RevisionReviewForm::validationKey by using a per-user temp secret key

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


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

2012-01-06 Thread MediaWiki Mail
Reedy posted a comment on MediaWiki.r108052.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108052#c29058

Commit summary for MediaWiki.r108052:

API module to search pages by coordinate

Reedy's comment:

$wgMaxGeoSearchRadius at the top is unused, as is $count

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108196 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108196

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108196:

* Always show 'revreview-editnotice' msg on edit form if the edit can't be 
automatically reviewed. Also tweaked the message for consistency (also matches 
edit form review checkbox).
* Made editWillBeAutoreviewed() use quickUserCan() for performance.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r101316 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101316

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101316:

* Made $oldFrev a member of RevisionReviewForm and get the tags from it, rather 
than fetching them in a seperate query.
* Removed getRevisionTags(), now unused.
* Renamed $oflags - $oldFlags.
* Minor comment style fixes.

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r108169 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108169

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108169:

Add missing variable. Thanks to Yuryu for bug fix.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r101292 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101292

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101292:

* Fixed undefined $rc var from r101286
* Made getRevQuality() and revIsFlagged() only need a rev ID
* Removed unused $wgUser

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r100948 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/100948

Old status:  new
 New status: ok

Commit summary for MediaWiki.r100948:

Hotfix for breaking change of WikiError - Status

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r101359 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101359

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101359:

guessVariant doc and explicit boolean typecast

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r103953 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/103953

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r103953:

MFT r103945: use canonical user NS in export, workaround for search prefix bug 
31697 and friends

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r105009 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/105009

Old status:  new
 New status: ok

Commit summary for MediaWiki.r105009:

Use wfGetIP() instead of WebRequest method that is not defined in this MW 
version (bug 32754)

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r107332 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107332

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107332:

Originally fixed in trunk back in r93677, so no idea why it is still there.

Fixes fatal in production

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r107541 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107541

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107541:

1.18wmf1: Update MarkAsHelpful and MoodBar to trunk state

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r104133 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/104133

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r104133:

add logout link to footer

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r106134 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106134

Old status:  new
 New status: ok

Commit summary for MediaWiki.r106134:

small spacing issues

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r106133 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106133

Old status:  new
 New status: ok

Commit summary for MediaWiki.r106133:

remove duplicate variables

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


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

2012-01-06 Thread MediaWiki Mail
MaxSem posted a comment on MediaWiki.r108052.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108052#c29059

Commit summary for MediaWiki.r108052:

API module to search pages by coordinate

MaxSem's comment:

r108257

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r106131 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106131

Old status:  new
 New status: ok

Commit summary for MediaWiki.r106131:

modify wgAutoloadClasses array creation

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108172 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108172

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108172:

Branching ShortUrl into 1.18wmf1 for bug 1450 and bug 29827

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108260 to ok and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108260#c29060

Old Status: new
 New Status: ok

Commit summary for MediaWiki.r108260:

Improve RELEASE-NOTES-1.18 for r108045

^demon's comment:

Not perfect, but better :p

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108045 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108045

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r108045:

* (bug 29102) Upgrade fails Unknown character set: 'mysql4'.

Merges r108044

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108259 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108259

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108259:

release-notes: remove notes for r108203; adding new ones for r108230

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r107770 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107770

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r107770:

(bug 33447) Link to the broken image tracking category from Special:Wantedfiles

Also add a note about how if you use a foreign repo, you're going to get plenty 
false positives.

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


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

2012-01-06 Thread MediaWiki Mail
MaxSem changed the status of MediaWiki.r108160 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108160

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108160:

[Bug 33547] New: Toggling last section in MobileFrontend shows JS error

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r108256 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108256

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108256:

NULL - null, ping r107199

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r108257 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108257

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108257:

rm unused vars

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r107932 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107932

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r107932:

Fix Bug 33384 - database drivers cannot be provided by extension
Author: Adam Wight

Love the simplicity of this fix.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108218 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108218

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108218:

Read the language options from the content language. In case of content 
language != user language you would get an empty string.
Prefix the language options message with the extenaion name to avoid conflicts 
with core and/or other extension message keys
Refine message documentation

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108237 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108237

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108237:

attempt to get rid of weird error

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


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

2012-01-06 Thread MediaWiki Mail
Krinkle changed the status of MediaWiki.r108235 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108235

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108235:

Followup r108184: fix loading in Opera. Before, Opera would only begin to 
render the document after all async scripts were loaded, now it's the other way 
around (scripts load after the document is ready). This is not an improvement 
compared to the current situation, but it's not a regression either.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108245 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108245

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108245:

Bug fix: when an error comes back from the submit api, it's a message token, 
not a translated message

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r107455 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107455

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107455:

r107451 - Add message to special page

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108171 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108171

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108171:

Add mw-content-ltr/rtl class: fixes bug 33538: Line numbers are outside of left 
border when using source ... line start=  on RTL wikis

Also make sure mw-content-ltr/rtl is the same as the dir attribute value. 
Restrict that value to either ltr or rtl (not sure if rtl really needed; source 
code is always ltr). Also remove text-align:left; as it can/should be set 
manually in the source tag.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108042 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108042

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108042:

deprecated from project - these classes do not seem to be in use by search.
they will be removed in the next version together with thier data file

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


Re: [Wikitech-l] wikicaptcha on GitHub

2012-01-06 Thread Platonides
Hello Cristian,
Kudos to Alex and you for starting the ball rolling on this project.
I recommend you to get an account in our repository [1].

Integrating this into ConfirmEdit extension shouldn't be hard. It's the 
extra features what makes this tricky. This system is interesting for 
gathering translations, but doesn't work for verifying that the answer 
is right. How would you verify that?
The approach that comes to my mkind is to show both the current captcha 
plus another, optional, captcha, with a note about how filling that 
second captcha helps wikisource, and that the answer will be logged with 
their username/ip.
Then dump all answers in a log restricted to captcha-reviewers.

1- http://www.mediawiki.org/wiki/Commit_access#Requesting_commit_access

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r106656 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106656

Old status:  new
 New status: ok

Commit summary for MediaWiki.r106656:

WikimediaMaintenance: add script for getting the job queue length on all wikis

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108261 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108261

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108261:

Fix mixed whitespace in r107455

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r108179 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108179

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108179:

ServerNames is a space delimited string, not an array.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r107222 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107222

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107222:

Fix conf name. Follow up to r107220.

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r102370 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102370

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102370:

Split AbuseFilter schema updates for SQLite out after briefly talking
with MaxSem about the desired course of action.  Perhaps later these
can be merged back, but this seemed to be the quickest way to get
AbuseFilter working with SQLite working for now.

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


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

2012-01-06 Thread MediaWiki Mail
^demon changed the status of MediaWiki.r107221 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107221

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107221:

Changes for compatibility with LdapAuthentication 2.0a

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


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r102174 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102174

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102174:

Fixed compatibility with 1.17

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


Re: [Wikitech-l] Pages-logging dump - more information?

2012-01-06 Thread Platonides
On 06/01/12 10:42, Katja Mueller wrote:
 Dear list,

 do you provide more information about the pages-logging dump somewhere?
 While parsing it we came across some questions that we are trying to
 clarify:

 * Why are there only ~40 million logs (at over 450 million revisions)?
 Which logs does the pages-logging dump / the public logging table
 (not) contain? (We double-checked the number of logs on the database,
 using our Toolserver-Account (Logging table).)

The logs live at Special:Log Most edits don't produce revisions, 
although some actions produce both and a log (upload, protect, move...) 
and others only leave a log entry (such as a block or patrol)


 * Can you give us more information about the TextElement that is defined
 forlogitem  in the XML Schema?
 Definition:element name=text type=mw:TextType/

 In pages-logging it sometimes occurs as:
 text deleted=deleted /

 How is this being used?

It's used to hide log entries when they contain personal info.
Replaces logtitle and params.
It's not used for anything in non-revdeleted entries.


___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


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

2012-01-06 Thread MediaWiki Mail
Reedy changed the status of MediaWiki.r101991 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101991

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101991:

Bug 21788 - allow fuzzy.php to take optional namespace prefix

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


  1   2   >