Re: [Wikitech-l] should we keep $wgDeprecationWhitelist

2012-01-12 Thread Krinkle
Just my 2 cents:

I don't think we need this kind of deprecation-warning filter in core.

As far as I know deprecation warnings are never shown more than once for
every method,
so it isn't going to pollute or obfuscate the error output when you're
working with the code.

And users/production wouldn't be seeing them anyway, right ?

And if you maintain compatibility with an older version of MediaWiki, you
might also find
it useful to raise $wgDeprecationReleaseLimit ocasionally to only if
there's any usage of
more problematic / longest deprecated methods, and lower it again when you
want to see
them all.

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


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

2012-01-12 Thread MediaWiki Mail
Siebrand changed the status of MediaWiki.r54363 to reverted
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/54363

Old status:  deferred
 New status: reverted

Commit summary for MediaWiki.r54363:

Consistent spelling. All-caps rendering vis CSS.

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


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

2012-01-12 Thread MediaWiki Mail
Raymond changed the status of MediaWiki.r108694 to fixme and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108694#c29387

Old Status: new
 New Status: fixme

Commit summary for MediaWiki.r108694:

Load the Recaptcha class in a way that has some chance of even working

Raymond's comment:

PHP Fatal error: Class 'ReCaptcha' not found in 
/www/w/extensions/ConfirmEdit/ConfirmEditHooks.php on line 15

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


Re: [Wikitech-l] MediaWiki security and maintenance release 1.18.1

2012-01-12 Thread Bryan Tong Minh
On Wed, Jan 11, 2012 at 11:11 PM, Sam Reed re...@wikimedia.org wrote:
 Full release notes:
 http://svn.wikimedia.org/svnroot/mediawiki/tags/REL1_18_1/phase3/RELEASE-NOT
 ES

This link is broken; should be suffixed -1.18

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


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

2012-01-12 Thread MediaWiki Mail
Siebrand changed the status of MediaWiki.r108694 to new
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108694

Old status:  fixme
 New status: new

Commit summary for MediaWiki.r108694:

Load the Recaptcha class in a way that has some chance of even working

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108700:

spacing fixes

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


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

2012-01-12 Thread MediaWiki Mail
Hashar changed the status of MediaWiki.r106780 to resolved and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106780#c29388

Old Status: fixme
 New Status: resolved

Commit summary for MediaWiki.r106780:

Fix for bug 33270: default to 'default' search profile
Had this fixed in one of my wikies but not committed

Hashar's comment:

Resolved by r108712

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r106780.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106780#c29389

Commit summary for MediaWiki.r106780:

Fix for bug 33270: default to 'default' search profile
Had this fixed in one of my wikies but not committed

Hashar's comment:

uncommented with r108712

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r107975.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107975#c29390

Commit summary for MediaWiki.r107975:

MFT r105341, t105853, r106780

Hashar's comment:

Need a merge of r108712 to be fixed.

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r108712.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108712#c29391

Commit summary for MediaWiki.r108712:

bug 33583 search ns user pref ignored!

r106780 to fix bug 33270 introduced a new bug that prevented selected
namespaces for search to be applied. This patch fix the issue.

Credits to Brad Jorsch

Hashar's comment:

Tagging 1.18wmf1 since it fix a regression that was applied to live site with 
r107975.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108713:

Fix 3 transliteration rules for Assamese- Use atomic code points instead of 
decomposed forms for ড়, ঢ়, য়

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r108488.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108488#c29392

Commit summary for MediaWiki.r108488:

r108353: Distinguish null/false in FileBackend::fileExists(). This is intended 
for things that might really care.

Hashar's comment:

Of course my english is crappy. The above question was:


doGetFileStat() does not seems to return null. So is that really needed? :-)




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


Re: [Wikitech-l] JavaScript on Special:UserLogin?

2012-01-12 Thread Happy Melon
On 12 January 2012 02:47, Daniel Barrett d...@vistaprint.com wrote:

 On 11 January 2012 21:51, Daniel Barrett d...@vistaprint.com wrote:
  * Remove any trailing @companyname.com from the username. Users in
 our company are accustomed to logging in this way on
 their Windows boxes, and we'd get several support calls per week from
 people who can't log into the wiki because they
 were adding @companyname.com onto their wiki usernames.

 The way we do it on our Mantis bugtracker is to use the LDAP server
 for all logins. Something like that for MediaWiki would do the job
 too. One password!

 Yes, we use the LDAPAuthentication extension for MediaWiki. But usernames
 on the login page still must be foo rather than f...@companyname.com.
 Hence the JavaScript to remove @.*.

 DanB


Of course, if the login form code wasn't such a swamp, there'd be a hook
you could use to preprocess the usernames server-side...  :-(

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


Re: [Wikitech-l] New committer

2012-01-12 Thread Robert Vogel
Hi DJ and hi everybody!

Thanks for the warm welcome :), and DJ thanks for your hints/questions 
concerning SVN. I am quite busy at the moment so I did not set up my SVN client 
(Tortoise) for the MW repo yet. But I hope to get it done next week.

Afterwards I will start adding some of the announced extensions. The Azure 
stuff is already in SVN. Markus committed it.

I am very excited about your Azure extensions. You said you had everything 
working with MW 1.18 except the media storage? As far as I know you work on the 
database abstraction layer for Azure SQL. Is that right? So with the file 
backend we should have a complete Azure bundle, shouldn't we?

--
Robert Vogel
Social Web Technologien
Softwareentwicklung
Hallo Welt! - Medienwerkstatt GmbH


__

Untere Bachgasse 15
93047 Regensburg

Tel.  +49 (0) 941 - 66 0 80 - 198
Fax   +49 (0) 941 - 66 0 80 - 189

www.hallowelt.biz
vo...@hallowelt.biz


Sitz: Regensburg
Amtsgericht: Regensburg
Handelsregister: HRB 10467
E.USt.Nr.: DE 253050833
Geschäftsführer: Anja Ebersbach, Markus Glaser, Dr. Richard Heigl, Radovan 
Kubani

-Ursprüngliche Nachricht-
Von: wikitech-l-boun...@lists.wikimedia.org 
[mailto:wikitech-l-boun...@lists.wikimedia.org] Im Auftrag von DJ Bauch
Gesendet: Donnerstag, 12. Januar 2012 02:15
An: Wikimedia developers
Betreff: Re: [Wikitech-l] New committer

I will be trying again tonight. I got some hints from Ben Lobaugh. It
looks like the problem I've been having is that I was using the host
name in conjunction with my svn+ssh connection attempt, when I should
have been using the name of my PuTTY session.

