Re: [Wikitech-l] IE6

2011-06-04 Thread Ashar Voultoiz
On 04/06/11 02:07, Mono mium wrote:
> I still support.

On a multiple millions visitors per day website, this is unacceptable.

Most users do not even know what a 'browser' is. They just look at their 
computer desktop and double click the 'e' icon.

Plus, 'internet explorer' sounds much better than:
  - 'Firefox' : will it light my computer?
  - 'Opera' : I listen to R'n'B

I do support your crash method... on your personal website.

-- 
Ashar Voultoiz


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


Re: [Wikitech-l] Code review process (was: Status of more regular code deployments)

2011-06-04 Thread Ashar Voultoiz
On 04/06/11 00:33, Rob Lanphier wrote:
> Here's an immediate challenge we face: in order to release 1.18, we
> need to get the red line in this graph to zero:
> http://toolserver.org/~robla/crstats/crstats.118all.html

It is a thin red line.
Lets just pretend we are in the Crimean Peninsula.


-- 
Ashar Voultoiz
Would let users figures it out by searching their favourite encyclopaedia.


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


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

2011-06-04 Thread MediaWiki Mail
User "MaxSem" changed the status of MediaWiki.r89387.

Old Status: new
New Status: ok

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

Fix for r89166: need self

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


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

2011-06-04 Thread MediaWiki Mail
User "Hashar" changed the status of MediaWiki.r89451.

Old Status: new
New Status: ok

User "Hashar" also posted a comment on MediaWiki.r89451.

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

Followup r88124: fix the most glaring bug of the week

Comment:

\o/

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


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

2011-06-04 Thread MediaWiki Mail
User "Hashar" changed the status of MediaWiki.r88124.

Old Status: fixme
New Status: ok

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

Add Rotem Liss and Platonides to author list.

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


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

2011-06-04 Thread MediaWiki Mail
User "Reedy" changed the status of MediaWiki.r89455.

Old Status: new
New Status: ok

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

svn:eol-style native

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


Re: [Wikitech-l] On the IE6 query string security check breakage (bug 28235)

2011-06-04 Thread Tim Starling
On 04/06/11 03:53, Brion Vibber wrote:
> On Thu, Jun 2, 2011 at 5:21 PM, Tim Starling wrote:
>> The main issue here is that we don't a wide variety of web servers set
>> up for testing. We know that Apache lets you detect %2E versus dot via
>> $_SERVER['REQUEST_URI'], but we don't know if any other web servers do
>> that.
>>
>> Note that checking for %2E alone is not sufficient, a lot of
>> installations (including Wikimedia) have an alias /wiki ->
>> /w/index.php which can be used to exploit action=raw.
>>
> 
> Well that should be fine; as long as we can see the "/wiki?/foo.bat" then we
> can identify that it doesn't contain an unencoded dot in the path.
> 
> It sounds like simply checking REQUEST_URI when available would eliminate a
> huge portion of our false positives that affect real-world situations.
> Apache is still the default web server in most situations for most folks,
> and of course runs our own production servers.

You mean by checking $_SERVER["SERVER_SOFTWARE"] or something to check
if it's Apache that we're running under? I suppose that could work.

It's easy enough to find out if REQUEST_URI is available. What we
don't know is whether REQUEST_URI is really what was sent to the
server, or whether it has %2E converted to "." before PHP gets to see it.

>>> Are there any additional exploit vectors for API output other than HTML
>> tags
>>> mixed unescaped into JSON?
>>
>> Yes, all other content types, as I said above.
>>
> 
> Only as drive-by downloads, or as things that execute without interaction?

Presumably that depends on what plugins are registered. I think it's
better to avoid taking risks like this unless there is some good
reason for doing so. With a REQUEST_URI check in place, in addition to
all the other mitigating measures we now have in place, overblocking
should be vanishingly rare.

-- Tim Starling


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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89359.

Old Status: new
New Status: ok

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

(bug 29117) A string is missing when the UI is in Hebrew

Backport r88658

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89420.

Old Status: new
New Status: ok

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

Extension to r89419, removing a bugous CSS code

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89401.

Old Status: new
New Status: ok

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

(bug 29121) jquery.suggestions uses jquery.autoEllipsis but didn't have it in 
its dependency list, causing JS errors in non-Vector skins

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


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

