Re: [Wikitech-l] IE6
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)
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
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
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
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
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)
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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?
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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?
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
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
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
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
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