On Wed, Jan 11, 2012 at 5:38 PM, Platonides platoni...@gmail.com wrote:
 On 11/01/12 18:25, DJ Bauch wrote:
 Welcome, Robert!
 You and I should be crossing paths. I have everything working through
 version 1.18 on Windows Azure, except for the media storage. Right
 this moment I am trying to figure out how to get my code committed,
 but I have been struggling with setting up the Subversion access from
 my Windows 7 box. Any hints or pointers on how to set things up may
 help me transition from a potential to an actual contributer.

 So, what are you trying to do? The cli or soething else?, did you read [1]?
 You will need to use plink on windows to make the ssh tunnel, pageant is
 recommended but not required (unless it's a plink with no console?).

 Take a look at [2].

 1-http://www.mediawiki.org/wiki/Commit_access#Getting_started_and_set_up
 2-http://agateau.wordpress.com/2007/07/03/windows-svnssh-and-the-subversion-command-line-client/


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

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

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


[Wikitech-l] Please Welcome Andre Engels

2012-01-12 Thread Tomasz Finc
Greetings all,

I'm pleased to announce that Andre Engles has decided to join the Mobile
team as our data analyst contractor. In this role he'll provide the mobile
team with key metrics to guide our software development efforts. He'll be
working closely with Diederik, Phil, Erick Zachte, me, and others to
standardize, automate, and expand our growing analytics needs.

For those that don't know ... Andre is one of the 'oldest' Wikipedians (in
terms of years on the site). His first edits on Wikipedia were from March
2001 (two things to show how early: 1. That's before the first
slashdotting; 2. English Wikipedia several years ago had a list of the
oldest still active editors, He was at number 8. Number 7 was Jimbo). Apart
from being an editor and sysop he has also been active as both an operator
and a programmer of bots (still active as an operator, but only very
little as a programmer nowadays), and has Steward rights. He also helped in
the creation of Wikimedia Commons.
Apart from Wikipedia, he is active in various open content projects. These
include PGDP (Project Gutenberg Distributed Proofreaders) and
Openstreetmap; recently has done similar things in the area of Dutch
etymology.

Andre studied mathematics, got his PhD in computer science. His last job
was at C-Content, which is a company working in search engines. For them he
mostly did harvesting and search request analysis. Before that he worked
for the RD department of Ericsson Netherlands.

He just celebrated his 40th birthday, and is (and always have been) single,
living in the city of ´s-Hertogenbosch. His free time is mostly spent
behind his computer, either with the above mentioned projects or playing
games. Apart from that he's an avid Go player, having reached the grade of
3 dan.

I'm really honored to bring on such a long time community member to the
Mobile team.

Please join me in welcoming Andre!

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

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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108715:

Add  Ahirani(ahr) language support by using Lohit Devanagari font.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108716:

(bug 32508) Fix updater output

site_stats updated is still a database action.
Purge cache is now the very last one.

Output:

snip
...site_stats is populated...done.
Purging caches...done.

Done.

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


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

2012-01-12 Thread MediaWiki Mail
Siebrand changed the status of MediaWiki.r108717 to deferred
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108717

Old status:  new
 New status: deferred

Commit summary for MediaWiki.r108717:

new extension: PrivatePageProtection

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


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

2012-01-12 Thread MediaWiki Mail
Duesentrieb posted a comment on MediaWiki.r108717.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108717#c29394

Commit summary for MediaWiki.r108717:

new extension: PrivatePageProtection

Duesentrieb's comment:

this extension is complementary to Lockdown. It implements group based access 
on a per-page basis, using a parser function to list the groups on-page

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


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

2012-01-12 Thread MediaWiki Mail
Siebrand posted a comment on MediaWiki.r108717.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108717#c29395

Commit summary for MediaWiki.r108717:

new extension: PrivatePageProtection

Siebrand's comment:

pre
+   'privatepp-desc' = 'Allowes restricting page access based on user 
group',
+   
+   'privatepp-lockout-prevented' = 'Lockout prevented: You have tried to 
restrict access to this page to {{PLURAL:$2|the group|one of the groups}} $1. '
+   . 
'Since you are not a member of {{PLURAL:$2|this group|any of these groups}}, 
you would not be able to access the page after saving it. '
+   . 
'Saving was aborted to avoid this.',
/pre

* typo: Allowes - Allows
* please do not concatenate strings, but just add newlines. 
[[extension:localisationUpdate]] chokes on it, I believe, and 
[[extension:translate]] cannot handle it either.

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


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

2012-01-12 Thread MediaWiki Mail
MaxSem posted a comment on MediaWiki.r108549.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108549#c29396

Commit summary for MediaWiki.r108549:

Added localisation

MaxSem's comment:

Thanks, fixed.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108710:

Fix fatal in r108694.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108711:

stylize.php, some indentation fixes and some brackets added.

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


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

2012-01-12 Thread MediaWiki Mail
Nikerabbit posted a comment on MediaWiki.r108715.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108715#c29397

Commit summary for MediaWiki.r108715:

Add  Ahirani(ahr) language support by using Lohit Devanagari font.

Nikerabbit's comment:

Did you also update docs on mediawiki.org?

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


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

2012-01-12 Thread MediaWiki Mail
Nikerabbit posted a comment on MediaWiki.r108720.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108720#c29398

Commit summary for MediaWiki.r108720:

Follow-up r108549: {{PLURAL}}

Nikerabbit's comment:

You don't and shouldn't pass the number twice. Plural expects formatted numbers.

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


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

2012-01-12 Thread MediaWiki Mail
Catrope changed the status of MediaWiki.r108701 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108701

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108701:

Bug fix: incorrect impressions on form load in
modules/jquery.articleFeedbackv5/jquery.articleFeedbackv5.js:
- Updated load():
- to accept a second argument from so that the location can 
be passed in,
- to allow auto as the first argument (what to display),
- and to pass from on to showForm() and showCTA()
- Updated showForm() and showCTA() to accept from as an argument, and 
to
  use that for the tracking id
- Updated both uses of load(), and the use of showCTA() on success, to 
send
  the new arguments

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


Re: [Wikitech-l] Please Welcome Andre Engels

2012-01-12 Thread Roan Kattouw
On 1/12/12, Tomasz Finc tf...@wikimedia.org wrote:
 Greetings all,

 I'm pleased to announce that Andre Engles has decided to join the Mobile
 team as our data analyst contractor. In this role he'll provide the mobile
 team with key metrics to guide our software development efforts. He'll be
 working closely with Diederik, Phil, Erick Zachte, me, and others to
 standardize, automate, and expand our growing analytics needs.

Yay, another Dutchman :) . The Dutch cabal is taking over, slowly but
surely, I tell you.

Welcome, Andre!

Roan

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


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

2012-01-12 Thread MediaWiki Mail
Jeroen De Dauw posted a comment on MediaWiki.r108574.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108574#c29399

Commit summary for MediaWiki.r108574:

Don't include files with $magicWords, but use $wgExtensionMessagesFiles.

Inspired by r108558.

Jeroen De Dauw's comment:

You updated the version of SRF from 1.7 to 1.7.1. For all extensions I 
maintain, please never do this, and rather change it to 1.7.1 alpha, or leave 
it as it is if there already is some kind of status identifier after the 
version number. Right now I will not be able to distinguish between people 
reporting bugs that have version 1.7.1 that has been pulled from svn at some 
point before the rel or actually got the rel :/

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


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

2012-01-12 Thread MediaWiki Mail
Fomafix posted a comment on MediaWiki.r80495.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/80495#c29400

Commit summary for MediaWiki.r80495:

This override is no longer needed. 
Tables in div class=toccolours or in other places no longer force a white 
background which was only added to avoid underlines from h2 headings from  
shining through (which is no longer the case since bug 26449 was solved in 
r79087 and r79091)
* This commit fixes bug 26708

Fomafix's comment:

In 
[[svn:trunk/phase3/resources/mediawiki.action/mediawiki.action.history.diff.css]]

syntaxhighlight lang=css
table.diff,
td.diff-otitle,
td.diff-ntitle {
background-color: transparent;
}
/syntaxhighlight

from r105280 is not necessary anymore. 

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108721:

Fix r108720: No need to introduce 2
Consistency tweaks in preparation for adding extension to translatewiki.net

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108722:

r108549: Register extension for translatewiki.net.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108706:

Reverted r108695: as these are still used by upload stash and I don't feel like 
changing that or thumb-handler.php. The whole public temp file usage by 
MediaWiki needs to be thought out more, but this can happen later.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102972:

(bug 17865) Fix double-parsed error messages in Cite. Modified patch by Brad 
Jorsch

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r103761:

(bug 32451) Clicking on a special characters section loses the selection in 
IE8. Patch by Lupo

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r101611:

Fix r101532 which was so broken that I refuse to believe it was tested even 
once:
* Feeding a fully-qualified URL through mw.util.wikiGetlink() prefixes 
wgServer, which breaks the link
* articlefeedback-survey-disclaimer containted wiki syntax for an external 
link, but this doesn't work in JS
* Links in the survey dialog show up in black due to some annoying JUI CSS, so 
even if that wiki syntax had worked, it wouldn't have produced a visually 
discernable link

Fixes:
* Remove mw.util.wikiGetLink() calls
* Split up articlefeedback-survey-disclaimer the way 
articlefeedback-form-panel-helpimprove-note is, with a separate message for the 
link text
** Document these split messages in qqq
* Move the link target out of a message and into a global
* Add a CSS workaround to make the privacy link in the survey dialog show up in 
blue rather than in black

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r106679:

Remove server-side validation of redirect URLs, not needed and getting in the 
way for AFTv5

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r107425:

Followup r105574 per CR: clone arrays from mw.config so we can modify them 
safely, rather than corrupting the global arrays

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


Re: [Wikitech-l] Please Welcome Andre Engels

2012-01-12 Thread Sumana Harihareswara
On 01/12/2012 06:22 AM, Tomasz Finc wrote:
 Greetings all,
 
 I'm pleased to announce that Andre Engles has decided to join the Mobile
 team as our data analyst contractor. In this role he'll provide the mobile
 team with key metrics to guide our software development efforts. He'll be
 working closely with Diederik, Phil, Erick Zachte, me, and others to
 standardize, automate, and expand our growing analytics needs.
 
 For those that don't know ... Andre is one of the 'oldest' Wikipedians (in
 terms of years on the site). His first edits on Wikipedia were from March
 2001 (two things to show how early: 1. That's before the first
 slashdotting; 2. English Wikipedia several years ago had a list of the
 oldest still active editors, He was at number 8. Number 7 was Jimbo). Apart
 from being an editor and sysop he has also been active as both an operator
 and a programmer of bots (still active as an operator, but only very
 little as a programmer nowadays), and has Steward rights. He also helped in
 the creation of Wikimedia Commons.
 Apart from Wikipedia, he is active in various open content projects. These
 include PGDP (Project Gutenberg Distributed Proofreaders) and
 Openstreetmap; recently has done similar things in the area of Dutch
 etymology.
 
 Andre studied mathematics, got his PhD in computer science. His last job
 was at C-Content, which is a company working in search engines. For them he
 mostly did harvesting and search request analysis. Before that he worked
 for the RD department of Ericsson Netherlands.
 
 He just celebrated his 40th birthday, and is (and always have been) single,
 living in the city of ´s-Hertogenbosch. His free time is mostly spent
 behind his computer, either with the above mentioned projects or playing
 games. Apart from that he's an avid Go player, having reached the grade of
 3 dan.
 
 I'm really honored to bring on such a long time community member to the
 Mobile team.
 
 Please join me in welcoming Andre!
 
 --tomasz

I'm constantly awed at the caliber of people I get to work with.

Welcome, Andre!

-- 
Sumana Harihareswara
Volunteer Development Coordinator
Wikimedia Foundation

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


[Wikitech-l] postgreSQL testing

2012-01-12 Thread Antoine Musso
Hello,

I have added a new continuous integration job to check our postgres  
support.
This is exactly the same job as MediaWiki-phpunit, only the database  
backend
change.

The link is:
   https://integration.mediawiki.org/ci/job/MediaWiki-postgres-phpunit/

As of now, there are two tests failing.


-- 
Antoine hashar Musso


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


[Wikitech-l] proposed tech conference anti-harassment policy

2012-01-12 Thread Sumana Harihareswara
The Wikimedia Foundation is dedicated to a harassment-free conference
experience for everyone.  I'm proposing a fairly short and standard
anti-harassment policy of the type that's becoming best practice for
tech conferences and hackathons.

Draft: https://www.mediawiki.org/wiki/User:Sumanah/AHP

I don't imagine I'll get much response on this, but just wanted to put
it out there before implementing.  I intend on putting this into place
by the middle of next week, in time for the San Francisco hackathon
(starting January 20th).

Comments on the talk page, please.
-- 
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] proposed tech conference anti-harassment policy

2012-01-12 Thread David Gerard
On 12 January 2012 16:00, Sumana Harihareswara suma...@wikimedia.org wrote:

 The Wikimedia Foundation is dedicated to a harassment-free conference
 experience for everyone.  I'm proposing a fairly short and standard
 anti-harassment policy of the type that's becoming best practice for
 tech conferences and hackathons.
 Draft: https://www.mediawiki.org/wiki/User:Sumanah/AHP


Nice one :-) A candidate for WMF and wider Wikimedia events in general, too.


- d.

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

Re: [Wikitech-l] proposed tech conference anti-harassment policy

2012-01-12 Thread Chad
On Thu, Jan 12, 2012 at 11:04 AM, David Gerard dger...@gmail.com wrote:
 On 12 January 2012 16:00, Sumana Harihareswara suma...@wikimedia.org wrote:

 The Wikimedia Foundation is dedicated to a harassment-free conference
 experience for everyone.  I'm proposing a fairly short and standard
 anti-harassment policy of the type that's becoming best practice for
 tech conferences and hackathons.
 Draft: https://www.mediawiki.org/wiki/User:Sumanah/AHP


 Nice one :-) A candidate for WMF and wider Wikimedia events in general, too.


I don't see anything preventing harassment over choice of DBMS ;-)

All kidding aside, this looks great. Agree with David wholeheartedly
here.

-Chad

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

Re: [Wikitech-l] proposed tech conference anti-harassment policy

2012-01-12 Thread Tei
On 12 January 2012 17:09, Chad innocentkil...@gmail.com wrote:
 On Thu, Jan 12, 2012 at 11:04 AM, David Gerard dger...@gmail.com wrote:
 On 12 January 2012 16:00, Sumana Harihareswara suma...@wikimedia.org wrote:

 The Wikimedia Foundation is dedicated to a harassment-free conference
 experience for everyone.  I'm proposing a fairly short and standard
 anti-harassment policy of the type that's becoming best practice for
 tech conferences and hackathons.
 Draft: https://www.mediawiki.org/wiki/User:Sumanah/AHP


 Nice one :-) A candidate for WMF and wider Wikimedia events in general, too.


 I don't see anything preventing harassment over choice of DBMS ;-)

 All kidding aside, this looks great. Agree with David wholeheartedly
 here.