2011-06-04 Thread MediaWiki Mail
User "Happy-melon" posted a comment on MediaWiki.r89406.

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

Start unpicking r85288 (magic __get() accessor for RequestContext).  Instead, 
bring back some of r86872 (abstract base class for classes providing access to 
RequestContext methods), which is a more 'classical' solution.

Comment:

Grr, my error_reporting = E_ALL|E_STRICT) config is only in php.ini 
for ''some'' of my nine versions of PHP... /sulks

I'm afraid uni finals are a higher priority for me right now than redoing this, 
or otherwise reworking r85288.  If people agree with the general principle of 
using a ContextSource base class, do go ahead and stick it back in.  Otherwise 
I'll try and get back round to it sometime towards the end of June.

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r87164.

Old Status: new
New Status: fixme

User "Krinkle" also posted a comment on MediaWiki.r87164.

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

Recommit r87129 and follow-ups but with a fix for the bug Brion found (sorry)

Comment:


+   if ( isset( $row->user_name ) ) {
+   $this->mName = $row->user_name;
..
+   }
+
+   if ( isset( $row->user_name ) ) {
+   $this->mRealName = $row->user_real_name;
..
+   }
+
if ( isset( $row->user_id ) ) {
$this->mId = intval( $row->user_id );


The second one should be checking user_'''real'''_name instead of 
user_name

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r87164.

Old Status: fixme
New Status: new

User "Krinkle" also posted a comment on MediaWiki.r87164.

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

Recommit r87129 and follow-ups but with a fix for the bug Brion found (sorry)

Comment:

I fixed it myself for now since it was causing issues in another project I'm 
working on.

r89464. Setting status back to new.

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


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

2011-06-04 Thread MediaWiki Mail
User "Huji" posted a comment on MediaWiki.r89419.

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

(bug 29263) Add LTR class to the shared CSS to be used for left-to-right text 
such as SQL queries shown in dberrortext and similiar messages in RTL 
environements

Comment:

I made ltr a generic reusable class style with r89466.

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


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

2011-06-04 Thread MediaWiki Mail
User "Nikerabbit" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

Notice: Use of RequestContext::__get() is deprecated; use $context->getFoo() 
instead is deprecated. [Called from StubUserLang::_newObject in 
/www/sandwiki/includes/StubObject.php at line 151] in 
/www/sandwiki/includes/GlobalFunctions.php on line 3342
 
Notice: Use of RequestContext::__set() is deprecated; use $context->setFoo() 
instead is deprecated. [Called from SpecialPageFactory::executePath in 
/www/sandwiki/includes/SpecialPageFactory.php at line 447] in 
/www/sandwiki/includes/GlobalFunctions.php on line 3342

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89421.

Old Status: new
New Status: ok

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

Extension to r89419, I was too excited fixing the bug

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89466.

Old Status: new
New Status: ok

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

Follow up to r89419:
Per Brion's comment on Code Review, a generically named ltr class is created 
and used to reduce the number of rules

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89419.

Old Status: fixme
New Status: new

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

(bug 29263) Add LTR class to the shared CSS to be used for left-to-right text 
such as SQL queries shown in dberrortext and similiar messages in RTL 
environements

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89440.

Old Status: new
New Status: ok

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

Localisation updates for ToolserverI18N messages from translatewiki.net 
(2011-06-03 20:33:00 UTC)

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89459.

Old Status: new
New Status: fixme

User "Krinkle" also posted a comment on MediaWiki.r89459.

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

Add Jarry1250's Translate tool

Possibly completely wrong, I've no idea

Comment:

This broke the TsIntuition sandbox with an E_FATAL.

Where are the comma's after the array values ?

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89459.

Old Status: fixme
New Status: ok

User "Krinkle" also posted a comment on MediaWiki.r89459.

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

Add Jarry1250's Translate tool

Possibly completely wrong, I've no idea

Comment:

Also, altho non-breaking, in order to be shown in the list of tools on the 
[http://toolserver.org/~intuition/#tab-about Dashboard], a 'title' message must 
be present.

Added in r89467.

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89460.

Old Status: new
New Status: ok

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

Add minutes/seconds

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


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

2011-06-04 Thread MediaWiki Mail
User "IAlex" posted a comment on MediaWiki.r89464.

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

Fixing fixme in r87164 (wrong if check)

Comment:

Thanks for the fix!

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" posted a comment on MediaWiki.r89470.

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

TsIntuition fixes
* Re-alphabetize order Domains.php (r89459)
* Synchronize Optional and Ignore marks between comments after ['en'], 
templates in ['qqq'] and Translate/toolserver-domains.txt
* Add Templatecount to translation routine on TranslateWiki

Comment:

The rest of this commit is in r89469.

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89402.

Old Status: new
New Status: ok

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

(bug 29104) Fix r86622: make splitlines default to false, makes much more sense

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


[Wikitech-l] Showing stub links by default - is it possible in a Wikimedia project?

2011-06-04 Thread Tisza Gergö
Hi all,

MediaWiki has a user setting to add a CSS class to article links whose length 
is 
below a certain threshold (preferences/appearance/advanced options/threshold 
for 
stub link formatting). Is it possible to enable this by default on a Wikimedia 
wiki? Some of the advantages:
- a lot of editors seem to use it already to find articles which need some 
care, 
and currently it slows down page loading for them because it breaks caching, so 
they could work quicker.
- like red links, it could bring in new editors by making readers realize that 
some of the linked articles need help.
- it could make readers waste less time because they could realize the article 
is a stub without clicking at it.
On the other hand, I suppose it would raise database load because the length of 
all linked pages would need to be queried at HTML generation. (Or is that query 
already already necessary to see which links are red?)



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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" posted a comment on MediaWiki.r89461.

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

* (bug 29265) Mediawiki API documentation incomplete

Flesh out documentation for the maxlag parameter

Comment:

Hm.. after this the $host: $ part is bold and italic. Intentional ?

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89419.

Old Status: new
New Status: ok

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

(bug 29263) Add LTR class to the shared CSS to be used for left-to-right text 
such as SQL queries shown in dberrortext and similiar messages in RTL 
environements

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


Re: [Wikitech-l] [Foundation-l] YouTube and Creative Commons

2011-06-04 Thread Michael Dale
Comments inline:

On Fri, Jun 3, 2011 at 4:51 PM, Brion Vibber  wrote:

> (I'm not sure offhand if I'm set up to cross-post to Foundation-l; if this
> doesn't make it, somebody please CC a mention if necessary. Thanks!)
>
> On Fri, Jun 3, 2011 at 4:42 PM, aude  wrote:
>
> > Aside from the very real privacy issue, YouTube videos can disappear at
> any
> > time.  I would much rather we host them on Commons.
> >
> > A youtube2commons script is pretty easy to implement,
>


yes a basic youtube2commons script was posted by Jan on wikivideo-l list
recently:
http://lists.wikimedia.org/pipermail/wikivideo-l/2011-May/56.html But as
you point out we really need to work on increasing the upload size limit.



>
> There's been some ongoing work on TimedMediaHandler extension which will
> replace the older OggHandler
>


Yes, been hammering away on associated bugs. People can help by testing and
filing bugs :) thedj has helped file a lot of bugs, and Brion too recently
has been taking a look at the transcoding side of things and Roan did a good
first pass review and thous suggestions have since been integrated.  I hope
to have a new version up prototype soon that integrates all the known
requested features / bugs listed in bugzilla some time next week. (with the
exception of features tagged for version 1.1 like server side srt parsing
and timed wikitext -> html -> srt text with html tag removal )  Once I get
this update out to prototype I will try and do a blog post at that point to
invite people to put test it out.



> Basic uploads by URL work in theory, but I'm not sure the deployment
> status.
> Background large-file downloads are currently disabled in the latest code
> and needs to be reimplemented if that's to be used.
>


Yea we have bug:
https://bugzilla.wikimedia.org/show_bug.cgi?id=20512tracking
re-enabling copy by url. Once we have webm TMH deployed it would
make for simple youtube cc content with importing without conversion :)


>
> For straight uploads, regular uploads of large files are a bit problematic
> in general (they hit memory limits and such and have to make it through
> caching proxies and whatnot), but there's also been some new work on
> improved chunked uploads for FireFogg (and perhaps for general modern
> browsers that can do fancier uploads). Michael Dale can probably give some
> updates on this, but it'll be a bit yet before it's ready to go.
>

Yes we are reimplementing the firefogg chunk uploading as ResumableUpload (
name of new extension ) in a way that allows both HTML5 XHR browsers to use
the chunk protocol in addition to firefogg ( if your converting video from a
proprietary source ). In addition we had disscutions at the Berlin
Hack-a-ton that cleared up some confusion about the concerns with the
firefogg protocol and modified it to explicitly state the byte ranges of
chunks in requests and server responses. Also had a brief chat with Russell
on IRC, so that we can support this append chunks system as we move to the
swiftMedia back end.

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


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

2011-06-04 Thread MediaWiki Mail
User "Reedy" posted a comment on MediaWiki.r89461.

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

* (bug 29265) Mediawiki API documentation incomplete

Flesh out documentation for the maxlag parameter

Comment:

Nope, but the formatting does it itself for whatever reason.

If you do api.php?version you will see it does the same thing there

No big deal

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89461.

Old Status: new
New Status: ok

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

* (bug 29265) Mediawiki API documentation incomplete

Flesh out documentation for the maxlag parameter

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


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

2011-06-04 Thread MediaWiki Mail
User "Solitarius" posted a comment on MediaWiki.r88301.

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

Added UTF-8 encoding for data retrieved from DB

Comment:

In my MSSQLBackCompat external databases, nvarchar() columns are already 
encoded in utf-8. This patch cause a double encoding (example: "répertoire").


Other points to take into consideration:
* In my apache configuration, I have php_value mssql.charset 
"UTF-8".
* Are other Database (Oracle, IBM DB2, ...) classes using a UTF-8 charset?


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


Re: [Wikitech-l] [Foundation-l] YouTube and Creative Commons

2011-06-04 Thread emijrp
A nice script to download YouTube videos is youtube-dl[1]. Link that with a
flv/mp4 -> ogg converter and an uploader to Commons is trivial.

[1] http://rg3.github.com/youtube-dl/

2011/6/4 Michael Dale 

> Comments inline:
>
> On Fri, Jun 3, 2011 at 4:51 PM, Brion Vibber  wrote:
>
> > (I'm not sure offhand if I'm set up to cross-post to Foundation-l; if
> this
> > doesn't make it, somebody please CC a mention if necessary. Thanks!)
> >
> > On Fri, Jun 3, 2011 at 4:42 PM, aude  wrote:
> >
> > > Aside from the very real privacy issue, YouTube videos can disappear at
> > any
> > > time.  I would much rather we host them on Commons.
> > >
> > > A youtube2commons script is pretty easy to implement,
> >
>
>
> yes a basic youtube2commons script was posted by Jan on wikivideo-l list
> recently:
> http://lists.wikimedia.org/pipermail/wikivideo-l/2011-May/56.html But
> as
> you point out we really need to work on increasing the upload size limit.
>
>
>
> >
> > There's been some ongoing work on TimedMediaHandler extension which will
> > replace the older OggHandler
> >
>
>
> Yes, been hammering away on associated bugs. People can help by testing and
> filing bugs :) thedj has helped file a lot of bugs, and Brion too recently
> has been taking a look at the transcoding side of things and Roan did a
> good
> first pass review and thous suggestions have since been integrated.  I hope
> to have a new version up prototype soon that integrates all the known
> requested features / bugs listed in bugzilla some time next week. (with the
> exception of features tagged for version 1.1 like server side srt parsing
> and timed wikitext -> html -> srt text with html tag removal )  Once I get
> this update out to prototype I will try and do a blog post at that point to
> invite people to put test it out.
>
>
>
> > Basic uploads by URL work in theory, but I'm not sure the deployment
> > status.
> > Background large-file downloads are currently disabled in the latest code
> > and needs to be reimplemented if that's to be used.
> >
>
>
> Yea we have bug:
> https://bugzilla.wikimedia.org/show_bug.cgi?id=20512tracking
> re-enabling copy by url. Once we have webm TMH deployed it would
> make for simple youtube cc content with importing without conversion :)
>
>
> >
> > For straight uploads, regular uploads of large files are a bit
> problematic
> > in general (they hit memory limits and such and have to make it through
> > caching proxies and whatnot), but there's also been some new work on
> > improved chunked uploads for FireFogg (and perhaps for general modern
> > browsers that can do fancier uploads). Michael Dale can probably give
> some
> > updates on this, but it'll be a bit yet before it's ready to go.
> >
>
> Yes we are reimplementing the firefogg chunk uploading as ResumableUpload (
> name of new extension ) in a way that allows both HTML5 XHR browsers to use
> the chunk protocol in addition to firefogg ( if your converting video from
> a
> proprietary source ). In addition we had disscutions at the Berlin
> Hack-a-ton that cleared up some confusion about the concerns with the
> firefogg protocol and modified it to explicitly state the byte ranges of
> chunks in requests and server responses. Also had a brief chat with Russell
> on IRC, so that we can support this append chunks system as we move to the
> swiftMedia back end.
>
> --michael
> ___
> 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


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

