[00:02:48] 10MediaWiki-General-or-Unknown, 6Security, 5MW-1.23-release, 5MW-1.25-release, and 4 others: Error thrown by VirtualRESTService when POST variable starts with '@' - https://phabricator.wikimedia.org/T118032#1902794 (10csteipp) [00:03:50] (03CR) 10Alex Monk: [C: 032] Allow blocking talk page access and send email access optionally [extensions/CheckUser] - 10https://gerrit.wikimedia.org/r/184116 (https://phabricator.wikimedia.org/T41213) (owner: 10Glaisher) [00:07:38] (03Merged) 10jenkins-bot: Allow blocking talk page access and send email access optionally [extensions/CheckUser] - 10https://gerrit.wikimedia.org/r/184116 (https://phabricator.wikimedia.org/T41213) (owner: 10Glaisher) [00:09:29] 10MediaWiki-Authentication-and-authorization, 6Security, 5MW-1.23-release, 5MW-1.25-release, and 5 others: Passwords generated by User::randomPassword() may be shorter than $wgMinimalPasswordLength - https://phabricator.wikimedia.org/T115522#1902798 (10csteipp) [00:09:56] (03CR) 10Bartosz Dziewoński: [C: 04-1] "The help texts look good, but removing the file extension results in 'badfilename' error: https://phabricator.wikimedia.org/F3140507 – cli" [core] - 10https://gerrit.wikimedia.org/r/253550 (https://phabricator.wikimedia.org/T116086) (owner: 10Prtksxna) [00:10:59] (03PS16) 10Jdlrobson: Modern module loading [core] - 10https://gerrit.wikimedia.org/r/260071 (https://phabricator.wikimedia.org/T108655) [00:11:01] (03PS1) 10Jdlrobson: WIP: Port mediawiki.util to requires module [core] - 10https://gerrit.wikimedia.org/r/260883 [00:11:36] 10CirrusSearch, 6Discovery: Make "insource" work for JavaScript, CSS and Lua pages - https://phabricator.wikimedia.org/T122290#1902803 (10Ricordisamoa) [00:11:38] 10MediaWiki-Search, 10CirrusSearch, 6Discovery: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1902804 (10Ricordisamoa) [00:12:15] 10MediaWiki-User-blocking, 6Security, 6Security-Team, 6Stewards-and-global-tools: Incorrect parsing of IPs for global block - https://phabricator.wikimedia.org/T97897#1902807 (10csteipp) [00:12:25] 10MediaWiki-extensions-CheckUser, 6Stewards-and-global-tools: Checkuser mass blocker does not affect already blocked users - https://phabricator.wikimedia.org/T24890#1902810 (10Krenair) Ran into this independently while reviewing different changes. We should at least not claim the block succeeded, and probably... [00:15:13] 10Phabricator-Upstream: Phabricator project auto-complete is arbitrary (in a bad way) - https://phabricator.wikimedia.org/T99739#1902814 (10matmarex) [00:16:37] (03CR) 10jenkins-bot: [V: 04-1] WIP: Port mediawiki.util to requires module [core] - 10https://gerrit.wikimedia.org/r/260883 (owner: 10Jdlrobson) [00:17:32] 3Discovery-Analysis-Sprint: Massively expand A/B test documentation - https://phabricator.wikimedia.org/T121757#1902815 (10Deskana) 5Open>3Resolved [00:19:57] 10MediaWiki-General-or-Unknown, 10UI-Standardization, 6Collaboration-Team-Backlog, 10Flow, and 4 others: Should be a standard way to access tablet browser width threshold - https://phabricator.wikimedia.org/T93675#1902818 (10Jdlrobson) Having a weird PHPUnit issue. Hopefully will get to bottom of it... [00:21:29] (03CR) 10Jdlrobson: "check experimental" [extensions/RelatedArticles] - 10https://gerrit.wikimedia.org/r/260765 (https://phabricator.wikimedia.org/T120715) (owner: 10Bmansurov) [00:22:01] 10MediaWiki-extensions-RelatedArticles, 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: RelatedArticles browser tests should run on a commit basis - https://phabricator.wikimedia.org/T120715#1902819 (10Jdlrobson) [00:22:05] 10Wikimedia-General-or-Unknown, 7Tracking: Steward, global sysop and SWMT tasks bugs (tracking) - https://phabricator.wikimedia.org/T43492#1902822 (10Krenair) [00:22:07] 10MediaWiki-extensions-CheckUser, 6Stewards-and-global-tools, 7Easy, 5Patch-For-Review: Block interface after "Get users" check should allow talk page/send email blocking - https://phabricator.wikimedia.org/T41213#1902820 (10Krenair) 5Open>3Resolved code review santa got around to it, bit earlier than... [00:22:26] 10MediaWiki-Redirects, 6Security, 6Security-Team, 5MW-1.23-release, and 6 others: A combination of Special:MyPage redirects and pagecounts allows an external site to know the wikipedia login of an user - https://phabricator.wikimedia.org/T109724#1902823 (10csteipp) [00:23:12] 10MediaWiki-Redirects, 6Security, 6Security-Team, 5MW-1.23-release, and 7 others: A combination of Special:MyPage redirects and pagecounts allows an external site to know the wikipedia login of an user - https://phabricator.wikimedia.org/T109724#1902825 (10csteipp) [00:28:23] (03PS3) 10FreedomFighterSparrow: Allow wikilinks in template names [extensions/MultiBoilerplate] - 10https://gerrit.wikimedia.org/r/260516 (https://phabricator.wikimedia.org/T43787) (owner: 10Gerrit Patch Uploader) [00:37:04] 10MediaWiki-extensions-MultiBoilerplate, 5Patch-For-Review: Allow wikilinks in boilerplate page - https://phabricator.wikimedia.org/T43787#1902835 (10FreedomFighterSparrow) @Jhf2442, I uploaded a new changeset (somehow Gerrit hasn't notified Phabricator yet): - The same change had to be applied for the special... [00:38:02] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1902844 (10Callanecc) @binbot, regarding common UAs, yes, however the collateral from a single UA is much less than blocking the everyone on the range. Just as we do wi... [00:39:51] 6Discovery, 10Maps: Passing invalid priority crashes tilerator - https://phabricator.wikimedia.org/T122383#1902847 (10Yurik) 3NEW a:3Yurik [00:40:16] 6Discovery, 10Maps: Upgrade Tilerator to latest Kue - https://phabricator.wikimedia.org/T122384#1902856 (10Yurik) 3NEW a:3Yurik [00:43:35] 10MediaWiki-extensions-GeoData, 6Discovery, 10Maps, 3Discovery-Maps-Sprint, and 2 others: Allow searches over rectangular areas - https://phabricator.wikimedia.org/T112034#1902864 (10MaxSem) 5Open>3Resolved a:3MaxSem [00:46:18] (03CR) 10Bartosz Dziewoński: [C: 04-1] mw.ForeignStructuredUpload.BookletLayout: Use lastModified as default date (031 comment) [core] - 10https://gerrit.wikimedia.org/r/247513 (https://phabricator.wikimedia.org/T115863) (owner: 10Prtksxna) [00:52:22] 10MediaWiki-extensions-MultimediaViewer, 7Browser-Tests: Fix failed MultimediaViewer browsertests Jenkins jobs - https://phabricator.wikimedia.org/T94157#1902877 (10Jdlrobson) @gilles do you need the mmw.performance browser test still? It doesn't seem to quite work and I'm wondering if it's safe to delete? [00:55:18] 10Phabricator-Upstream: Cannot add a two-letter user in herald rule if there are more-letters ones with the same prefix - https://phabricator.wikimedia.org/T121273#1902879 (10matmarex) [00:55:19] 10Phabricator-Upstream: Phabricator project auto-complete is arbitrary (in a bad way) - https://phabricator.wikimedia.org/T99739#1902880 (10matmarex) [00:59:26] 6Community-Liaison: Publish a VE newsletter for December 2015 - https://phabricator.wikimedia.org/T121906#1902885 (10Whatamidoing-WMF) The English version has been sent to the English Wikipedia. [00:59:46] 3Reading-Web-Sprint-63-Ellip…: Create user page link seems dead - https://phabricator.wikimedia.org/T122376#1902888 (10JKatzWMF) [01:00:28] 3Reading-Web-Sprint-63-Ellip…: Create user page link seems dead - https://phabricator.wikimedia.org/T122376#1902532 (10JKatzWMF) [01:00:30] 6Discovery, 10Maps: Switch to v5 keyspace and delete obsolete ones. - https://phabricator.wikimedia.org/T122385#1902892 (10Yurik) 3NEW a:3Yurik [01:01:03] 3Reading-Web-Sprint-63-Ellip…: Create user page link seems dead - https://phabricator.wikimedia.org/T122376#1902532 (10JKatzWMF) @phuedx updated description to answer your questions, but also here: page http://en.m.wikipedia.beta.wmflabs.org/wiki/User:Mr%20Selenium?mobileaction=beta console: nothing shows up w... [01:04:26] 10WikiCon-Francophone-2016: Bénévoles pour la WikiConFR 2016 - https://phabricator.wikimedia.org/T122012#1902904 (10Pyb) [01:05:12] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: Move hamburger code back to the top of the page - https://phabricator.wikimedia.org/T118326#1902905 (10bmansurov) Thanks, Jon for the feedback. Your proposed solution doesn't solve the problem though. I've changed the position to top and throttled the net speed... [01:11:35] (03PS1) 10Aaron Schulz: Upgrade mismatched commit() warnings to exceptions [core] - 10https://gerrit.wikimedia.org/r/260888 [01:12:05] (03PS1) 10Ebe123: Show talk pages to users with fewer than 5 edits [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) [01:13:38] 10MobileFrontend, 7Easy, 5Patch-For-Review: Show talk pages on mobile web if user is logged in - https://phabricator.wikimedia.org/T122311#1902915 (10Ebe123) a:3Ebe123 [01:13:57] 6Discovery, 3Discovery-Analysis-Sprint: Analyse existing data on preferences to figure out if it can tell us about people opting in and out of the completion suggester beta feature - https://phabricator.wikimedia.org/T120260#1902916 (10Deskana) 5Open>3Resolved [01:16:32] 10MediaWiki-Search, 6Discovery, 6Wikisource: Search media (images, videos, sounds, etc) by relevant metadata - https://phabricator.wikimedia.org/T15370#1902917 (10Deskana) p:5Low>3Lowest [01:17:26] (03PS1) 10Aaron Schulz: [WIP] Upgrade mismatched begin() warnings to exceptions [core] - 10https://gerrit.wikimedia.org/r/260890 [01:17:43] 6Discovery, 3Discovery-UX-Sprint: [PAB1] Deploy A/B test (DUE Nov 23) - https://phabricator.wikimedia.org/T115960#1902920 (10Deskana) 5Open>3Resolved [01:17:44] 6Discovery, 3Discovery-UX-Sprint, 7Epic: EPIC: [Portal A/B test 1]: Change the size of search text field and search button on Wikipedia.org - https://phabricator.wikimedia.org/T112173#1902922 (10Deskana) [01:17:49] 6Discovery, 3Discovery-UX-Sprint, 7Epic: EPIC: [Portal A/B test 1]: Change the size of search text field and search button on Wikipedia.org - https://phabricator.wikimedia.org/T112173#1628030 (10Deskana) [01:17:50] 6Discovery, 3Discovery-UX-Sprint, 5Patch-For-Review: [PAB1] Actual code to change size of portal text box/button, inactive except during future A/B testing (DUE Nov 23) - https://phabricator.wikimedia.org/T115957#1902923 (10Deskana) 5Open>3Resolved [01:18:08] 3Reading-Web-Sprint-63-Ellip…: Create user page link seems dead - https://phabricator.wikimedia.org/T122376#1902927 (10bmansurov) Seems like a Gather bug. Here is what I see in the JS console when I open http://en.m.wikipedia.beta.wmflabs.org/wiki/User:Mr%20Selenium?debug=true&mobileaction=beta#/editor/0 ``` Un... [01:19:24] (03PS4) 10Bartosz Dziewoński: mediawiki.Upload: Add #getApi method, use it instead of undocumented properties [core] - 10https://gerrit.wikimedia.org/r/257670 [01:19:48] (03CR) 10Bartosz Dziewoński: "Rebased." [core] - 10https://gerrit.wikimedia.org/r/257670 (owner: 10Bartosz Dziewoński) [01:19:55] 6Discovery, 10Maps: Tilerator Out Of Memory importing large tileset changes - https://phabricator.wikimedia.org/T122386#1902928 (10Yurik) 3NEW a:3Yurik [01:21:01] (03PS3) 10Bartosz Dziewoński: mw.Upload.BookletLayout: Show an error rather than explode when uploads are disabled [core] - 10https://gerrit.wikimedia.org/r/257671 (https://phabricator.wikimedia.org/T119314) [01:21:37] 10Wikimedia-General-or-Unknown, 7Easy, 7I18n: [[Wikimedia:Wikipedia-ios-alert-no-internet/en]] i18n issue - https://phabricator.wikimedia.org/T122259#1899740 (10Ebe123) [01:23:25] (03PS1) 10Aaron Schulz: Make sure docs encourage __METHOD__ use for begin/commit [core] - 10https://gerrit.wikimedia.org/r/260891 [01:23:32] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…, 5WMF-deploy-2016-01-12_(1.27.0-wmf.10): [Regression] No talk or language button in beta - https://phabricator.wikimedia.org/T122233#1902949 (10bmansurov) Not only that, other things (section toggle icons for example) on the page are also broken. Something is... [01:25:03] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: Move hamburger code back to the top of the page - https://phabricator.wikimedia.org/T118326#1902951 (10Jdlrobson) Well immediately is not the fairest test :) and 2g is never going to be instant. It wasn't before and has no chance now Moving to top should redu... [01:25:54] (03CR) 10Bartosz Dziewoński: "@Krinkle, all good points, but also all are about code that was already here. :) I'll submit a separate commit to clean them up." [core] - 10https://gerrit.wikimedia.org/r/257671 (https://phabricator.wikimedia.org/T119314) (owner: 10Bartosz Dziewoński) [01:26:02] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: Move hamburger code back to the top of the page - https://phabricator.wikimedia.org/T118326#1902960 (10Jdlrobson) In terms of your current solution if the js fails to load completely the menu becomes unusable and broken. [01:26:54] (03PS19) 10Gergő Tisza: [WIP] Use AuthManager on Special:Userlogin [core] - 10https://gerrit.wikimedia.org/r/240052 (https://phabricator.wikimedia.org/T110277) [01:29:21] 6Discovery, 3Discovery-UX-Sprint, 7Epic: EPIC: Prototyping Knowledge Engine - https://phabricator.wikimedia.org/T114263#1902964 (10Deskana) >>! In T114263#1780515, @Wittylama wrote: > If "knowledge engine" means what I think it means, then this is an attempt to b the all-singing-all-dancing solution to the I... [01:29:35] (03PS1) 10Neonowy: Use OO.ui.confirm() for the confirmation dialog in ext.tmh.transcodetable [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/260892 (https://phabricator.wikimedia.org/T117078) [01:30:03] 6Discovery, 3Discovery-UX-Sprint, 7Epic: EPIC: Prototyping Knowledge Engine - https://phabricator.wikimedia.org/T114263#1902968 (10Deskana) And, on reflection, almost all of what I said above is pretty off-topic for a task management system like Phabricator. If you have any questions, feel free to email the... [01:30:51] legoktm: is there a way to reconcile https://gerrit.wikimedia.org/r/#/c/253300/7/OATHAuth.php with https://gerrit.wikimedia.org/r/#/c/259794/ ? [01:31:09] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: Move hamburger code back to the top of the page - https://phabricator.wikimedia.org/T118326#1902969 (10bmansurov) > In terms of your current solution if the js fails to load completely the menu becomes unusable and broken. Not really, the menu is usable without... [01:31:19] 3Discovery-Analysis-Sprint: Create a document listing all the steps for a Discovery A/B test - https://phabricator.wikimedia.org/T112604#1902970 (10Deskana) 5Open>3Resolved [01:31:59] (03PS8) 10Gergő Tisza: [WIP] Update for AuthManager [extensions/OATHAuth] - 10https://gerrit.wikimedia.org/r/253300 (https://phabricator.wikimedia.org/T110457) [01:32:14] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Use AuthManager on Special:Userlogin [core] - 10https://gerrit.wikimedia.org/r/240052 (https://phabricator.wikimedia.org/T110277) (owner: 10Gergő Tisza) [01:33:20] 3Discovery-Analysis-Sprint: Set up a staging/testing space for the dashboards (Discovery dashboards beta) - https://phabricator.wikimedia.org/T111979#1902971 (10Deskana) 5Open>3Resolved [01:33:34] (03PS11) 10Gergő Tisza: [WIP] Use AuthManager [extensions/OAuthAuthentication] - 10https://gerrit.wikimedia.org/r/251930 (https://phabricator.wikimedia.org/T110460) [01:33:42] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Update for AuthManager [extensions/OATHAuth] - 10https://gerrit.wikimedia.org/r/253300 (https://phabricator.wikimedia.org/T110457) (owner: 10Gergő Tisza) [01:34:18] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Use AuthManager [extensions/OAuthAuthentication] - 10https://gerrit.wikimedia.org/r/251930 (https://phabricator.wikimedia.org/T110460) (owner: 10Gergő Tisza) [01:34:40] 10MediaWiki-extensions-RelatedArticles, 6Discovery, 3Discovery-Cirrus-Sprint, 7Elasticsearch, and 3 others: RelatedArticles API query can take 2 seconds - https://phabricator.wikimedia.org/T121254#1902972 (10Deskana) >>! In T121254#1902507, @dr0ptp4kt wrote: > @dcausse, were you thinking to have CirrusSear... [01:34:55] 7Design, 10MediaWiki-User-preferences, 7Accessibility: Accessibility settings - https://phabricator.wikimedia.org/T91201#1902973 (10violetto) A couple of weeks ago @Volker_E spotted LinkedIn's execution in skip navigation link: {F3140697} In the future, we would like to have an "accessibility setting cor... [01:37:33] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1902976 (10MZMcBride) Adding this functionality to the AbuseFilter extension would probably be significantly easier. [01:38:05] (03PS20) 10Gergő Tisza: [WIP] Use AuthManager on Special:Userlogin [core] - 10https://gerrit.wikimedia.org/r/240052 (https://phabricator.wikimedia.org/T110277) [01:38:49] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Use AuthManager on Special:Userlogin [core] - 10https://gerrit.wikimedia.org/r/240052 (https://phabricator.wikimedia.org/T110277) (owner: 10Gergő Tisza) [01:39:37] (03PS4) 10Bartosz Dziewoński: mw.Upload.BookletLayout: Show an error rather than explode when uploads are disabled [core] - 10https://gerrit.wikimedia.org/r/257671 (https://phabricator.wikimedia.org/T119314) [01:39:39] (03PS1) 10Bartosz Dziewoński: mw.Upload.BookletLayout: Rewrite some code to use promise chaining [core] - 10https://gerrit.wikimedia.org/r/260894 [01:39:46] (03CR) 10Bartosz Dziewoński: "https://gerrit.wikimedia.org/r/260894" [core] - 10https://gerrit.wikimedia.org/r/257671 (https://phabricator.wikimedia.org/T119314) (owner: 10Bartosz Dziewoński) [01:43:05] 10Commons-App-Android-Upload: Daily reports - https://phabricator.wikimedia.org/T119164#1902980 (10josephine_l) [01:43:12] 10MediaWiki-extensions-CheckUser: Recursive checkuser featured needed - https://phabricator.wikimedia.org/T11858#1902981 (10MZMcBride) >>! In T11858#1816719, @Billinghurst wrote: > @Rich_Farmbrough checkusers do not retain data of good users, that was a clear statement made by WMF legal to checkusers and steward... [01:45:29] 10Commons-App-Android-Upload: Daily reports - https://phabricator.wikimedia.org/T119164#1902982 (10josephine_l) [01:47:26] 10Commons-App-Android-Upload: Phase 1: Display nearby categories when user uploads a photo - https://phabricator.wikimedia.org/T119163#1902986 (10josephine_l) [01:47:28] 10Commons-App-Android-Upload: Implement asynchronous calls to the APIs to match an uploaded picture's GPS coordinates with nearby Commons categories - https://phabricator.wikimedia.org/T119280#1902985 (10josephine_l) 5Open>3Resolved [01:51:47] (03CR) 10Bartosz Dziewoński: mw.Title: Implement #makeTitle for titles with predefined namespace (033 comments) [core] - 10https://gerrit.wikimedia.org/r/260613 (owner: 10Bartosz Dziewoński) [01:51:51] (03PS2) 10Bartosz Dziewoński: mw.Title: Implement #makeTitle for titles with predefined namespace [core] - 10https://gerrit.wikimedia.org/r/260613 [01:58:13] (03CR) 10jenkins-bot: [V: 04-1] mw.Title: Implement #makeTitle for titles with predefined namespace [core] - 10https://gerrit.wikimedia.org/r/260613 (owner: 10Bartosz Dziewoński) [01:58:49] (03CR) 10Bartosz Dziewoński: [C: 04-1] "@Krinkle: So, as you can see, JSDuck doesn't like the JSCS-compliant version of constructor documentation. Unless you have a solution, I'm" [core] - 10https://gerrit.wikimedia.org/r/260613 (owner: 10Bartosz Dziewoński) [02:00:27] (03PS15) 10Krinkle: [WIP] RollbackAction: Implement AJAX interface and require POST [core] - 10https://gerrit.wikimedia.org/r/242050 (https://phabricator.wikimedia.org/T88044) [02:02:48] 10MediaWiki-Search, 10CirrusSearch, 6Discovery: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1902987 (10MZMcBride) @EBernhardson: Is there somewhere that I can query to see what data Elasticsearch has for (for example) the "MediaWiki:Common.... [02:06:12] 10MediaWiki-extensions-CheckUser: Recursive checkuser featured needed - https://phabricator.wikimedia.org/T11858#1902991 (10Billinghurst) >>! In T11858#1902981, @MZMcBride wrote: >>>! In T11858#1816719, @Billinghurst wrote: >> @Rich_Farmbrough checkusers do not retain data of good users, that was a clear stateme... [02:25:28] (03PS1) 10Sn1per: Reenable InfiniteScroll and settings tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) [02:25:29] 10Commons-App-Android-Upload: Display the resulting categories to the user alongside the "previous categories" that are currently already displayed - https://phabricator.wikimedia.org/T119281#1903001 (10josephine_l) @Nicolas_Raoul @Niedzielski - In order to ensure uniqueness in the categories displayed to the us... [02:26:06] 7Design, 10Math, 5Patch-For-Review: Provide an "indented display" styling for formulæ in lieu of the ":" hack - https://phabricator.wikimedia.org/T111712#1903005 (10Whatamidoing-WMF) [02:34:57] 3Reading-Web-Sprint-63-Ellip…: Create user page link seems dead - https://phabricator.wikimedia.org/T122376#1903006 (10bmansurov) OK, so the edit link is working now. The problem has something to do with the Beta cluster then. [02:36:49] (03PS1) 10Bmansurov: Add Gemfile [extensions/RelatedArticles] (dev) - 10https://gerrit.wikimedia.org/r/260896 (https://phabricator.wikimedia.org/T120715) [02:37:14] (03Abandoned) 10Bmansurov: Add Gemfile [extensions/RelatedArticles] - 10https://gerrit.wikimedia.org/r/260765 (https://phabricator.wikimedia.org/T120715) (owner: 10Bmansurov) [02:37:42] (03CR) 10jenkins-bot: [V: 04-1] Add Gemfile [extensions/RelatedArticles] (dev) - 10https://gerrit.wikimedia.org/r/260896 (https://phabricator.wikimedia.org/T120715) (owner: 10Bmansurov) [02:43:18] (03CR) 10jenkins-bot: [V: 04-1] Reenable InfiniteScroll and settings tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) (owner: 10Sn1per) [02:57:15] (03CR) 10Bmansurov: [C: 032] Add the MenuBuilder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/229728 (owner: 10Phuedx) [02:57:44] (03PS11) 10Bmansurov: Menu code should be separated from skin and work on all skins [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/245598 (owner: 10Jdlrobson) [03:02:17] (03Merged) 10jenkins-bot: Add the MenuBuilder [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/229728 (owner: 10Phuedx) [03:02:51] 10Living-Style-Guide, 10MediaWiki-skins-Blueprint, 7Accessibility, 5Patch-For-Review: Blueprint should include skip-to-nav and skip-to-content links - https://phabricator.wikimedia.org/T112308#1903021 (10violetto) https://invis.io/NZ5F7S4EQ Hitting on "Skip to navigation" skips users' cursor focus to the... [03:04:24] (03PS12) 10Bmansurov: Breaking change: Make SkinMinerva use the MenuBuilder class [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/229729 (https://phabricator.wikimedia.org/T65459) (owner: 10Phuedx) [03:12:26] (03CR) 10jenkins-bot: [V: 04-1] Breaking change: Make SkinMinerva use the MenuBuilder class [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/229729 (https://phabricator.wikimedia.org/T65459) (owner: 10Phuedx) [03:13:09] (03CR) 10Bmansurov: [C: 032] Breaking change: Make SkinMinerva use the MenuBuilder class [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/229729 (https://phabricator.wikimedia.org/T65459) (owner: 10Phuedx) [03:17:05] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…, 5WMF-deploy-2016-01-12_(1.27.0-wmf.10): [Regression] No talk or language button in beta - https://phabricator.wikimedia.org/T122233#1903027 (10bmansurov) Aaand it's working again automagically. [03:17:24] (03Merged) 10jenkins-bot: Breaking change: Make SkinMinerva use the MenuBuilder class [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/229729 (https://phabricator.wikimedia.org/T65459) (owner: 10Phuedx) [03:29:24] 10UI-Standardization, 10OOjs-UI: Consider consolidating progressive and constructive buttons - https://phabricator.wikimedia.org/T110555#1903031 (10Volker_E) @violetto One thing I'm missing in the discussion above and the conclusion from different inputs is concerning the [[ https://doc.wikimedia.org/oojs-ui/m... [03:47:05] 10UI-Standardization, 10OOjs-UI: Consider consolidating progressive and constructive buttons - https://phabricator.wikimedia.org/T110555#1903048 (10violetto) >>! In T110555#1903031, @Volker_E wrote: > @violetto One thing I'm missing in the discussion above and the conclusion from different inputs is concerning... [03:56:04] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1903053 (10binbot) @MZMcBride: As far as I remember, this was discussed on mailing list. The tool should remain in the scope of checkusers. AbuseFilter is open to too m... [04:15:00] legoktm: I'm trying to push a patch, but for some reason, it says rewrite wikihiero.php is ony 73%. https://dpaste.de/2aet [04:19:52] (03CR) 10Aaron Schulz: [C: 032] SpecialContributions: Simplify check for 'Special:Contributions/newbies' [core] - 10https://gerrit.wikimedia.org/r/260859 (https://phabricator.wikimedia.org/T36659) (owner: 10Bartosz Dziewoński) [04:20:01] 7Design, 10MediaWiki-User-preferences, 7Accessibility: Accessibility settings - https://phabricator.wikimedia.org/T91201#1903065 (10Quiddity) >>! In T91201#1902973, @violetto wrote: > [...] These are some early thoughts and notes. What do you guys think? > IIUC, this is along the lines of the examples I w... [04:21:49] 7Design, 10MediaWiki-User-preferences, 7Accessibility: Accessibility settings - https://phabricator.wikimedia.org/T91201#1903066 (10violetto) [04:25:17] 10MediaWiki-Interwiki, 6Performance-Team, 7HHVM: Use static arrays for Interwiki cache in Wikimedia production (instead of cdb files) - https://phabricator.wikimedia.org/T122362#1903067 (10Krinkle) [04:25:42] legoktm:^ [04:27:12] (03Merged) 10jenkins-bot: SpecialContributions: Simplify check for 'Special:Contributions/newbies' [core] - 10https://gerrit.wikimedia.org/r/260859 (https://phabricator.wikimedia.org/T36659) (owner: 10Bartosz Dziewoński) [04:31:25] MtDu: don't worry about it. It's fancy git mumbo jumbo [04:33:39] OH-: When I do git review, it gives me a whole bunch of stuff. Looks like my config file is wrong. But I fixed it. Am I supposed to run vagrant up first? [04:33:53] it shouldn't matter [04:33:53] (03CR) 10Mattflaschen: [C: 032] Add missing API i18n messages [extensions/GettingStarted] - 10https://gerrit.wikimedia.org/r/260399 (owner: 10Anomie) [04:43:42] 10MediaWiki-extensions-CheckUser, 6Stewards-and-global-tools, 7Easy, 5Patch-For-Review, 5WMF-deploy-2016-01-12_(1.27.0-wmf.10): Block interface after "Get users" check should allow talk page/send email blocking - https://phabricator.wikimedia.org/T41213#1903070 (10Glaisher) Thanks a lot @krenair! :) [04:49:36] 10Commons-App-Android-Upload: Daily reports - https://phabricator.wikimedia.org/T119164#1903079 (10josephine_l) [04:50:57] (03PS2) 10Sn1per: Re-enable InfiniteScroll and settings tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) [04:52:41] 10Phabricator-Sprint-Extension: Random error after phabricator upgrade - https://phabricator.wikimedia.org/T120176#1903083 (10Christopher) @rkraig it is not possible to get that error if the sprint update was applied. Has everything been reloaded? [04:55:43] 10MediaWiki-General-or-Unknown, 10MediaWiki-extensions-Translate, 5Patch-For-Review: DBPageLanguage isn't returned by Title::getPageLanguage - https://phabricator.wikimedia.org/T121666#1903084 (10Kunalgrover05) However according to the comments that were listed there, it will definitely take quite some time... [04:57:43] yay, the tests work! [05:03:00] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1903088 (10Glaisher) >>! In T100070#1902976, @MZMcBride wrote: > Adding this functionality to the AbuseFilter extension would probably be significantly easier. See {T5... [05:08:12] legoktm: What do I do? https://dpaste.de/JNpr [05:08:20] (03CR) 10jenkins-bot: [V: 04-1] Re-enable InfiniteScroll and settings tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) (owner: 10Sn1per) [05:08:25] OH-:^ [05:09:05] (03CR) 10Sn1per: "recheck" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) (owner: 10Sn1per) [05:09:46] MtDu: You have to explain what you're trying to do [05:10:48] OH-: I'm trying to delete it. But I'm not sure if I'm supposed to force it. [05:10:52] OH-: See the paste. [05:11:39] eh, -f it [05:12:19] (03CR) 10Glaisher: Update Jenkins tests (031 comment) [extensions/Newsletter] - 10https://gerrit.wikimedia.org/r/260736 (https://phabricator.wikimedia.org/T121352) (owner: 10Paladox) [05:19:31] OH-: Can you help me with my git review? [05:19:34] OH-: https://dpaste.de/Uxn7 [05:28:42] MtDu: Can you do git fetch? [05:29:18] also, if you just directly use git push I don't think that will be shown [05:30:11] Glaisher: git fetch what? [05:30:26] just git fetch [05:30:31] MtDu: When you get that message, it either means your 'master' is out of date, or your patch is based on another patch [05:30:46] Also, when you type 'y', 'y' !== 'yes', so it won't work [05:31:04] y does work for some things [05:31:14] (I don't know about git review) [05:31:32] I think git review requires it be exactly 'yes' [05:33:56] Glaisher: So even though it says all that, I should still review? I thought that meant it wasn't in the right place, so it showed me all these unrelated commits. [05:34:38] MtDu: after doing git fetch, have you done git review? [05:34:54] Glaisher: One sec. [05:41:20] Glaisher: So i started fresh, and this is what I have. https://dpaste.de/pgoG I have not typed anything into the prompt at the end. [05:42:47] MtDu: type no [05:42:50] then do git fetch [05:43:16] Glaisher: Done. [05:43:27] And what happens? [05:43:49] Glaisher: It prompts me for my ssh password and once I type that in, nothing. [05:43:59] mhm [05:44:16] can you do git fetch --all? [05:45:19] Glaisher: Sure. Here. https://dpaste.de/F7Ek [05:46:40] 10MediaWiki-Search, 10CirrusSearch, 6Discovery: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1903093 (10ori) >>! In T88247#1902987, @MZMcBride wrote: > My guess is that `_source` was added to `?action=cirrusdump` and that we haven't regenera... [05:46:59] MtDu: and then doing git review does not work still? [05:47:12] Glaisher: Trying now. [05:50:51] Glaisher: It still gives me the same stuff. :( [05:51:01] ugh.. just use git push then... [05:51:10] MtDu: git push gerrit HEAD:refs/for/master [05:52:22] Glaisher: It's not trying to push it to wikihiero, which is weird. https://dpaste.de/jGNO [05:52:38] (03PS1) 10Mattflaschen: Have foreachwiki exit with non-zero status if any wiki fails [vagrant] - 10https://gerrit.wikimedia.org/r/260901 [05:53:35] (03CR) 10jenkins-bot: [V: 04-1] Have foreachwiki exit with non-zero status if any wiki fails [vagrant] - 10https://gerrit.wikimedia.org/r/260901 (owner: 10Mattflaschen) [05:53:44] wth.. [05:53:51] MtDu: What does git remote -v show? [05:54:21] Glaisher: https://dpaste.de/7Gnf [05:55:08] huh.. I wonder how Echo and MV got there.. [05:55:55] MtDu: let's try manually deleting those then [05:56:14] Glaisher: So manually delete their git config files? [05:56:21] no [05:56:48] git remote set-url origin ssh://mdew192837@gerrit.wikimedia.org:29418/mediawiki/extensions/wikihiero.git [05:56:51] Wait. I think I know why. I added this to their git configs. https://dpaste.de/09J1 [05:57:09] Glaisher:^ [05:57:14] Glaisher: For Echo and MMV [05:57:41] you manually added that?? [05:57:51] Glaisher: Yeah. So it would work. [05:58:12] Glaisher: For some reason, I had to add that to both configs so I could push commits. [05:58:17] (03PS1) 10Mattflaschen: DO NOT MERGE: Output exception to help track down an issue [extensions/Flow] (REL1_26) - 10https://gerrit.wikimedia.org/r/260902 (https://phabricator.wikimedia.org/T122329) [05:58:30] Glaisher: I ran that command. Would you like me to try git review now? [05:58:39] yeah, try that [05:58:51] why did you modify that file? [05:59:32] Glaisher: Because it wasn't pushing it to the extensions. It was pushing it to MW core or something, so there were a ton of unrelated commits. [05:59:43] Glaisher: Some people on the IRC helped me. [06:00:24] AFAIK, git review automatically sets up that correctly [06:01:00] Glaisher: Ok. Btw, I also added that to the config for this extension. Will it affect it? [06:01:09] (03PS1) 10Ori.livneh: For source code pages, index page contents as `source_text` [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/260903 (https://phabricator.wikimedia.org/T88247) [06:01:11] (03PS2) 10TTO: Namespace translations for Lojban (jbo) [core] - 10https://gerrit.wikimedia.org/r/260756 (https://phabricator.wikimedia.org/T117841) [06:01:34] If it's wrong then it will affect it, yes [06:01:42] (03CR) 10jenkins-bot: [V: 04-1] DO NOT MERGE: Output exception to help track down an issue [extensions/Flow] (REL1_26) - 10https://gerrit.wikimedia.org/r/260902 (https://phabricator.wikimedia.org/T122329) (owner: 10Mattflaschen) [06:01:44] but why don't you just let git review handle that? [06:02:03] Glaisher: I tried that before with Echo and MMV, but it didn't work. [06:02:09] Glaisher: As described above. [06:02:22] Glaisher: Uh. Still giving me that stuff. [06:02:48] paste? [06:02:56] maybe that's because you're doing it in the vagrant dir.. IIRC, some issues come up if it's like that [06:04:27] Glaisher: https://dpaste.de/ecba [06:04:43] Glaisher: So if it's that, then what should I try? [06:05:50] *shrugs* I don't use vagrant but I remember seeing people complain about it [06:06:14] MtDu: Can you try doing git fetch --all for one last time? :) [06:08:15] hi [06:08:26] hi [06:08:40] legoktm: Hello! [06:08:45] MtDu: are you still having issues? [06:08:50] Yes. [06:08:54] legoktm:^ [06:09:25] hi [06:09:30] * OH- hi [06:10:22] 10MediaWiki-Internationalization, 5Patch-For-Review: Lojban translations - https://phabricator.wikimedia.org/T117841#1903103 (10TTO) >>! In T117841#1900838, @KSFT wrote: > Oh, I did make a mistake. "Template" should be "termo'a", not "morna". "Template talk" should be "casnu lo termo'a". Fixed. [06:10:33] MtDu: okay, so what does a plain "git review" output? [06:10:42] ooh, Lojban [06:11:11] 10MediaWiki-Search, 10CirrusSearch, 6Discovery, 3Discovery-Cirrus-Sprint, 5Patch-For-Review: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1903104 (10Deskana) [06:11:33] legoktm: Running one again now. [06:11:50] 10MediaWiki-Search, 10CirrusSearch, 6Discovery, 3Discovery-Cirrus-Sprint, 5Patch-For-Review: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1903107 (10Deskana) a:3ori Thanks for the patch, @ori. We'll get it reviewed. [06:13:19] (03CR) 10Cindy-the-browser-test-bot: [C: 04-1] "features/frozen_index_api.feature:30" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/260903 (https://phabricator.wikimedia.org/T88247) (owner: 10Ori.livneh) [06:13:37] legoktm: Here you go. https://dpaste.de/XNCz [06:14:54] MtDu: ok, lets try this. type in "git format-patch HEAD~1", it should create a file named 0001-something.patch in the directory, pastebin that .patch file (and the git output) [06:15:31] (03CR) 10Ori.livneh: "Where can I get more information about the test failure? I read the spec file and I don't see why the tests should be affected by this cha" [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/260903 (https://phabricator.wikimedia.org/T88247) (owner: 10Ori.livneh) [06:15:45] legoktm: Once I run that command, what do I do? [06:18:10] pastebin that .patch file [06:18:15] legoktm: https://dpaste.de/0eqX [06:18:35] ok, open the 0001-Convert-wikihiero-to-use-extension-registration.patch file in your text editor, and pastebin the contents [06:19:16] legoktm: https://dpaste.de/DFYB [06:20:14] nice :) I'm going to show you a trick now [06:20:45] that .patch file contains all of your changes plus the commit message, it's your entire commit in plaintext form [06:20:47] legoktm: Sweet. :) [06:20:57] so, clone the wikihiero repository into a new directory [06:21:19] legoktm: Hm? [06:21:20] then run git review -S to set up the new remote [06:21:38] like, git clone ssh://username@gerrit.wikimedia.org/.... [06:21:57] legoktm: Ok. While I'm cded in wikihiero? [06:23:16] legoktm: ^ [06:24:45] MtDu: no, in a new directory [06:25:17] MtDu: cd to your extensions directory, and git clone URL wikihiero2 (that'll clone the repo into a directory named wikihiero2) [06:25:40] legoktm: So this? git clone ssh://mdew192837@gerrit.wikimedia.org:29418/mediawiki/extensions/wikihiero.git wikihiero2 [06:25:57] yeah, that should work. [06:26:32] legoktm: Now git review -s? [06:26:54] legoktm: Well first cd into wikihiero2, then git review -s? [06:28:40] legoktm:^ [06:30:09] MtDu: yes [06:30:23] MtDu: and then, copy your .patch file from wikihiero into wikihiero2 [06:30:59] now, while cd'd into wikihiero2, run "git am 0001-Convert-wikihiero-to-use-extension-registration.patch". This applies the patch you created on top of the new git repository. [06:31:07] And then try "git review" and see what happens :) [06:31:09] legoktm: This doesn't look right. https://dpaste.de/YK6P [06:31:21] why doesn't that look right? [06:31:31] legoktm: Shouldn't it say some stuff? [06:32:00] I don't think so...? [06:32:15] But I don't actually use git review, so I'm not sure. [06:32:56] MtDu: try it and see? [06:33:01] legoktm: Trying now. [06:36:29] legoktm: Ugh. It gives me that huge list of stuff again. [06:38:40] MtDu: uhhh inside of wikihiero2?? [06:38:48] legoktm: Yep. :( [06:38:55] MtDu: okay, lets just hit yes and see what happens [06:39:23] legoktm: Let me run it again. I typed q. Sorry [06:43:07] legoktm: Weird. [06:43:10] legoktm: https://dpaste.de/r7hW [06:44:13] okayyyyyyyyy [06:44:22] MtDu: run "git remote -v" and pastebin that [06:44:52] legoktm: https://dpaste.de/sBjk [06:45:04] wtf [06:45:36] MtDu: run "git remote set-url gerrit ssh://mdew192837@gerrit.wikimedia.org:29418/mediawiki/extensions/wikihiero.git" [06:45:50] and then pastebin "git remote -v" after that [06:46:25] legoktm: https://dpaste.de/sJvX [06:47:28] MtDu: hrm. [06:47:33] MtDu: try "git remote remove gerrit" [06:47:42] "git remote add gerrit ssh://mdew192837@gerrit.wikimedia.org:29418/mediawiki/extensions/wikihiero.git" [06:47:48] and then pastebin git remote -v [06:49:00] legoktm: https://dpaste.de/JNsG [06:49:40] $ git remote remove gerrit [06:49:40] error: Could not remove config section 'remote.gerrit' [06:49:42] wait what [06:50:28] okay, that happens if there is no gerrit remote [06:50:42] so where in the world is that remote coming from??? [06:51:49] MtDu: I'm utterly lost as to what is going wrong. [06:52:04] Somehow Echo is forcing itself as a global remote?? [06:52:23] legoktm: Do you want me to pastebin my echo .git config file? [06:52:32] legoktm: his .git/config's have been tampered with [06:52:44] OH-: but we just cloned wikihiero2 fresh? [06:52:59] MtDu: no, I'm thinking right now [06:53:04] check vagrant .git/config? [06:53:12] not sure if git checks parent dir configs [06:54:22] MtDu: can you pastebin "git config --global --list" ? [06:55:04] legoktm: There's the problem. https://dpaste.de/SZuA [06:55:23] okayyyyyyyyyyy [06:55:27] how did that get in there??? [06:55:33] legoktm: I don't know. [06:55:43] MtDu: run "git config --global --edit", and delete the remote.gerrit lines and save it [06:56:09] legoktm: Ok. [06:56:16] legoktm: Done [06:56:27] [22:47:42] "git remote add gerrit ssh://mdew192837@gerrit.wikimedia.org:29418/mediawiki/extensions/wikihiero.git" [06:56:27] [22:47:48] and then pastebin git remote -v [06:56:29] try that again [06:56:33] it should actually work this time ;) [06:56:58] * ori awards legoktm the Medal of Patient Debugging for bravery in the field of battle [06:57:18] legoktm: https://dpaste.de/4z0v [06:57:22] legoktm: It works. :) [06:57:32] MtDu: yaaaay, now run "git review" :) [06:57:36] legoktm: Now, should I go back to wikihiero? [06:57:39] ori: :) [06:57:43] legoktm: Or just run it in wikihiero2? [06:57:51] MtDu: uh, lets stick with wikihiero2 for now [06:59:32] legoktm: https://dpaste.de/7BTk [06:59:41] legoktm: Should I run the command in line 14? [06:59:54] ohohoh, that's easy [07:00:16] MtDu: no, the hook is installed, but we ran "git am" which bypassed it [07:00:41] MtDu: run "git commit --amend" like you're going to update the patch, but don't change anything, just save, and it'll add the Change-Id line. then run git review again [07:01:27] (03PS1) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [07:01:33] YAAAAAAAAAY [07:01:34] legoktm: L) [07:01:41] legoktm: :):):):) [07:02:14] legoktm: So in future patches for this, I cd to wikihiero2? [07:02:31] MtDu: you can delete the original wikihiero and move the wikihiero2 to just wikihiero now [07:02:57] (03CR) 10jenkins-bot: [V: 04-1] Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) (owner: 10MtDu) [07:03:20] I'm going to take a break now, I'll review your patch in a little bit [07:03:31] you should probably take a look at the jenkins failures and fix those [07:03:33] * OH- gives legoktm a hat [07:04:00] * legoktm huggles OH- [07:08:06] (03PS2) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [07:09:05] (03CR) 10jenkins-bot: [V: 04-1] Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) (owner: 10MtDu) [07:19:48] (03PS3) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [07:20:35] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1903117 (10Callanecc) @MZMcBride as binbot says, if the UA data was going to be available to users who aren't CheckUsers it might prevent CUs from using it as it would... [07:21:27] (03CR) 10jenkins-bot: [V: 04-1] Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) (owner: 10MtDu) [07:30:13] (03PS4) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [07:31:20] (03CR) 10jenkins-bot: [V: 04-1] Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) (owner: 10MtDu) [07:34:47] legoktm: Are you back yet? [08:27:35] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1903137 (10MZMcBride) >>! In T100070#1903088, @Glaisher wrote: >>>! In T100070#1902976, @MZMcBride wrote: >> Adding this functionality to the AbuseFilter extension woul... [08:38:47] 10MediaWiki-Search, 10CirrusSearch, 6Discovery, 3Discovery-Cirrus-Sprint, 5Patch-For-Review: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1903138 (10MZMcBride) >>! In T88247#1903093, @ori wrote: >>>! In T88247#1902987, @MZMcBride wrote:... [08:43:37] (03CR) 10Legoktm: For source code pages, index page contents as `source_text` (031 comment) [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/260903 (https://phabricator.wikimedia.org/T88247) (owner: 10Ori.livneh) [08:46:55] (03CR) 10Legoktm: [C: 04-1] Convert wikihiero to use extension registration (035 comments) [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) (owner: 10MtDu) [09:02:48] (03CR) 10Aude: "this looks better now..." (031 comment) [core] - 10https://gerrit.wikimedia.org/r/260255 (owner: 10Hoo man) [09:06:15] 10MediaWiki-Search, 10CirrusSearch, 6Discovery, 3Discovery-Cirrus-Sprint, 5Patch-For-Review: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1903141 (10MZMcBride) Thank you for ! I poked around the Cir... [09:21:45] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1903143 (10Legoktm) How would these blocks be logged? >>! In T100070#1903053, @binbot wrote: > @MZMcBride: As far as I remember, this was discussed on mailing list. W... [09:36:51] (03PS6) 10Gilles: Custom thumbor filter for conditional sharpening [vagrant] - 10https://gerrit.wikimedia.org/r/249755 (https://phabricator.wikimedia.org/T112545) [09:37:30] (03CR) 10Gilles: "Dependency has now been merged" [vagrant] - 10https://gerrit.wikimedia.org/r/249755 (https://phabricator.wikimedia.org/T112545) (owner: 10Gilles) [09:38:48] (03CR) 10Gilles: "Dependency has now been merged" [thumbor/purger] - 10https://gerrit.wikimedia.org/r/252578 (https://phabricator.wikimedia.org/T111005) (owner: 10Gilles) [09:39:37] (03CR) 10Gilles: "Dependency has been merged" [vagrant] - 10https://gerrit.wikimedia.org/r/252639 (https://phabricator.wikimedia.org/T118439) (owner: 10Gilles) [09:39:42] (03PS2) 10Gilles: Set up file storage of results for thumbor [vagrant] - 10https://gerrit.wikimedia.org/r/252639 (https://phabricator.wikimedia.org/T118439) [09:41:18] 10MediaWiki-extensions-ContentTranslation, 7Easy: X icon in MT card is not well aligned - https://phabricator.wikimedia.org/T122282#1903145 (10Abhinand) a:3Abhinand [09:45:21] 10MediaWiki-General-or-Unknown, 7Tracking: Incorporate analytics into MediaWiki's info action (tracking) - https://phabricator.wikimedia.org/T43326#1903154 (10Nemo_bis) [09:46:26] (03PS1) 10Gilles: Abstract command wrapping for external use [thumbor/base-engine] - 10https://gerrit.wikimedia.org/r/260906 (https://phabricator.wikimedia.org/T120205) [09:48:50] (03PS6) 10Gilles: Make base engine more robust and send debugging info through exception [thumbor/base-engine] - 10https://gerrit.wikimedia.org/r/259693 [09:49:03] (03CR) 10Gilles: Make base engine more robust and send debugging info through exception (031 comment) [thumbor/base-engine] - 10https://gerrit.wikimedia.org/r/259693 (owner: 10Gilles) [09:49:11] (03PS2) 10Gilles: Abstract command wrapping for external use [thumbor/base-engine] - 10https://gerrit.wikimedia.org/r/260906 (https://phabricator.wikimedia.org/T120205) [09:57:00] 10MediaWiki-extensions-MultimediaViewer, 7Browser-Tests: Fix failed MultimediaViewer browsertests Jenkins jobs - https://phabricator.wikimedia.org/T94157#1903164 (10Gilles) It depends on why it fails. It's a safeguard against media viewer's performance becoming worse than the file page's. If it's started faili... [10:05:35] 10MediaWiki-User-blocking, 10MediaWiki-extensions-CheckUser: Allow User agent (UA)-based IP Blocks - https://phabricator.wikimedia.org/T100070#1903172 (10binbot) Either on Checkuser-l, or I am wrong. [10:10:58] 10MediaWiki-General-or-Unknown: Getting new messages adds my own user page to my watch list without my permission - https://phabricator.wikimedia.org/T6276#1903174 (10Nemo_bis) [10:12:05] 10MediaWiki-Search, 10CirrusSearch, 6Discovery, 3Discovery-Cirrus-Sprint, 5Patch-For-Review: insource should search article text on non-wikitext pages. Probably. - https://phabricator.wikimedia.org/T88247#1903178 (10Nemo_bis) p:5Low>3High [10:12:52] 10MediaWiki-extensions-MultimediaViewer, 7Browser-Tests: Fix failed MultimediaViewer browsertests Jenkins jobs - https://phabricator.wikimedia.org/T94157#1903179 (10Gilles) The error log clearly shows that it's a syntax problem, probably a change in the framework or cucumber due to an update: https://integrat... [10:14:21] 10MediaWiki-extensions-ContentTranslation, 7WorkType-Maintenance: when translating from non-main namespace and no target title is provided, ContentTranslation should auto-adapt namespace name - https://phabricator.wikimedia.org/T122393#1903180 (10Amire80) 3NEW [10:18:56] 10MediaWiki-Internationalization, 5Patch-For-Review: Lojban translations - https://phabricator.wikimedia.org/T117841#1903187 (10TTO) 5stalled>3Open [10:49:12] 6Community-Liaison, 10WMF-Product-Strategy, 10Wikimedia-Developer-Summit-2016: 10 tips for communicating with communities when developing software - https://phabricator.wikimedia.org/T113490#1903192 (10Trizek-WMF) [10:49:13] (03CR) 10Bmansurov: [C: 032] Re-enable InfiniteScroll and settings tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) (owner: 10Sn1per) [10:54:50] (03Merged) 10jenkins-bot: Re-enable InfiniteScroll and settings tests [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260895 (https://phabricator.wikimedia.org/T106628) (owner: 10Sn1per) [10:55:52] (03CR) 10Paladox: Update Jenkins tests (031 comment) [extensions/Newsletter] - 10https://gerrit.wikimedia.org/r/260736 (https://phabricator.wikimedia.org/T121352) (owner: 10Paladox) [10:56:26] 10MediaWiki-Watchlist, 7Easy, 7Performance: Watch user page and user talk page by default - https://phabricator.wikimedia.org/T51719#1903195 (10Nemo_bis) By the way it's easy for any MediaWiki (system) administrator to add more pages to watchlist by default at user creation, with something like this in `Loca... [10:59:26] 10MediaWiki-Unit-tests, 5Patch-For-Review: Certain tests failing in bypassed target mode - https://phabricator.wikimedia.org/T104362#1903198 (10bmansurov) [10:59:28] 10MobileFrontend, 3Google-Code-In-2015, 5Patch-For-Review: InfiniteScroll and api qunit tests are being skipped. - https://phabricator.wikimedia.org/T106628#1903197 (10bmansurov) 5Open>3Resolved [11:00:29] 6Community-Liaison, 10WMF-Product-Strategy, 10Wikimedia-Developer-Summit-2016: 10 tips for communicating with communities when developing software - https://phabricator.wikimedia.org/T113490#1903199 (10Trizek-WMF) WMF internal survey first results: - 53 responses - Most WMF people interact with volunteer com... [11:34:20] 10MediaWiki-extensions-MultimediaViewer, 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: MultimediaViewer browser tests should run on a per commit basis - https://phabricator.wikimedia.org/T113620#1903210 (10bmansurov) A test is failing: https://integration.wikimedia.org/ci/job/mwext-mw-selenium/2603/console [11:42:16] 10MediaWiki-extension-requests: Need a webpage to produce Pie chart according to Graphic Lab's recommendations - https://phabricator.wikimedia.org/T14603#1903214 (10AdHuikeshoven) a:3AdHuikeshoven On en.wp I added a [[https://en.wikipedia.org/wiki/Template:PieChart}template for piecharts]], including [[https:/... [11:44:38] 10MediaWiki-extension-requests, 10Graph: Need a webpage to produce Pie chart according to Graphic Lab's recommendations - https://phabricator.wikimedia.org/T14603#178823 (10AdHuikeshoven) Pie charts can now be added with functionality provided by extension:graph [11:51:42] 10Wikidata-Page-Banner: Cleanup WikidataPageBanner architecture - https://phabricator.wikimedia.org/T122096#1903224 (10bmansurov) While we are at it, document the config variables too. [11:58:10] (03CR) 10Physikerwelt: [C: 04-1] WIP: Add classes and modify hook in Math for PT:math (032 comments) [extensions/Math] - 10https://gerrit.wikimedia.org/r/259167 (https://phabricator.wikimedia.org/T67397) (owner: 10Llyrian) [12:04:45] 10MediaWiki-extensions-ContentTranslation, 7Easy: X icon in MT card is not well aligned - https://phabricator.wikimedia.org/T122282#1903238 (10Abhinand) Should I make it centered or add some space to left. I think centering it will look good. [12:20:57] 10Gather, 3Reading-Web-Sprint-63-Ellip…: Search box bug on Special:Gather - https://phabricator.wikimedia.org/T121946#1903243 (10bmansurov) a:3bmansurov [12:27:50] 10MediaWiki-Uploading, 6Commons, 6Multimedia, 7JavaScript, 7Voice & Tone: Large amounts of unwanted files (mostly copyvios) uploaded via cross-wiki upload tool (A/B test of different upload interfaces) - https://phabricator.wikimedia.org/T120867#1903246 (10Pginer-WMF) Thanks for sharing the preliminary r... [12:28:23] (03CR) 10Paladox: [C: 032] "Thanks." [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/259594 (owner: 10BryanDavis) [12:30:00] (03Merged) 10jenkins-bot: Rename Composer package and add description [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/259594 (owner: 10BryanDavis) [12:34:57] 10MediaWiki-extension-requests, 10Graph: Need a webpage to produce Pie chart according to Graphic Lab's recommendations - https://phabricator.wikimedia.org/T14603#1903248 (10Yurik) @AdHuikeshoven awesome work! I'm very happy that graphs are beginning to pick up )) I modified it a bit to make it smaller by de... [12:36:16] 10MediaWiki-extensions-LabeledSectionTransclusion, 10MediaWiki-extensions-ProofreadPage, 6Wikisource: Page number inclusion on transcluded works is confused or undefined under certain conditions... - https://phabricator.wikimedia.org/T122395#1903249 (10ShakespeareFan00) 3NEW [12:37:37] (03PS1) 10Bmansurov: Make the search input selector more specific [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260913 (https://phabricator.wikimedia.org/T121946) [12:38:48] 10Gather, 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: Search box bug on Special:Gather - https://phabricator.wikimedia.org/T121946#1903259 (10bmansurov) A temporary solution has been added. [12:43:40] (03PS31) 10Paladox: Use FormAction instead of UnknownAction hook [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/250007 (https://phabricator.wikimedia.org/T117280) [12:44:25] (03PS1) 10Paladox: Security fix [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260914 [12:46:51] Hi tto, I guess you are celebrating Christmas right now?:) [12:47:05] (03PS1) 10Paladox: Fix postgres support [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260915 [12:47:08] Hi Georggi, not quite yet! [12:47:19] I will be going to sleep soon, do you have a quick question? [12:47:34] Not really, wasnt able to look at the code until now [12:47:38] (03CR) 10Paladox: [C: 032] Security fix [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260914 (owner: 10Paladox) [12:48:41] (03Merged) 10jenkins-bot: Security fix [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260914 (owner: 10Paladox) [12:49:04] Hi TTO. Remember me? We just had a small conference on phabricator yesterday about localtime change on orwiki [12:49:09] (03PS32) 10Paladox: Use FormAction instead of UnknownAction hook [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/250007 (https://phabricator.wikimedia.org/T117280) [12:49:50] Hi JSahu. We usually allow a little while for the discussion to take place, in order to see if there are any objections. [12:50:04] Because of the holiday break, there will be no deployments until a couple of weeks into January [12:50:17] So we won't be able to change the default timezone until then. [12:50:32] (03CR) 10jenkins-bot: [V: 04-1] Make the search input selector more specific [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260913 (https://phabricator.wikimedia.org/T121946) (owner: 10Bmansurov) [12:50:40] That is okay but I am happy that i have completed all set up to start contributing to wikimedia. [12:51:04] Already clones the dev version and all. [12:51:04] (03CR) 10Florianschmidtwelzow: [C: 04-1] "I would love to see a change for Gather, so that it stops using this module on desktop. This actually means, that we fix bugs in other ext" (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260913 (https://phabricator.wikimedia.org/T121946) (owner: 10Bmansurov) [12:51:23] I just wanted to say Thank You. /tto [12:51:28] (03PS6) 10Paladox: Update Jenkins tests [extensions/Newsletter] - 10https://gerrit.wikimedia.org/r/260736 (https://phabricator.wikimedia.org/T121352) [12:52:38] JSahu: Not a problem! [12:53:59] (03PS1) 10Paladox: Update some packages [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260916 [12:54:27] (03CR) 10Paladox: [C: 032] Update some packages [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260916 (owner: 10Paladox) [12:55:27] (03Merged) 10jenkins-bot: Update some packages [extensions/DeletePagesForGood] - 10https://gerrit.wikimedia.org/r/260916 (owner: 10Paladox) [12:58:09] (03CR) 10Bmansurov: "Fixing the problem in Gather won't happen overnight. It will be a major architectural change and will take quite some time I imagine." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260913 (https://phabricator.wikimedia.org/T121946) (owner: 10Bmansurov) [13:03:18] (03CR) 10Sbisson: "Looks good, but how can we test this?" (031 comment) [extensions/Echo] - 10https://gerrit.wikimedia.org/r/260392 (https://phabricator.wikimedia.org/T121929) (owner: 10Mooeypoo) [13:13:55] 10MobileFrontend, 7Easy, 3Google-Code-In-2015, 3Reading-Web-Sprint-63-Ellip…: Special:Nearby replaces the list with text "Loading..." when an item is mid-clicked on desktop (= open in a new tab) - https://phabricator.wikimedia.org/T114389#1903300 (10Sumit) Imported as https://codein.withgoogle.com/dashboar... [13:19:12] (03PS1) 10Gilles: Configure Thumbor XCF, PDF, DjVU and TIFF support [vagrant] - 10https://gerrit.wikimedia.org/r/260919 (https://phabricator.wikimedia.org/T120203) [13:19:15] (03PS1) 10Gilles: cgconfigparser needs to run on startup [vagrant] - 10https://gerrit.wikimedia.org/r/260920 [13:20:39] 10MediaWiki-Interface, 10MediaWiki-Page-editing, 7JavaScript, 7Tracking: Edit toolbar & Edit tools (tracking) - https://phabricator.wikimedia.org/T19653#1903315 (10TheDJ) [13:22:08] (03PS2) 10Gilles: Configure Thumbor XCF, PDF, DjVU and TIFF support [vagrant] - 10https://gerrit.wikimedia.org/r/260919 (https://phabricator.wikimedia.org/T120203) [13:22:28] (03PS2) 10Gilles: cgconfigparser needs to run on startup [vagrant] - 10https://gerrit.wikimedia.org/r/260920 [13:26:55] 10MediaWiki-extensions-LabeledSectionTransclusion, 10MediaWiki-extensions-ProofreadPage, 6Wikisource: Page number inclusion on transcluded works is confused or undefined under certain conditions... - https://phabricator.wikimedia.org/T122395#1903319 (10Billinghurst) Clarity of mud. As has been mentioned to... [13:27:17] 10MediaWiki-extensions-ProofreadPage, 6Wikisource: Page number inclusion on transcluded works is confused or undefined under certain conditions... - https://phabricator.wikimedia.org/T122395#1903320 (10Billinghurst) [13:27:41] 10MediaWiki-extension-requests, 6Editing-Department, 10Possible-Tech-Projects: Support for editable cladograms in wiki markup - https://phabricator.wikimedia.org/T56222#1903322 (10Yurik) Turns out Vega added support for all sorts of radial and cartesian trees and dendrograms about a month ago. See examples... [13:28:01] 7Design, 10UI-Standardization, 10OOjs-UI, 10VisualEditor, and 2 others: Gray for "inactive" elements in the toolbar is too light - https://phabricator.wikimedia.org/T89271#1903323 (10Elitre) >>! In T89271#1891535, @Quiddity wrote: >>>! In T89271#1891365, @violetto wrote: >> ping @quiddity @elitre, want to... [13:29:39] (03PS5) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [13:30:40] 10CirrusSearch, 6Discovery, 7Documentation: Update user-facing documentation to remove reference to old parameter "srbackend=LuceneSearch", as the parameter no longer does anything on Wikimedia wikis running CirrusSearch - https://phabricator.wikimedia.org/T122288#1903324 (10Aklapper) documentation => #docum... [13:31:40] 10MediaWiki-extensions-MultimediaViewer: ror: could not load image from https://upload.wikimedia.org/wikipedia/commons/thumb/7/72/StadionJamor.JPG/1024px-StadionJamor.JPG URL: https://pt.wikipedia.org/wiki/Lista_de_est%C3%A1dios_de_futebol_de_Portugal#/media/File... - https://phabricator.wikimedia.org/T122318#1903326 [13:33:18] 6Developer-Relations, 3DevRel-December-2015, 3DevRel-January-2016: Get materials for FOSDEM stand - https://phabricator.wikimedia.org/T122374#1903328 (10Aklapper) [13:34:31] 10MediaWiki-extensions-ContentTranslation, 7Easy: X icon in MT card is not well aligned - https://phabricator.wikimedia.org/T122282#1903330 (10Nikerabbit) The space for the icon is reserved with padding, so it doesn't have its own enclosing element. Plainly centering the icon won't work in this case. [13:37:47] 10MediaWiki-extensions-MultiBoilerplate, 5Patch-For-Review: Allow wikilinks in boilerplate page - https://phabricator.wikimedia.org/T43787#1903331 (10Aklapper) >>! In T43787#1902835, @FreedomFighterSparrow wrote: > @Jhf2442, I uploaded a new changeset (somehow Gerrit hasn't notified Phabricator yet): It won't.... [13:39:20] 10MediaWiki-File-management, 6Commons, 6Multimedia: Allow CSV file downloads - https://phabricator.wikimedia.org/T122358#1903332 (10Aklapper) [13:41:39] 10MediaWiki-extensions-UniversalLanguageSelector, 5Patch-For-Review, 7Performance: Performance analysis of ULS "Compact Interlanguage links" feature - https://phabricator.wikimedia.org/T122341#1903334 (10Nikerabbit) Thanks a lot for this analysis. Do you think fixing all of these would make the performance a... [13:43:57] (03PS6) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [13:44:33] 10MediaWiki-extensions-UniversalLanguageSelector, 10ULS-CompactLinks, 5Patch-For-Review, 7Performance: Performance analysis of ULS "Compact Interlanguage links" feature - https://phabricator.wikimedia.org/T122341#1903340 (10Aklapper) [13:45:30] 6Phabricator, 15User-JAufrecht: Set up Community Tech to default all tasks to New Work - https://phabricator.wikimedia.org/T122240#1903341 (10Aklapper) >>! In T122240#1901830, @JAufrecht wrote: > A custom sql shim in phlogiston. That seems to have nothing to do with Phabricator, hence removing that project fr... [13:47:41] (03CR) 10Jdlrobson: [C: 04-1] "Could we not just stop mobile.loggingSchemas from loading?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260913 (https://phabricator.wikimedia.org/T121946) (owner: 10Bmansurov) [13:48:35] 10QuickSurveys, 3Reading-Web-Sprint-63-Ellip…: Validate and handle a survey's platform - https://phabricator.wikimedia.org/T119367#1903344 (10Jdlrobson) What's going on with this @jhobs? I have seen no movement here all week so not sure how I can help... [13:48:54] (03PS7) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [13:49:37] 10MediaWiki-General-or-Unknown, 7Tracking: Deprecate legacy ajax interface (tracking) - https://phabricator.wikimedia.org/T42787#1903345 (10Aklapper) [13:49:46] 10MediaWiki-General-or-Unknown, 7Technical-Debt, 7Tracking: Deprecate legacy ajax interface (tracking) - https://phabricator.wikimedia.org/T42787#457299 (10Aklapper) [13:49:57] (03CR) 10Jdlrobson: "recheck" [extensions/RelatedArticles] (dev) - 10https://gerrit.wikimedia.org/r/260896 (https://phabricator.wikimedia.org/T120715) (owner: 10Bmansurov) [13:50:20] 10MediaWiki-General-or-Unknown, 7JavaScript, 7Technical-Debt: Remove AjaxDispatcher from MediaWiki core - https://phabricator.wikimedia.org/T42786#1903347 (10Aklapper) [13:53:19] 10MediaWiki-extensions-GWToolset, 6Commons, 6Multimedia: "HTTP request timed out" for large files - https://phabricator.wikimedia.org/T86024#1903348 (10Aklapper) p:5Unbreak!>3High > Because of that bug we have GLAM partners stuck in their upload processes. More information welcome. [13:53:31] 3Reading-Web-Sprint-63-Ellip…: Create user page link seems dead - https://phabricator.wikimedia.org/T122376#1903350 (10Jdlrobson) 5Open>3Resolved [13:53:59] 10MediaWiki-extensions-UniversalLanguageSelector, 10ULS-CompactLinks, 5Patch-For-Review, 7Performance: Performance analysis of ULS "Compact Interlanguage links" feature - https://phabricator.wikimedia.org/T122341#1901894 (10KartikMistry) Thank you, @krinkle [13:54:00] 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…, 5WMF-deploy-2016-01-12_(1.27.0-wmf.10): [Regression] No talk or language button in beta - https://phabricator.wikimedia.org/T122233#1903357 (10Jdlrobson) 5Open>3Resolved yup! thanks guys! [13:54:40] 6Phabricator: Remove workboard from #revscoring - https://phabricator.wikimedia.org/T122253#1903361 (10Aklapper) >>! In T122253#1901003, @Halfak wrote: > Is it not possible *at all* to delete the workboard via the UI? No. See link in T122253#1900925. The other option is to fix T89865. [14:00:07] hello! where can I find the code sniffer ruleset and coding styles for download? [14:01:05] (03CR) 10Jdlrobson: [C: 04-1] "Im not a 100% sure if this will show just to autoconfirmed users - on my local wiki it shows to everyone even anons which was not the inte" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [14:02:20] 10MediaWiki-extensions-GWToolset, 6Commons, 6Multimedia: "HTTP request timed out" for large files - https://phabricator.wikimedia.org/T86024#1903374 (10Kelson) >>! In T86024#1903348, @Aklapper wrote: >> Because of that bug we have GLAM partners stuck in their upload processes. > > More information welcome.... [14:02:47] (03PS4) 10Jdlrobson: Hygiene: Remove skins.minerva.beta.images module and other icon cruft [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260871 [14:03:51] (03PS8) 10MtDu: Convert wikihiero to use extension registration [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/260904 (https://phabricator.wikimedia.org/T87995) [14:04:04] (03CR) 10Florianschmidtwelzow: [C: 04-1] Show talk pages to users with fewer than 5 edits (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [14:07:19] 10MediaWiki-Database, 7Wikimedia-log-errors, 7user-notice: Query Special Pages periodically fail. - https://phabricator.wikimedia.org/T113721#1903375 (10IKhitron) 5Resolved>3Open [14:07:45] 10MediaWiki-Database, 7Wikimedia-log-errors, 7user-notice: Query Special Pages periodically fail. - https://phabricator.wikimedia.org/T113721#1674687 (10IKhitron) Same same. I think you have a problem with a timer. [14:10:19] 10MediaWiki-extensions-ProofreadPage, 6Wikisource: Page number inclusion on transcluded works is confused or undefined under certain conditions... - https://phabricator.wikimedia.org/T122395#1903377 (10Billinghurst) it appears to me that you are trying to take page numbering outside the basic concept of the nu... [14:10:22] (03CR) 10jenkins-bot: [V: 04-1] Hygiene: Remove skins.minerva.beta.images module and other icon cruft [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260871 (owner: 10Jdlrobson) [14:14:09] (03PS1) 10Paladox: Update grunt-jsonlint to 1.0.7 [core] - 10https://gerrit.wikimedia.org/r/260921 [14:18:05] 10MediaWiki-Uploading, 6Commons, 6Multimedia, 7JavaScript, 7Voice & Tone: Large amounts of unwanted files (mostly copyvios) uploaded via cross-wiki upload tool (A/B test of different upload interfaces) - https://phabricator.wikimedia.org/T120867#1903378 (10Yann) Many users do not understand what "own" wo... [14:22:09] 10MediaWiki-extensions-RelatedArticles, 6Discovery, 3Discovery-Cirrus-Sprint, 7Elasticsearch, and 3 others: RelatedArticles API query can take 2 seconds - https://phabricator.wikimedia.org/T121254#1903383 (10dcausse) I think there are multiple ways to cache and/or optimize this query: **Caching** 1. Use t... [14:22:45] (03CR) 10jenkins-bot: [V: 04-1] Update grunt-jsonlint to 1.0.7 [core] - 10https://gerrit.wikimedia.org/r/260921 (owner: 10Paladox) [14:23:59] 10Wikimedia-Developer-Summit-2016-Organization: Define the list of "must have" sessions for WikiDev '16 - https://phabricator.wikimedia.org/T119593#1903384 (10Qgil) Why fighting with wikitext markup when VisualEditor offers such good support for editing content in tables. Easier than here in Phabricator, and in... [14:24:07] (03CR) 10Paladox: "recheck" [core] - 10https://gerrit.wikimedia.org/r/260921 (owner: 10Paladox) [14:30:32] (03PS1) 10Sbisson: Browser tests: look for 'talk' instead of 'Talk' [extensions/Flow] - 10https://gerrit.wikimedia.org/r/260927 [14:34:05] 10CirrusSearch, 6Discovery, 3Discovery-Cirrus-Sprint: Integrate popularity score into completion suggester and full text search scoring - https://phabricator.wikimedia.org/T120796#1903386 (10dcausse) a:3dcausse [14:36:40] 10MediaWiki-extension-requests, 6Editing-Department, 10Possible-Tech-Projects: Support for editable cladograms in wiki markup - https://phabricator.wikimedia.org/T56222#1903391 (10Milimetric) +1 to allowing an SVG-rendering option. [14:37:11] (03PS5) 10Jdlrobson: Hygiene: Remove skins.minerva.beta.images module and other icon cruft [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260871 [14:40:12] 10MediaWiki-extension-requests, 6Editing-Department, 10Possible-Tech-Projects: Support for editable cladograms in wiki markup - https://phabricator.wikimedia.org/T56222#1903393 (10Yurik) Related tasks (not blocking since this is an additional functionality): T96309 and T78579 [14:40:34] 6Wikisource: Beleg Tâl - https://phabricator.wikimedia.org/T122401#1903396 (10beleg_tal) 3NEW [14:40:51] 6Wikisource: Fatal Exception when creating MathML output - https://phabricator.wikimedia.org/T122401#1903405 (10beleg_tal) [14:41:07] 10MediaWiki-extensions-RelatedArticles: Create a message group for translation - https://phabricator.wikimedia.org/T122402#1903407 (10Danmichaelo) 3NEW [14:42:56] (03PS15) 10Jdlrobson: Hygiene: Image ResourceLoader modules move [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/244816 [14:47:04] 10MediaWiki-extensions-MultimediaViewer, 5Patch-For-Review, 3Reading-Web-Sprint-63-Ellip…: MultimediaViewer browser tests should run on a per commit basis - https://phabricator.wikimedia.org/T113620#1903427 (10Jdlrobson) Hmm can't replicate locally but see there is a php warning on the file page https://inte... [14:47:51] 10MediaWiki-extensions-ProofreadPage, 6Wikisource: Page number inclusion on transcluded works is confused or undefined under certain conditions... - https://phabricator.wikimedia.org/T122395#1903429 (10Aklapper) Please do use a respectful tone. See the [[ https://www.mediawiki.org/wiki/Bug_management/Phabricat... [14:56:01] 10Phabricator-Sprint-Extension: Auto calculate story points from subtasks - https://phabricator.wikimedia.org/T122404#1903430 (10sougat818) 3NEW [14:58:33] 6Discovery, 10Maps: [WikiMaps Warper] Failure on "Edit tab" (Sorry you do not have permission to view that.) - https://phabricator.wikimedia.org/T121344#1903439 (10Chippyy) Hello, Folks - we are tracking tickets etc on GitHub currently https://github.com/wikimaps-dev/mapwarper/issues We use Phabricator for... [14:59:11] 6Discovery, 10Maps, 7Maps-Style: Country borders are not rendered in v5 - https://phabricator.wikimedia.org/T122405#1903440 (10Yurik) 3NEW a:3MaxSem [14:59:47] 6Discovery, 10Maps: [Wikimaps Warper] Problem with file names with an apostrophe character - https://phabricator.wikimedia.org/T121346#1903449 (10Chippyy) Hello, Folks - we are tracking tickets etc on GitHub currently https://github.com/wikimaps-dev/mapwarper/issues We use Phabricator for issues relating to... [15:00:20] 6Discovery, 10Maps: [Wikimaps Warper] Edit tab is not available for maps uploaded by other users - https://phabricator.wikimedia.org/T121345#1903450 (10Chippyy) Hello, Folks - we are tracking tickets etc on GitHub currently https://github.com/wikimaps-dev/mapwarper/issues We use Phabricator for issues relat... [15:03:43] 6Discovery, 10Maps: [WikiMaps Warper] Failure on "Edit tab" (Sorry you do not have permission to view that.) - https://phabricator.wikimedia.org/T121344#1903459 (10Yurik) @chippyy, but it seems there is no appropriate project here in Phabricator for the MapsWrapper, and it might need to be [[ https://www.media... [15:06:30] 10MediaWiki-extensions-ConfirmEdit-(CAPTCHA-extension), 3Google-Code-In-2015: ReCaptchaNoCaptcha always in English - https://phabricator.wikimedia.org/T122300#1903460 (10alex-mashin) >>! In T122300#1900937, @Aklapper wrote: > Wondering how it can gather the wiki user language (API) in case a contributor would... [15:07:18] (03PS1) 10Paladox: Update grunt-jsonlint to 1.0.7 [skins/Metrolook] - 10https://gerrit.wikimedia.org/r/260929 [15:08:14] (03CR) 10Paladox: [C: 032] Update grunt-jsonlint to 1.0.7 [skins/Metrolook] - 10https://gerrit.wikimedia.org/r/260929 (owner: 10Paladox) [15:09:19] (03Merged) 10jenkins-bot: Update grunt-jsonlint to 1.0.7 [skins/Metrolook] - 10https://gerrit.wikimedia.org/r/260929 (owner: 10Paladox) [15:09:29] 6Discovery, 10Maps: [WikiMaps Warper] Failure on "Edit tab" (Sorry you do not have permission to view that.) - https://phabricator.wikimedia.org/T121344#1903461 (10Yurik) To clarify - we can either create a new project and reassign these issues there, or we could simply close the issue with the link to github... [15:14:24] (03PS8) 10Anomie: Add missing API i18n message, and override DB for WMF Jenkins [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/260397 (https://phabricator.wikimedia.org/T122007) [15:14:47] 6Discovery, 10Maps, 7Maps-Style: Admin borders are not generated - https://phabricator.wikimedia.org/T122405#1903470 (10Yurik) [15:19:50] 6Discovery, 10Maps, 7Maps-Style: Admin borders are not generated - https://phabricator.wikimedia.org/T122405#1903474 (10Yurik) Yep, I just realized we never ran the `select populate_admin();` after the import. Should we re-run it on every update? Separate cron job once a month? [15:20:07] 10Wikimedia-Site-Requests: Change title or sitename and logo for gu.wiktionary - https://phabricator.wikimedia.org/T122407#1903475 (10BhatakatiAatma) 3NEW [15:20:34] (03PS2) 10Ebe123: Show talk pages to users with fewer than 5 edits [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) [15:21:06] (03CR) 10Ebe123: Show talk pages to users with fewer than 5 edits (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [15:24:00] (03CR) 10Jdlrobson: [C: 031] "WFM but commit message should be updated before merging" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [15:25:34] 10Wikimedia-Site-Requests: Change title or sitename and logo for gu.wiktionary - https://phabricator.wikimedia.org/T122407#1903486 (10BhatakatiAatma) [15:27:28] 6Phabricator: Phabricator Task Daemon has problems - https://phabricator.wikimedia.org/T122408#1903488 (10Luke081515) 3NEW [15:29:50] 6Phabricator: Phabricator Task Daemon has problems - https://phabricator.wikimedia.org/T122408#1903495 (10Luke081515) (Adding phabricator admins, hope there is someone with console access) [15:31:28] (03CR) 10jenkins-bot: [V: 04-1] Show talk pages to users with fewer than 5 edits [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [15:32:11] (03PS3) 10Florianschmidtwelzow: Show talk pages to users with fewer than 5 edits [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [15:32:22] (03PS4) 10Florianschmidtwelzow: Show talk pages to logged in users with fewer than 5 edits [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [15:32:28] (03CR) 10Florianschmidtwelzow: [C: 032] Show talk pages to logged in users with fewer than 5 edits [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/260889 (https://phabricator.wikimedia.org/T122311) (owner: 10Ebe123) [15:32:46] 6Phabricator: Phabricator Task Daemon has problems - https://phabricator.wikimedia.org/T122408#1903497 (10Luke081515) [15:33:19] 10MediaWiki-Interface, 7Accessibility, 7Parsoid-DOM: Wrap MediaWiki footer to HTML5