Is amazing how this whole wikimedia thing as changed since 2002.
I don't understand half the messages on this mail list.   This is
really a complex and professional organization, that has advanced a
lot in the past years. I lurk on the mail list, tryiing to find a way
to help, but is really hard, because there are here top-notch
profesionals doing things the best way that is possible. You guys
rocks.



-- 
--
ℱin del ℳensaje.

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

[Wikitech-l] MediaWiki 1.18 learnings from a wiki admin extension writer

2012-01-12 Thread Daniel Barrett
As MediaWiki 1.19 is getting ready, I'd like to offer information on how 
MediaWiki 1.18.0 was the most difficult MW upgrade I've ever been through.

Some background: my team administers an internal wiki at a major company with 
~2000 users, over 100 extensions (many of them custom/unreleased), and 100K 
articles. I've been upgrading MW regularly since 1.11 - every release and patch 
- and have never had this much trouble before, mainly because of extensions 
that broke in 1.18.  The typical MW upgrade takes me a day or two including 
regression-testing our extensions.  But 1.18 has taken me weeks and I'm still 
not done.

This message is meant to be constructive  helpful, not blameful: it's quite 
possible that every issue was our fault for not keeping up on exactly which 
functions  globals were being deprecated, etc. I'd just like to describe what 
kinds of things broke for a reasonably active wiki run by well-meaning people, 
and to document how we fixed them.

So, here's the list of what we had trouble with, and what we did.  I welcome 
any improvements to our fixes!

1.  The global variable $action disappeared, breaking a bunch of our 
extensions.  I switched to $wgRequest-getVal('action').

2.  The removal of Xml::hidden() caused one of our extensions to break.  I 
switched to Xml::input(..., array('type', 'hidden'))

3.  A few of our older extensions were not ported to ResourceLoader yet and 
were adding JS and CSS via $wgOut-add... calls. They worked in 1.17 and all 
broke in 1.18. I ported them to use ResourceLoader, but this is not a good 
solution yet because of bug 31676 (the 32-stylesheet limit of IE, 
https://bugzilla.wikimedia.org/show_bug.cgi?id=31676) which IMHO is a very 
serious time-bomb waiting to explode.  I hope it makes it into 1.19wmf 
deployment as planned.

4.  Some of our parser tag extensions had a bug, in that they didn't return 
a value in the tag callback.  (These tags had no visual display.)  This didn't 
cause problems in 1.17 and earlier, but in 1.18.0 it caused a UNIQ.QINU 
string to render on the page.  I fixed our extensions to return the empty 
string, and the problem went away.

5.  The removal of $wgMessageCache-addMessage() broke many extensions, 
some ours and some from mediawiki.org like SimpleForms.  Some fixes just 
required use of the i18n file. Our more difficult issue was that we were 
injecting system messages into articles to add tracking categories. On advice 
from this list (thanks!), we used code patterned after 
Parser::addTrackingCategory() to inject categories and it works fine, actually 
much better than what we had.

6.  The removal of ts_makeSortable() from wikibits.js threw off a bunch of 
our JavaScript: we were using the function to sort on a different column than 
the first one on render, and in extensions that create tables within dialogs. 
We left the problem unfixed until I can understand the new jQuery UI way of 
doing things (jquery.ui.sortable.js).

7.  Nearly 100% of our customizations to WikiEditor 1.17 broke in 1.18. We 
had followed the documented rules on mediawiki.org, using extensions, 
ResourceLoader, etc., and everything worked in 1.17.  Nevertheless in 1.18, 
toolbars and menus disappeared in IE. Menus appeared multiple times instead of 
once in Firefox. JavaScript objects in one module became undefined in others, 
even with proper dependencies. Some of these issues are still not worked out, 
but most were fixed by a variety of changes.

8.  Our MediaWiki:common.js stopped running on the login page. I realize 
this was a security fix; it just took me by surprise.  Fixed by writing a 
custom extension using the hook UserLoginForm to inject the few lines of JS we 
needed, and I'm evaluating other non-JS solutions for more security.

9.  The addHandler() function in JavaScript does not seem to work in IE8 
anymore. We worked around this by using jQuery's bind function.

At this point, our test wiki is stable and I am not anticipating any further 
large issues, so we should roll out in the next two weeks or so.

Thanks for reading, and I hope this helps someone,
DanB


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


Re: [Wikitech-l] MediaWiki 1.18 learnings from a wiki admin extension writer

2012-01-12 Thread Sumana Harihareswara
On 01/12/2012 11:31 AM, Daniel Barrett wrote:
 So, here's the list of what we had trouble with, and what we did.  I welcome 
 any improvements to our fixes!

Daniel, thank you so much for taking the time to list these out.  And my
sympathies that your upgrade was so troublesome!
-- 
Sumana Harihareswara
Volunteer Development Coordinator
Wikimedia Foundation

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


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

2012-01-12 Thread MediaWiki Mail
Hashar changed the status of MediaWiki.r108075 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108075

Old status:  ok
 New status: resolved

Commit summary for MediaWiki.r108075:

[TestSwarm] Update testswarm-mw-fetcher.php script
* Improve, add, update inline documentation
* Add ability to set TestSwarmAPI::doAddJob options from configuration file
* Change default job_name to include anchor tag to MW.org CR (fixes bug 33501)

UNTESTED - TO BE TESTED IN TESTSWARM.WMFLABS

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


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

2012-01-12 Thread MediaWiki Mail
Jeroen De Dauw posted a comment on MediaWiki.r108602.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108602#c29401

Commit summary for MediaWiki.r108602:

fix bug 16985 based on attached patch by Nakon

Jeroen De Dauw's comment:

I was thinking the same thing when applying the patch but did not see any 
obvious way to not do this. Then again, I'm not that familiar with the relevant 
part of the schema, so if you know a better way to get the needed info, I'd be 
very welcome :)

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


Re: [Wikitech-l] proposed tech conference anti-harassment policy

2012-01-12 Thread Amir E. Aharoni
2012/1/12 Chad innocentkil...@gmail.com

 On Thu, Jan 12, 2012 at 11:04 AM, David Gerard dger...@gmail.com wrote:
  On 12 January 2012 16:00, Sumana Harihareswara suma...@wikimedia.org 
  wrote:
 
  The Wikimedia Foundation is dedicated to a harassment-free conference
  experience for everyone.  I'm proposing a fairly short and standard
  anti-harassment policy of the type that's becoming best practice for
  tech conferences and hackathons.
  Draft: https://www.mediawiki.org/wiki/User:Sumanah/AHP
 
 
  Nice one :-) A candidate for WMF and wider Wikimedia events in general, too.
 

 I don't see anything preventing harassment over choice of DBMS ;-)

In all seriousness, i was harassed more than once IRL and online for
saying that SVN doesn't suck as much as die-hard Git fans say it does.
This policy can say preferred technology in addition to Creative
Commons license.

--
Amir Elisha Aharoni · אָמִיר אֱלִישָׁע אַהֲרוֹנִי
http://aharoni.wordpress.com
‪“We're living in pieces,
I want to live in peace.” – T. Moore‬

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