2011-06-04 Thread MediaWiki Mail
User "Platonides" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

We can't really change the way the hook parameters are passed. We should revert 
this change.

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


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

2011-06-04 Thread MediaWiki Mail
User "Platonides" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

The point is, StubObject::_newObject is doing RequestContext::getMain()->lang; 
so it shouldn't have triggered __get(). Either php or our expectations are 
wrong.

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


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

2011-06-04 Thread MediaWiki Mail
User "^demon" changed the status of MediaWiki.r89292.

Old Status: new
New Status: ok

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

Tweak HttpTest following r89278:
* remove FTP test case URL (shouldn't actually work, and as a directory 
listing, not super stable)
* remove old test case URLs that are now 30x redirects (example.com and a 
toolserver link), which broke the comparison tests
* added in http://en.wikipedia.org/robots.txt so we've got a nice basic fetch 
on one of our servers too

Now this test class is down to 3 failures, in URL & domain validation stuff

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


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

2011-06-04 Thread MediaWiki Mail
User "Happy-melon" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

Hooks can still do things like $user->setSomething() and that will 
stick; it's just that they can't do $user = $otherUser.  Which as 
Brion says is not kosher anyway.  Changing it to be truly read-only would 
indeed not be feasible; but changing from pass-by-reference to pass-by-object 
is ok, IMO.

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


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

