[00:01:16] 07Design, 10UI-Standardization, 06WMF-Design, 10OOjs-UI, 03UI-Standardization-Kanban: Do an icon inventory - https://phabricator.wikimedia.org/T141801#2574054 (10JGirault) @Volker_E what is the intended difference between {icon check color=yellow} and {icon check color=green} ? [00:02:39] (03PS1) 10Legoktm: build: Pass --path vendor/bundle to bundle install [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/306079 (https://phabricator.wikimedia.org/T143601) [00:03:12] 10AbuseFilter, 13Patch-For-Review, 05WMF-deploy-2016-08-16_(1.28.0-wmf.15), 05WMF-deploy-2016-08-23_(1.28.0-wmf.16): abusefilter-modify should let you view histories of private filters - https://phabricator.wikimedia.org/T143365#2574057 (10Platonides) 05Open>03Resolved a:03Platonides It has just been... [00:03:52] 10MediaWiki-Maintenance-scripts, 13Patch-For-Review: namespaceDupes.php --merge can throw a DBUnexpectedError DatabaseBase::{closure}: Flushing an explicit transaction, getting out of sync! exception - https://phabricator.wikimedia.org/T143631#2574061 (10Dereckson) [00:03:54] 10Wikimedia-Site-requests: Namespace problem on Azerbaijani Wiktionary - https://phabricator.wikimedia.org/T143580#2574060 (10Dereckson) [00:05:19] 10MediaWiki-Export-or-Import: Special:Import does not import the actual file - https://phabricator.wikimedia.org/T143591#2574063 (10TTO) That'd be T8071 and its friend T15827. A very long-standing feature request. [00:05:21] (03PS5) 10Aaron Schulz: Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds [core] - 10https://gerrit.wikimedia.org/r/305952 [00:05:28] 10MediaWiki-Export-or-Import: Special:Import does not import the actual file - https://phabricator.wikimedia.org/T143591#2574067 (10TTO) [00:05:31] 10MediaWiki-Export-or-Import: Transwiki image import - https://phabricator.wikimedia.org/T8071#112905 (10TTO) [00:08:25] (03CR) 10MaxSem: [C: 031] "Is the ability to return to third-party sites via interwiki links problematic?" [core] - 10https://gerrit.wikimedia.org/r/305834 (owner: 10Aaron Schulz) [00:13:59] (03PS1) 10Aaron Schulz: Move invalidatePages() to new PurgeJobUtils class [core] - 10https://gerrit.wikimedia.org/r/306080 [00:15:32] MaxSem: https://gerrit.wikimedia.org/r/#/c/305688/ moar patch [00:17:58] 10Wikimedia-Site-requests: English Wikinews: request for AbuseFilter available actions to include 'block' - https://phabricator.wikimedia.org/T57868#2574072 (10Dereckson) [00:19:10] Hello! I completed GSoC 2015 with Moodle. Can I participate in GCI as mentor for Wikimedia this year? [00:21:38] Hi mbt [00:21:55] as mentor? [00:22:06] how familiar are you with mediawiki, mbt ? [00:22:51] Yes, I worked with PHP and JavaScript for my GSoC project. I have not yet contributed to mediawiki yet. [00:22:54] You probably want to participate a little bit to Phabricator tasks triage, some code contributions. That would help you to know how we work, and so how to answer questions of the students. [00:22:58] technically, it would be possible, of course [00:23:02] (03CR) 10Catrope: Maintenance: moderate notifications linked to missing pages (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/302480 (https://phabricator.wikimedia.org/T141816) (owner: 10Sbisson) [00:23:17] I just mean, for mentoring I would expect that you are familiar with mediawiki [00:23:30] I wanted to know if I am eligible. Thank you Platonides :) [00:23:47] yes, you are [00:24:08] Sure, I will try to contribute as much as I can before beginning of GCI 2016. [00:24:27] ok [00:24:29] you have enough time [00:24:43] Can you help me get started, Platonides? [00:25:56] Indeed, if the timeline is similar to last year one, you've 4 months to get familiar with MW code and MW development practices - https://developers.google.com/open-source/gci/timeline [00:26:08] mbt: https://www.mediawiki.org/wiki/How_to_become_a_MediaWiki_hacker [00:26:49] Thank you Dereckson and Platonides. :) [00:27:07] you're welcome, and thank you for volunteering [00:27:36] :) [00:33:03] 10Brickimedia Association, 10DeepSea: DeepSea lacks links in both the sidebar and the toolbar to create and view reviews - https://phabricator.wikimedia.org/T143633#2574078 (10lcf119) [00:33:27] 10Brickimedia Association, 10DeepSea: DeepSea lacks links in both the sidebar and the toolbar to create and view reviews - https://phabricator.wikimedia.org/T143633#2574091 (10lcf119) [00:38:53] bd808, tgr, ori, is there a way to provision global composer packages in Vagrant yet? [00:39:08] 10MediaWiki-Special-pages, 07Easy, 13Patch-For-Review: Make Special:UserRights available to everyone - https://phabricator.wikimedia.org/T27319#2574092 (10Huji) What is the status of the patch? On gerrit it says "cannot merge". I guess it needs to be rebased as it is very old. If so, I am happy to do that an... [00:40:42] matt_flaschen: you could do it with an exec. there's no define for doing that that I know of [00:41:06] (curious) what do you want globally installed? [00:42:09] 06Research-and-Data, 10Research-outreach: VIVO 2016 talk - https://phabricator.wikimedia.org/T141635#2505582 (10DarTar) Slidedeck: http://www.slideshare.net/dartar/verifiable-linked-open-knowledge-that-anyone-can-edit [00:42:20] bd808, psysh. I'm replacing phpsh with it. phpsh doesn't work at all (at least for multi-wiki, and I think single-wiki too per T123951), and we need a REPL that works for Zend. [00:42:20] T123951: "Unable to determine IP" when running phpsh - https://phabricator.wikimedia.org/T123951 [00:43:04] 10MediaWiki-Parser, 10Cite: Extension Cite makes headers become part of ordered lists and indented - https://phabricator.wikimedia.org/T95386#2574099 (10Ikesultan) 05Open>03Invalid After I removed the extension lingo the cite extension works perfectly fine. Sorry. I mistook this to be an issue with cite a... [00:43:18] tgr's idea but I had to few some more stuff for multi-wiki. [00:43:32] *nod* look at how this define works and tweak from there -- https://github.com/wikimedia/mediawiki-vagrant/blob/master/puppet/modules/php/manifests/composer/install.pp [00:44:07] maybe make a php::composer::global define if it's more than a one off [00:45:16] you'd want ${::php::composer::home}/bin added to the global path too [00:46:00] 10MediaWiki-API: Revision API should support sizediff or sizes property - https://phabricator.wikimedia.org/T143444#2574103 (10Jdlrobson) I assume a fix would also allow me to query the difference in size with this query too? https://en.wikipedia.org/wiki/Special:ApiSandbox#action=query&format=json&prop=revisio... [00:46:04] bd808, I think I can just use composer::install after provisioning .config/composer/composer.json . [00:46:13] 10MediaWiki-Parser, 10Cite: Extension Cite makes headers become part of ordered lists and indented - https://phabricator.wikimedia.org/T95386#1188381 (10Izno) >>! In T95386#2574099, @Ikesultan wrote: > After I removed the extension lingo the cite extension works perfectly fine. Sorry. I mistook this to be an... [00:46:34] But I'm not sure if that is the best solution. [00:47:41] matt_flaschen: yeah, that could work I think. You could get fancy with composer-merge-plugin and a conf.d style dir to allow multiple things to be installed [00:48:33] (03PS1) 10Bearloga: Add intelligent color palettes [wikimedia/discovery/polloi] - 10https://gerrit.wikimedia.org/r/306083 [00:49:15] or you could just make a new define that does `composer global ...` [00:49:26] (03PS2) 10Bearloga: Add intelligent color palettes [wikimedia/discovery/polloi] - 10https://gerrit.wikimedia.org/r/306083 [00:50:14] the latter would probably be simpler in the long run [00:51:24] 10MediaWiki-extensions-DonationInterface, 10FR-ActiveMQ, 10FR-Smashpig, 10Fundraising-Backlog, and 5 others: Run the orphan rectifier job from CRM Jenkins - https://phabricator.wikimedia.org/T141487#2574111 (10awight) Have some stuff for review in the config deployment repo, on the `awight-orphanage` branch. [00:51:55] bd808, the merge plugin is a good suggestion, but in both cases I don't know how to solve the onlyif/unless issue. [00:52:11] composer::install just uses creates which is a good solution for a single thing, but won't work here. [00:53:31] bd808, I think for now I'm just not going to make it global. Then I can just put a composer.json file somewhere, own it (no need to merge), and the composer::install will work as intended. [00:53:34] you'd need to tune it for each package, but creates on vendor/bin/foo [00:54:19] why were you looking to install it globally in the first place? [00:54:30] (03PS1) 10Bearloga: 'Bottom 10' to static 'Bottom 50', makes 'Top 10' static too [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306084 (https://phabricator.wikimedia.org/T140816) [00:54:59] (03CR) 10Bearloga: [C: 032 V: 032] Add intelligent color palettes [wikimedia/discovery/polloi] - 10https://gerrit.wikimedia.org/r/306083 (owner: 10Bearloga) [00:55:11] (03CR) 10Bearloga: [C: 032 V: 032] 'Bottom 10' to static 'Bottom 50', makes 'Top 10' static too [wikimedia/discovery/prince] - 10https://gerrit.wikimedia.org/r/306084 (https://phabricator.wikimedia.org/T140816) (owner: 10Bearloga) [00:55:23] matt_flaschen: ^ [00:55:27] ori, yeah, I don't think there's a real need to in this case. The psysh entry point will be global, but that doesn't mean I have to do the composer install globally. I don't want people to use the stock entry point anyway. [00:55:53] * ori nods [00:55:56] Mainly because tgr did it with global in his local workaround, so I just was thinking along those lines. [01:01:50] 06Discovery, 06Discovery-Analysis (Current work), 13Patch-For-Review: Wikipedia.org Portal Dashboard: add clicks by language - https://phabricator.wikimedia.org/T140816#2574115 (10mpopov) @debt: okie dokie :) http://discovery-beta.wmflabs.org/portal/#languages_visited [01:05:58] (03PS2) 10Aaron Schulz: Move invalidatePages() to new PurgeJobUtils class [core] - 10https://gerrit.wikimedia.org/r/306080 [01:06:28] bd808: there we go [01:08:29] AaronSchulz: *nit* random newline added to includes/page/Article.php [01:09:02] probably dirty in your working dir from something else I imagine [01:09:17] (03PS3) 10Aaron Schulz: Move invalidatePages() to new PurgeJobUtils class [core] - 10https://gerrit.wikimedia.org/r/306080 [01:09:50] (03CR) 10BryanDavis: [C: 032] Move invalidatePages() to new PurgeJobUtils class [core] - 10https://gerrit.wikimedia.org/r/306080 (owner: 10Aaron Schulz) [01:15:56] (03PS1) 10Chad: SECURITY: Check read permission when loading page content in ApiParse. [core] - 10https://gerrit.wikimedia.org/r/306086 (https://phabricator.wikimedia.org/T115333) [01:15:59] (03PS1) 10Chad: SECURITY: Make blocks log users out if $wgBlockDisablesLogin [core] - 10https://gerrit.wikimedia.org/r/306087 (https://phabricator.wikimedia.org/T129738) [01:16:02] (03PS1) 10Chad: SECURITY: Make $wgBlockDisablesLogin also restrict logged in permissions [core] - 10https://gerrit.wikimedia.org/r/306088 (https://phabricator.wikimedia.org/T129738) [01:16:05] (03PS1) 10Chad: SECURITY: Do not allow undeleting a revdel'd file if its top file [core] - 10https://gerrit.wikimedia.org/r/306089 (https://phabricator.wikimedia.org/T132926) [01:16:08] (03PS1) 10Chad: SECURITY: Require login to preview user CSS pages [core] - 10https://gerrit.wikimedia.org/r/306090 (https://phabricator.wikimedia.org/T133147) [01:16:11] (03PS1) 10Chad: SECURITY: Escape '<' and ']]>' in inline