Re: [Wikitech-l] MediaWiki 1.18 learnings from a wiki admin extension writer

2012-01-12 Thread David Gerard
On 12 January 2012 16:31, Daniel Barrett d...@vistaprint.com wrote:

 So, here's the list of what we had trouble with, and what we did.  I welcome 
 any improvements to our fixes!


This deserves a page on mediawiki.org.


- d.

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

Re: [Wikitech-l] Please Welcome Andre Engels

2012-01-12 Thread alolita.sha...@gmail.com
Welcome Andre! Great to have you on the engineering team finally!

--
Alolita Sharma

- Reply message -
From: Roan Kattouw roan.katt...@gmail.com
To: Wikimedia developers wikitech-l@lists.wikimedia.org
Cc: wmf...@lists.wikimedia.org
Subject: [Wikitech-l] Please Welcome Andre Engels
Date: Thu, Jan 12, 2012 5:46 am


On 1/12/12, Tomasz Finc tf...@wikimedia.org wrote:
 Greetings all,

 I'm pleased to announce that Andre Engles has decided to join the Mobile
 team as our data analyst contractor. In this role he'll provide the mobile
 team with key metrics to guide our software development efforts. He'll be
 working closely with Diederik, Phil, Erick Zachte, me, and others to
 standardize, automate, and expand our growing analytics needs.

Yay, another Dutchman :) . The Dutch cabal is taking over, slowly but
surely, I tell you.

Welcome, Andre!

Roan

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


Re: [Wikitech-l] MediaWiki 1.18 learnings from a wiki admin extension writer

2012-01-12 Thread Chad
Skipping most of this, but there's a few I at least have an answer for.

On Thu, Jan 12, 2012 at 11:31 AM, Daniel Barrett d...@vistaprint.com wrote:
 1.      The global variable $action disappeared, breaking a bunch of our 
 extensions.  I switched to $wgRequest-getVal('action').


This may have been available in the global context, but as far as I know it
was never documented as a stable global that should be used.

 2.      The removal of Xml::hidden() caused one of our extensions to break.  
 I switched to Xml::input(..., array('type', 'hidden'))


Xml::hidden() was removed entirely? There *is* Html::hidden() which should be
functionally similar.

 5.      The removal of $wgMessageCache-addMessage() broke many extensions, 
 some ours and some from mediawiki.org like SimpleForms.  Some fixes just 
 required use of the i18n file. Our more difficult issue was that we were 
 injecting system messages into articles to add tracking categories. On advice 
 from this list (thanks!), we used code patterned after 
 Parser::addTrackingCategory() to inject categories and it works fine, 
 actually much better than what we had.


MessageCache::addMessages() has been on a clear deprecation path for awhile
now and should've been well-documented. Even before it was removed entirely,
it should've already been a no-op.

 8.      Our MediaWiki:common.js stopped running on the login page. I realize 
 this was a security fix; it just took me by surprise.  Fixed by writing a 
 custom extension using the hook UserLoginForm to inject the few lines of JS 
 we needed, and I'm evaluating other non-JS solutions for more security.


This hasn't changed any time recently as far as I can tell...we've had this
in place for quite awhile.

-Chad

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

Re: [Wikitech-l] MediaWiki 1.18 learnings from a wiki admin extension writer

2012-01-12 Thread Daniel Barrett
Me:
 8.      Our MediaWiki:common.js stopped running on the login page. I realize 
 this was a security fix; it just took me by surprise.  Fixed by writing a 
 custom extension using the hook UserLoginForm to inject the few lines of JS 
 we needed, and I'm evaluating other non-JS solutions for more security.

Chad writes:
This hasn't changed any time recently as far as I can tell...we've had this
in place for quite awhile.

Thanks Chad. FYI, MediaWiki:common.js definitely runs on Special:UserLogin in 
1.17.1, the immediately previous release.
DanB
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] New committer

2012-01-12 Thread DJ Bauch
Yes. The database abstraction layer is working. A bug in the MS SQL
driver prevents viewing of pages 2-x of some special pages (like Most
Wanted Templates, etc.), but page 1 of each of those comes up fine.
The problem is that scrollable cursors are being demoted to
forward-only cursors for queries using CTEs. I would expect this to
change in a future release of the drivers or I will have to come up
with a workaround. The other notable issue is that pages for oddball
characters like fleur-de-lis and happy face (single character page
names) get confused with one another. I haven't had the spare time to
figure out ways to address these yet. I have about a six-week period
in which I have another project that is consuming all of my time for
what would otherwise be working on the Azure code.

On Thu, Jan 12, 2012 at 5:16 AM, Robert Vogel vo...@hallowelt.biz wrote:
 Hi DJ and hi everybody!

 Thanks for the warm welcome :), and DJ thanks for your hints/questions 
 concerning SVN. I am quite busy at the moment so I did not set up my SVN 
 client (Tortoise) for the MW repo yet. But I hope to get it done next week.

 Afterwards I will start adding some of the announced extensions. The Azure 
 stuff is already in SVN. Markus committed it.

 I am very excited about your Azure extensions. You said you had everything 
 working with MW 1.18 except the media storage? As far as I know you work on 
 the database abstraction layer for Azure SQL. Is that right? So with the file 
 backend we should have a complete Azure bundle, shouldn't we?

 --
 Robert Vogel
 Social Web Technologien
 Softwareentwicklung
 Hallo Welt! - Medienwerkstatt GmbH


 __

 Untere Bachgasse 15
 93047 Regensburg

 Tel.  +49 (0) 941 - 66 0 80 - 198
 Fax   +49 (0) 941 - 66 0 80 - 189

 www.hallowelt.biz
 vo...@hallowelt.biz


 Sitz: Regensburg
 Amtsgericht: Regensburg
 Handelsregister: HRB 10467
 E.USt.Nr.: DE 253050833
 Geschäftsführer: Anja Ebersbach, Markus Glaser, Dr. Richard Heigl, Radovan 
 Kubani

 -Ursprüngliche Nachricht-
 Von: wikitech-l-boun...@lists.wikimedia.org 
 [mailto:wikitech-l-boun...@lists.wikimedia.org] Im Auftrag von DJ Bauch
 Gesendet: Donnerstag, 12. Januar 2012 02:15
 An: Wikimedia developers
 Betreff: Re: [Wikitech-l] New committer

 I will be trying again tonight. I got some hints from Ben Lobaugh. It
 looks like the problem I've been having is that I was using the host
 name in conjunction with my svn+ssh connection attempt, when I should
 have been using the name of my PuTTY session.

 On Wed, Jan 11, 2012 at 5:38 PM, Platonides platoni...@gmail.com wrote:
 On 11/01/12 18:25, DJ Bauch wrote:
 Welcome, Robert!
 You and I should be crossing paths. I have everything working through
 version 1.18 on Windows Azure, except for the media storage. Right
 this moment I am trying to figure out how to get my code committed,
 but I have been struggling with setting up the Subversion access from
 my Windows 7 box. Any hints or pointers on how to set things up may
 help me transition from a potential to an actual contributer.

 So, what are you trying to do? The cli or soething else?, did you read [1]?
 You will need to use plink on windows to make the ssh tunnel, pageant is
 recommended but not required (unless it's a plink with no console?).

 Take a look at [2].

 1-http://www.mediawiki.org/wiki/Commit_access#Getting_started_and_set_up
 2-http://agateau.wordpress.com/2007/07/03/windows-svnssh-and-the-subversion-command-line-client/


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

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

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

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


Re: [Wikitech-l] MediaWiki 1.18 learnings from a wiki admin extension writer

2012-01-12 Thread Chad
On Thu, Jan 12, 2012 at 11:51 AM, Daniel Barrett d...@vistaprint.com wrote:
 Me:
 8.      Our MediaWiki:common.js stopped running on the login page. I 
 realize this was a security fix; it just took me by surprise.  Fixed by 
 writing a custom extension using the hook UserLoginForm to inject the few 
 lines of JS we needed, and I'm evaluating other non-JS solutions for more 
 security.

 Chad writes:
This hasn't changed any time recently as far as I can tell...we've had this
in place for quite awhile.

 Thanks Chad. FYI, MediaWiki:common.js definitely runs on Special:UserLogin in 
 1.17.1, the immediately previous release.
 DanB


Hrm...I distinctly remember user's personal JS was disabled on that page.
I wonder if ResourceLoader by grouping the JS also ends up disabling it.
In either case, it is a security issue and there's not much we can do about
it right now.

-Chad

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

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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r108729.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108729#c29402

Commit summary for MediaWiki.r108729:

testswarm: catch exception to avoid trace :)