2011-06-04 Thread MediaWiki Mail
User "Happy-melon" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

Yes it should; RequestContext::getMain()->lang is equivalent to 
$context->lang; the __get() magic accessor is to rewrite that to 
$context->getLang().

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


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

2011-06-04 Thread MediaWiki Mail
User "Reedy" changed the status of MediaWiki.r89472.

Old Status: new
New Status: ok

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

Adding my USERINFO file.

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


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

2011-06-04 Thread MediaWiki Mail
User "Reedy" posted a comment on MediaWiki.r89472.

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

Adding my USERINFO file.

Comment:

Yay. Tied your SVN account in with your MW account :)

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


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

2011-06-04 Thread MediaWiki Mail
User "Reedy" posted a comment on MediaWiki.r89471.

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

Add BlockTest to the tests autoloader. Needed by ApiBlockTest.php
Added tabs to other items (compare ignoring whitespace).

Comment:

Have you purposely seemingly extra indented it?

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


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

2011-06-04 Thread MediaWiki Mail
User "Platonides" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

Oops. You are right. I was looking just at the RequestContext::getMain() Fixed 
that at r89474.

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


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

2011-06-04 Thread MediaWiki Mail
User "Platonides" posted a comment on MediaWiki.r89408.

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

More unpicking of r85288.  I think this is all of the magic method calls, but 
they're very hard to grep for (part of the problem with them!), so let's leave 
the calls in with a wfDeprecated() for a while...