This happen because we do svn log on a revision that does not exist.
Returning null is fine.

Hashar's comment:

submitted as https://gerrit.wikimedia.org/r/1871

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r108727.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108727#c29403

Commit summary for MediaWiki.r108727:

testswarm: fix submitted job name

Hashar's comment:

https://gerrit.wikimedia.org/r/1871

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r108075.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108075#c29404

Commit summary for MediaWiki.r108075:

[TestSwarm] Update testswarm-mw-fetcher.php script
* Improve, add, update inline documentation
* Add ability to set TestSwarmAPI::doAddJob options from configuration file
* Change default job_name to include anchor tag to MW.org CR (fixes bug 33501)

UNTESTED - TO BE TESTED IN TESTSWARM.WMFLABS

Hashar's comment:

Deployed live with https://gerrit.wikimedia.org/r/1867

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


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

2012-01-12 Thread MediaWiki Mail
Hashar posted a comment on MediaWiki.r108725.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108725#c29405

Commit summary for MediaWiki.r108725:

update .ini following r108075

also had to rename 'username' to 'user'

Hashar's comment:

Deployed live with https://gerrit.wikimedia.org/r/1867

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz posted a comment on MediaWiki.r108488.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108488#c29406

Commit summary for MediaWiki.r108488:

r108353: Distinguish null/false in FileBackend::fileExists(). This is intended 
for things that might really care.

Aaron Schulz's comment:

Swift does, and I may change FS to do so as well.

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


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

2012-01-12 Thread MediaWiki Mail
Reedy posted a comment on MediaWiki.r108710.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108710#c29407

Commit summary for MediaWiki.r108710:

Fix fatal in r108694.

Reedy's comment:

Thanks Siebrand!

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


Re: [Wikitech-l] postgreSQL testing

2012-01-12 Thread Dan Nessett
On Thu, 12 Jan 2012 16:17:00 +0100, Antoine Musso wrote:

 Hello,
 
 I have added a new continuous integration job to check our postgres
 support.
 This is exactly the same job as MediaWiki-phpunit, only the database
 backend
 change.
 
 The link is:
https://integration.mediawiki.org/ci/job/MediaWiki-postgres-phpunit/
 
 As of now, there are two tests failing.

Excellent. Thank you for this.

-- 
-- Dan Nessett


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


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

2012-01-12 Thread MediaWiki Mail
Mglaser posted a comment on MediaWiki.r108463.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108463#c29408

Commit summary for MediaWiki.r108463:

include Windows Azure SDK for PHP into extension

Mglaser's comment:

I talked to Ben Lobaugh about this and we decided to remove the not so 
important bits of Azure SDK for PHP. This has two benefits: the lint tests 
should not break anymore and we reduced size. Done so in r108463

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


Re: [Wikitech-l] postgreSQL testing

2012-01-12 Thread Brion Vibber
On Thu, Jan 12, 2012 at 7:17 AM, Antoine Musso hashar+...@free.fr wrote:

 I have added a new continuous integration job to check our postgres
 support.
 This is exactly the same job as MediaWiki-phpunit, only the database
 backend
 change.

 The link is:
   https://integration.mediawiki.org/ci/job/MediaWiki-postgres-phpunit/


Awesome! This'll be a big help in protecting against some common
regressions.

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


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

2012-01-12 Thread MediaWiki Mail
Reedy posted a comment on MediaWiki.r108693.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108693#c29409

Commit summary for MediaWiki.r108693:

Make some things conditional for = 1.17

Reedy's comment:

Lol, yup. Fixed

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


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

2012-01-12 Thread MediaWiki Mail
Mglaser posted a comment on MediaWiki.r108465.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108465#c29410

Commit summary for MediaWiki.r108465:

Added installation instructions
Made path to SDK configurable

Mglaser's comment:

Since the Azure SDK now comes with the extension, I removed the 
$wgWindowsAzureSDKRoot settings entirely. r108737 should fix this issue.

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


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

2012-01-12 Thread MediaWiki Mail
Platonides changed the status of MediaWiki.r99909 to resolved and commented 
it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/99909#c29411

Old Status: new
 New Status: resolved

Commit summary for MediaWiki.r99909:

Followup r80375: let PreprocessorTest work on Preprocessor_Hash etc as well as 
Preprocessor_Dom

Using same technique as ApiExpandTemplates to serialize the object tree back to 
XML, rather than asking for the DOM implementation's internal XML return 
function.
Have to also perform normalization on the test cases, as they aren't normalized 
to what libxml2 serializes. :P

Note that there are 4 test failures currently with Preprocessor_Hash, as it 
makes a separate equals element around = which doesn't appear to be in 
Preprocessor_Dom's output.

Platonides's comment:

Yes, it's ok.

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


Re: [Wikitech-l] postgreSQL testing

2012-01-12 Thread Dan Nessett
On Thu, 12 Jan 2012 16:17:00 +0100, Antoine Musso wrote:

 Hello,
 
 I have added a new continuous integration job to check our postgres
 support.
 This is exactly the same job as MediaWiki-phpunit, only the database
 backend
 change.
 
 The link is:
https://integration.mediawiki.org/ci/job/MediaWiki-postgres-phpunit/
 
 As of now, there are two tests failing.

While I am grateful for the inclusion of a postgres backend in the 
integration tests, I just ran make safe on MW 108734 and got 1 error, 25 
failures and 12 incomplete tests. Any idea why the local run has 
different results that the automated run?

I have attached the most recent run output to bug 33663.

-- 
-- Dan Nessett


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


[Wikitech-l] New committer

2012-01-12 Thread Sumana Harihareswara
MrBlueSky now has core and extensions access and aims on working on
Core and extensions: small bugfixes and enhancements, adding and
updating documentation (comments), writing unit-tests. In general: small
things that need to be done.

Welcome, MrBlueSky!

-- 
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] postgreSQL testing

2012-01-12 Thread Dan Nessett
On Thu, 12 Jan 2012 17:59:09 +, Dan Nessett wrote:

 On Thu, 12 Jan 2012 16:17:00 +0100, Antoine Musso wrote:
 
 Hello,
 
 I have added a new continuous integration job to check our postgres
 support.
 This is exactly the same job as MediaWiki-phpunit, only the database
 backend
 change.
 
 The link is:
https://integration.mediawiki.org/ci/job/MediaWiki-postgres-phpunit/
 
 As of now, there are two tests failing.
 
 While I am grateful for the inclusion of a postgres backend in the
 integration tests, I just ran make safe on MW 108734 and got 1 error, 25
 failures and 12 incomplete tests. Any idea why the local run has
 different results that the automated run?
 
 I have attached the most recent run output to bug 33663.

Sorry, my mistake. I forgot to run update. After doing so, I also get 
only 2 failures.

-- 
-- Dan Nessett


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


Re: [Wikitech-l] JavaScript on Special:UserLogin?

2012-01-12 Thread Ryan Lane
 Of course, if the login form code wasn't such a swamp, there'd be a hook
 you could use to preprocess the usernames server-side...  :-(


The getCanonicalName function in the auth plugins will do this. I was
actually thinking of adding a hook to LdapAuthentication in that
function to let users munge the name however they'd like before
LdapAuthentication munges it further.

- Ryan

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


Re: [Wikitech-l] postgreSQL testing

2012-01-12 Thread Antoine Musso
Dan Nessett wrote:

 Sorry, my mistake. I forgot to run update. After doing so, I also get
 only 2 failures.

Good to know your previous issues were solved :-) Now we need people
to correct the last 2 ones :b

-- 
Antoine hashar Musso


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


Re: [Wikitech-l] proposed tech conference anti-harassment policy

2012-01-12 Thread Trevor Parscal
What if you need more attention and want to be harassed?

Amir: Also, SVN does kinda suck :P

Seriously though, this looks great.

I've been fortunate enough to not have been harassed at a conference much
more than being trolled on occasion, but I think that's a more borderline
case that doesn't quite warrant any serious actions.

That said, I think there's an excuse game that gets played a lot. When I've
expressed concern in the past I've often been told things like like yeah,
but they are a really good contributor, that's just how they are, deal with
it or they're [enter country of origin here], that's just how they are,
deal with it.

This lays some ground rules that are fair in any venue around the world.
Thanks for putting this together.

- Trevor

On Thu, Jan 12, 2012 at 8:35 AM, Amir E. Aharoni 
amir.ahar...@mail.huji.ac.il wrote:

 2012/1/12 Chad innocentkil...@gmail.com
 
  On Thu, Jan 12, 2012 at 11:04 AM, David Gerard dger...@gmail.com
 wrote:
   On 12 January 2012 16:00, Sumana Harihareswara suma...@wikimedia.org
 wrote:
  
   The Wikimedia Foundation is dedicated to a harassment-free conference
   experience for everyone.  I'm proposing a fairly short and standard
   anti-harassment policy of the type that's becoming best practice for
   tech conferences and hackathons.
   Draft: https://www.mediawiki.org/wiki/User:Sumanah/AHP
  
  
   Nice one :-) A candidate for WMF and wider Wikimedia events in
 general, too.
  
 
  I don't see anything preventing harassment over choice of DBMS ;-)

 In all seriousness, i was harassed more than once IRL and online for
 saying that SVN doesn't suck as much as die-hard Git fans say it does.
 This policy can say preferred technology in addition to Creative
 Commons license.

 --
 Amir Elisha Aharoni · אָמִיר אֱלִישָׁע אַהֲרוֹנִי
 http://aharoni.wordpress.com
 ‪“We're living in pieces,
 I want to live in peace.” – T. Moore‬

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

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

Re: [Wikitech-l] New committer

2012-01-12 Thread Antoine Musso
Sumana Harihareswara wrote:

 MrBlueSky now has core and extensions access and aims on working on
 Core and extensions: small bugfixes and enhancements, adding and
 updating documentation (comments), writing unit-tests. In general: small
 things that need to be done.

 Welcome, MrBlueSky!

Watch out for MissWhiteCloud :-b

Welcome!


-- 
Antoine hashar Musso


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


Re: [Wikitech-l] [Wmfall] Please Welcome Andre Engels

2012-01-12 Thread Erik Moeller
On Thu, Jan 12, 2012 at 3:22 AM, Tomasz Finc tf...@wikimedia.org wrote:
 I'm pleased to announce that Andre Engles has decided to join the Mobile
 team as our data analyst contractor. In this role he'll provide the mobile
 team with key metrics to guide our software development efforts. He'll be
 working closely with Diederik, Phil, Erick Zachte, me, and others to
 standardize, automate, and expand our growing analytics needs.

Welcome to the WMF team, Andre! Great to have your help. :-)

All best,
Erik
-- 
Erik Möller
VP of Engineering and Product Development, Wikimedia Foundation

Support Free Knowledge: http://wikimediafoundation.org/wiki/Donate

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


Re: [Wikitech-l] Please Welcome Andre Engels

2012-01-12 Thread James Forrester
On 12 January 2012 11:22, Tomasz Finc tf...@wikimedia.org wrote:
 Greetings all,

 I'm pleased to announce that Andre Engles has decided to join the Mobile
 team as our data analyst contractor. In this role he'll provide the mobile
 team with key metrics to guide our software development efforts. He'll be
 working closely with Diederik, Phil, Erick Zachte, me, and others to
 standardize, automate, and expand our growing analytics needs.

Fantastic news, congratulations Andre (or perhaps, congratulations WMF)!

J.
-- 
James D. Forrester
jdforres...@wikimedia.org | jdforres...@gmail.com
[[Wikipedia:User:Jdforrester|James F.]]

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


Re: [Wikitech-l] Small-ish gadget project: Bug update gadget

2012-01-12 Thread Rob Moen
Very cool.  Nice work Mark!

On Wed, Jan 11, 2012 at 9:10 PM, Mark A. Hershberger 
mhershber...@wikimedia.org wrote:

 mhershber...@wikimedia.org (Mark A. Hershberger) writes:

  Rob Moen rm...@wikimedia.org writes:
 
  I fiddled with the request parameters for a while trying to get the JSON
  RPC to not throw errors. I settled with a URL request for now which is
  working nicely.
  Feel free to shoot me ideas / thoughts on how this could be improved.
 
  I can think of all sorts of ways to improve this.  I may take a crack
  at it soon.

 You can see my improvements here:
 http://labs.wikimedia.deployment.wmflabs.org/wiki/Problem_reports

 (Following up here even though I mentioned it in another email because
 that one may have be TL;DR for some people.)

 Mark.




-- 
Rob Moen
Wikimedia Foundation
rm...@wikimedia.org
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


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

2012-01-12 Thread MediaWiki Mail
Nikerabbit posted a comment on MediaWiki.r108727.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108727#c29412

Commit summary for MediaWiki.r108727:

testswarm: fix submitted job name

Nikerabbit's comment:

//mediawiki.org will not work btw, it will aways redirect to 
http://www.mediawiki.org

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


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