Comment:

Can they, or will old extensions hooking it fail to run because they were 
expecting a reference and it is now given by value?

That's the reason we didn't fix all prototypes everywhere long time ago.

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


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

2011-06-04 Thread MediaWiki Mail
User "Reedy" changed the status of MediaWiki.r89474.

Old Status: new
New Status: fixme

User "Reedy" also posted a comment on MediaWiki.r89474.

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

Follow up r89408. Magic >lang to getLang().

Comment:

 Platonides: you missed the ()

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


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

2011-06-04 Thread MediaWiki Mail
User "IAlex" changed the status of MediaWiki.r89474.

Old Status: fixme
New Status: new

User "IAlex" also posted a comment on MediaWiki.r89474.

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

Follow up r89408. Magic >lang to getLang().

Comment:

Fixed in r89477.

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


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

2011-06-04 Thread MediaWiki Mail
User "MaxSem" changed the status of MediaWiki.r89479.

Old Status: new
New Status: fixme

User "MaxSem" also posted a comment on MediaWiki.r89479.

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

Also support macports

Comment:

Did you mean has_binary port?

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


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

2011-06-04 Thread MediaWiki Mail
User "MaxSem" changed the status of MediaWiki.r89479.

Old Status: fixme
New Status: resolved

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

Also support macports

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


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