2012-01-12 Thread MediaWiki Mail
GWicke changed the status of MediaWiki.r106642 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106642

Old status:  new
 New status: ok

Commit summary for MediaWiki.r106642:

ParserFunctions: made {{#time}} respect parser's timestamp, just as 
{{CURRENTDATE}} and friends do

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


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

2012-01-12 Thread MediaWiki Mail
GWicke changed the status of MediaWiki.r105459 to ok and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/105459#c29413

Old Status: new
 New Status: ok

Commit summary for MediaWiki.r105459:

(bug 32351) Make #time[l] support explicitly specified timezones. Patch by  Van 
de Bugger.

This would make something like {{#time:H:i:s| 9:30 January 1, 2012 MST}} 
convert the time from 9:30 mountatin standard time to whatever it is in UTC.

I made one minor change from the patch on bugzilla in changing an @ to a 
wfSuppressWarnings. (The @ was already in the code, it wasn't introduced by the 
patch).

GWicke's comment:

Minor nitpick: s/default inpu /default input /. Ran the parser tests, but did 
not test with PHP  5.2.

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


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

2012-01-12 Thread MediaWiki Mail
Nikerabbit changed the status of MediaWiki.r108734 to ok and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108734#c29414

Old Status: new
 New Status: ok

Commit summary for MediaWiki.r108734:

Consistency: made all API examples a verbs

Nikerabbit's comment:

Both form as verbs, in fact.

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


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

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

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108736:

Simplify r108693

Also fix stupid logic error noticed by Niklas

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz posted a comment on MediaWiki.r108743.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108743#c29416

Commit summary for MediaWiki.r108743:

Kill * @return void

Aaron Schulz's comment:

Whhhy? It distinguishes a lack of documentation with someone deliberately 
saying that nothing should be returned?

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz changed the status of MediaWiki.r99840 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/99840

Old status:  new
 New status: ok

Commit summary for MediaWiki.r99840:

Followup to r99653 (bug 31643) -- reenable client-side thumbnailing of SVGs on 
Special:Upload, with workaround for Firefox 7 regression.

Upstream firefox bug for hang loading some SVGs via data URI: 
https://bugzilla.mozilla.org/show_bug.cgi?id=694165
A fix may land as soon as Firefox 8.

This rev works around the bug by using window.URL.createObjectURL() instead of 
loading a data URI via FileReader; the object URL loads more cleanly and 
doesn't have the same hanging bug.

Needs to be replicated to UploadWizard (or moved to a shared lib!), some other 
fixes coming so no rush.

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz changed the status of MediaWiki.r89706 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89706

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r89706:

Reinstate r79122 (fix for bug 14404), reverting r83868. The real bug seem to 
have been r86131, fixed in r88902 (1.17) and  r88902 (1.18).
This is not merged with the r86131 change to Article::getParserOptions() since 
I don't see the point for the new function yet.
Reenabled its test ArticleTablesTest which was disabled in r85618

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


Re: [Wikitech-l] postgreSQL testing

2012-01-12 Thread OQ
On Thu, Jan 12, 2012 at 12:46 PM, Antoine Musso hashar+...@free.fr wrote:
 Dan Nessett wrote:

 Sorry, my mistake. I forgot to run update. After doing so, I also get
 only 2 failures.

 Good to know your previous issues were solved :-) Now we need people
 to correct the last 2 ones :b

One should be fixed[1], not sure on why the JobQueue related issue is
still broken. Only reason I can think of is it's not pulling from the
right table, or the job isn't being inserted correctly.

1 - https://integration.mediawiki.org/ci/job/MediaWiki-postgres-phpunit/7/

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz changed the status of MediaWiki.r108743 to reverted
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108743

Old status:  new
 New status: reverted

Commit summary for MediaWiki.r108743:

Kill * @return void

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


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

2012-01-12 Thread MediaWiki Mail
Reedy posted a comment on MediaWiki.r108745.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108745#c29417

Commit summary for MediaWiki.r108745:

Reverted r108743 per CR comment. This should at least be discussed first.

Reedy's comment:

And everytime I have (accidentally!) added @return void in doing much 
documentation stuff, I've always been told to remove it

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz posted a comment on MediaWiki.r108745.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108745#c29418

Commit summary for MediaWiki.r108745:

Reverted r108743 per CR comment. This should at least be discussed first.

Aaron Schulz's comment:

By whom?

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


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

2012-01-12 Thread MediaWiki Mail
Aaron Schulz changed the status of MediaWiki.r102112 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102112

Old status:  new
 New status: ok

Commit summary for MediaWiki.r102112:

* Made OuputPage::showPermissionsErrorPage() show a different messages for 
'read', 'edit', 'create' and 'upload' actions to saying You need to log in to 
do this action when 1) The user is not logged in 2) The only error is a 
permissions error (no block or something else) and 3) The error can simply be 
avoided by logging in
* This replaces OuputPage::loginToUse() functionnality, made it simply throw a 
PermissionsEror exception and updated all calls in core
* Same for the check in SpecialUpload::execute(), 
EditPage::userNotLoggedInPage() and EditPage::noCreatePermission()
* Throw the same exception in EditPage::attemptSave() whether the user is 
logged in or not and let OuputPage::showPermissionsErrorPage() decide which 
message to display
* Replaced call to deprecated OutputPage::blockedPage() in SpecialUpload
* Displayed messages are the same as now, except the title is always 
loginreqtitle
* 'nocreatetitle' and 'uploadnologin' messages are still used by extensions, so 
I kept them, but the message 'whitelistedittitle' is not used anymore and has 
been removed

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108746 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108746

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108746:

Minor type cleanups

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108744 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108744

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108744:

* Fixed a bit of bogus code left in r108740
* Added some class comments

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108742 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108742

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108742:

Unindent code to use elseif

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108741 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108741

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108741:

Documentation

Wrap some long lines

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108704 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108704

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108704:

added basic file handler to script and added a new arg to specify output file

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108705 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108705

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108705:

changed the day ranges for edit count periods

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108712 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108712

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108712:

bug 33583 search ns user pref ignored!

r106780 to fix bug 33270 introduced a new bug that prevented selected
namespaces for search to be applied. This patch fix the issue.

Credits to Brad Jorsch

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


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

2012-01-12 Thread MediaWiki Mail
Reedy posted a comment on MediaWiki.r108745.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108745#c29419

Commit summary for MediaWiki.r108745:

Reverted r108743 per CR comment. This should at least be discussed first.

Reedy's comment:

I wonder if I'm confusing it with @static...

Just gone back through my contribs, and can't see anything suggesting not to 
add @return void


HMM

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108720 to resolved
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108720

Old status:  new
 New status: resolved

Commit summary for MediaWiki.r108720:

Follow-up r108549: {{PLURAL}}

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108725 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108725

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108725:

update .ini following r108075

also had to rename 'username' to 'user'

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


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

2012-01-12 Thread MediaWiki Mail
Preilly changed the status of MediaWiki.r108726 to ok
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/108726

Old status:  new
 New status: ok

Commit summary for MediaWiki.r108726:

swap assertEquals parameters in TemplateCategoriesTest

First parameter is expected, second is the result

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


  1   2   >