2011-06-04 Thread MediaWiki Mail
User "MaxSem" changed the status of MediaWiki.r89254.

Old Status: new
New Status: reverted

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

* trying to create a way to update/install extension already included in the 
LocalSettings.php
* wgHooks and wgAutoloadClasses are initialized so they can be appended within 
extension inits

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" changed the status of MediaWiki.r89481.

Old Status: new
New Status: ok

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

Revert r89254:
* It breaks if any extension uses any global except $IP, $wgHooks or 
$wgAutoloadClasses. Some unit tests that try updating a virtual database are 
already affected.
* If we have LocalSettings.php, we already have it loaded - are there code 
paths that don't have it?
* Parsing PHP with regexes is fragile and scary

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


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

2011-06-04 Thread MediaWiki Mail
User "MaxSem" changed the status of MediaWiki.r89481.

Old Status: ok
New Status: new

User "MaxSem" also posted a comment on MediaWiki.r89481.

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

Revert r89254:
* It breaks if any extension uses any global except $IP, $wgHooks or 
$wgAutoloadClasses. Some unit tests that try updating a virtual database are 
already affected.
* If we have LocalSettings.php, we already have it loaded - are there code 
paths that don't have it?
* Parsing PHP with regexes is fragile and scary

Comment:

Another great way to break stuff I forgot to mention:

/* these extensions are broken -- don't reenable
require_once( "$IP/extensions/BreakYourSite/BreakYourSite.php" );
require_once( "$IP/extensions/WorksOnlyOnMW_1.1/WorksOnlyOnMW_1.1.php" );
*/


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


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

2011-06-04 Thread MediaWiki Mail
User "^demon" changed the status of MediaWiki.r89481.

Old Status: new
New Status: ok

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

Revert r89254:
* It breaks if any extension uses any global except $IP, $wgHooks or 
$wgAutoloadClasses. Some unit tests that try updating a virtual database are 
already affected.
* If we have LocalSettings.php, we already have it loaded - are there code 
paths that don't have it?
* Parsing PHP with regexes is fragile and scary

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


Re: [Wikitech-l] Showing stub links by default - is it possible in a Wikimedia project?

2011-06-04 Thread Platonides
Tisza Gergö wrote:
> Hi all,
>
> MediaWiki has a user setting to add a CSS class to article links whose length 
> is
> below a certain threshold (preferences/appearance/advanced options/threshold 
> for
> stub link formatting). Is it possible to enable this by default on a Wikimedia
> wiki?

No.
Currently it would mean not caching any page view.
The feature would need to be adapted to allow efficient stub linking
(I have some ideas about it, and the new linker makes things easier).

Still, we might not allow stub links for anons in miser mode.

Feel free to open a bug.


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


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

2011-06-04 Thread MediaWiki Mail
User "^demon" posted a comment on MediaWiki.r89475.

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

Added missing $PAGEEDITTIME replacement for impersonal mails

Comment:

Every time I see this I cry a little bit inside.

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


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

2011-06-04 Thread MediaWiki Mail
User "Krinkle" posted a comment on MediaWiki.r89476.

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

Fix for r89475: let's make this correctly

Comment:

Is $timecorrection not needed anymore ?

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


Re: [Wikitech-l] [Foundation-l] YouTube and Creative Commons

2011-06-04 Thread David Gerard
On 4 June 2011 17:47, Michael Dale  wrote:
> On Fri, Jun 3, 2011 at 4:51 PM, Brion Vibber  wrote:

>> There's been some ongoing work on TimedMediaHandler extension which will
>> replace the older OggHandler

> Yes, been hammering away on associated bugs. People can help by testing and
> filing bugs :)


A question that wasn't clear from reading the bug: why is reading a
file format (WebM) blocked on the entire Timed Media Handler?


- d.

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


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

2011-06-04 Thread MediaWiki Mail
User "IAlex" posted a comment on MediaWiki.r89476.

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

Fix for r89475: let's make this correctly

Comment:

This was a copy/paste error. We are in sendImpersonal() here, so we need to use 
server time. $timecorrection is only defined in sendPersonalised().

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