[00:00:03] (03CR) 10Reedy: [C: 032] Replace makeKnownLinkObj function calls with Linker::link [extensions/DPLforum] - 10https://gerrit.wikimedia.org/r/262042 (owner: 10Florianschmidtwelzow) [00:00:18] FlorianSW: I already did it [00:00:19] oops, no not news [00:00:23] :P [00:00:30] I've 5 commits outstanding ;) [00:00:31] (03Merged) 10jenkins-bot: Replace makeKnownLinkObj function calls with Linker::link [extensions/DPLforum] - 10https://gerrit.wikimedia.org/r/262042 (owner: 10Florianschmidtwelzow) [00:00:45] Wait, I was sooo slow? :D [00:01:03] (03CR) 10Florianschmidtwelzow: [C: 032] Remove makeKnownLinkObj() commented out [extensions/DynamicPageList] - 10https://gerrit.wikimedia.org/r/262041 (owner: 10Reedy) [00:01:18] (03CR) 10Florianschmidtwelzow: [C: 032] Remove makeKnownLinkObj() commented out [extensions/News] - 10https://gerrit.wikimedia.org/r/262040 (owner: 10Reedy) [00:01:22] (03CR) 10Reedy: [C: 032] Replace makeKnownLinkObj calls with Linker::link [extensions/DumpHTML] - 10https://gerrit.wikimedia.org/r/262045 (owner: 10Florianschmidtwelzow) [00:01:33] (03Merged) 10jenkins-bot: Remove makeKnownLinkObj() commented out [extensions/DynamicPageList] - 10https://gerrit.wikimedia.org/r/262041 (owner: 10Reedy) [00:01:38] (03CR) 10Florianschmidtwelzow: [C: 032] Replace makeKnownLinkObj() [extensions/MassEditRegex] - 10https://gerrit.wikimedia.org/r/262043 (owner: 10Reedy) [00:01:52] (03CR) 10Florianschmidtwelzow: [C: 032] Replace makeKnownLinkObj() [extensions/Patroller] - 10https://gerrit.wikimedia.org/r/262044 (owner: 10Reedy) [00:01:54] (03Merged) 10jenkins-bot: Remove makeKnownLinkObj() commented out [extensions/News] - 10https://gerrit.wikimedia.org/r/262040 (owner: 10Reedy) [00:01:56] (03Merged) 10jenkins-bot: Replace makeKnownLinkObj calls with Linker::link [extensions/DumpHTML] - 10https://gerrit.wikimedia.org/r/262045 (owner: 10Florianschmidtwelzow) [00:02:11] Gotta wait for gerrit to reindex now :P [00:02:21] (03Merged) 10jenkins-bot: Replace makeKnownLinkObj() [extensions/MassEditRegex] - 10https://gerrit.wikimedia.org/r/262043 (owner: 10Reedy) [00:02:36] (03Merged) 10jenkins-bot: Replace makeKnownLinkObj() [extensions/Patroller] - 10https://gerrit.wikimedia.org/r/262044 (owner: 10Reedy) [00:04:27] But it looks good (I used github and opened the master branch file for every one found) :) [00:04:44] Did we do WYSIWYG? [00:05:04] yep [00:05:04] https://github.com/wikimedia/mediawiki-extensions-WYSIWYG/commit/b3e81952dc7e03cd59f35eebabaa1cd601609b70 [00:05:20] Wheee [00:05:41] I think, this is good to go :) [00:05:51] (03CR) 10Florianschmidtwelzow: [C: 032] "*yay*" [core] - 10https://gerrit.wikimedia.org/r/262019 (owner: 10Reedy) [00:06:03] Yeah, we've certainly done best efforts to clean up the remaining [00:06:20] 10MediaWiki-Categories, 5Patch-For-Review: Category pages causes articles to list in one row and the "letter" in another - https://phabricator.wikimedia.org/T104541#1912987 (10matmarex) >>! In T104541#1912889, @Nnemo wrote: > Maybe putting the "h2" inside the "ul"? Is that allowed? It's probably not, but it w... [00:07:18] (03CR) 10Florianschmidtwelzow: Linker::makeLinkObj() was removed (031 comment) [core] - 10https://gerrit.wikimedia.org/r/262021 (owner: 10Reedy) [00:07:20] Reedy: ^ [00:07:26] this should be Linker::link, right? [00:07:39] lolol [00:07:40] Yup [00:07:50] let me fix [00:09:14] (03CR) 10Bartosz Dziewoński: "We should never include prefixed properties without testing them in the particular browser they're aimed at. That's the whole point of pre" [core] - 10https://gerrit.wikimedia.org/r/261723 (https://phabricator.wikimedia.org/T104541) (owner: 10Bartosz Dziewoński) [00:09:24] Sweet [00:09:27] So 13 in includes [00:09:34] I've got 2 to be merged [00:09:38] You've got one when we've done the WMF config [00:09:41] So that's 10 [00:09:52] class FSRepo extends FileRepo { [00:09:58] Have to ask AaronSchulz what to do about that one... [00:10:23] FlorianSW: So we've just wfMsg*() in GlobalFunctions to deal with now :) [00:10:53] 60 usages of wfMsgHtml :( [00:11:03] That sounds very productive :P We could use: https://phabricator.wikimedia.org/T70750 which I've updated just some time ago :) [00:11:04] 126 of wfMsgExt [00:11:33] wfMsgGetKey is just used internally (yay) [00:12:03] I don't quite know about fixing these up [00:12:05] I would say: Just do it :) I'll take MassEditRegex as my first candidate :) [00:12:28] or would you do it the other way around to go function based, not extension.based? [00:12:34] Doesn't really matter [00:12:40] Either works towards the goal :) [00:12:47] that's right :P [00:13:02] I would do it extension-based, if I already checked it out and working on it :) [00:13:02] I don't know how to fix these up [00:13:31] Aha, wfMsgForContentNoTrans only has one extension usage [00:13:43] Mostly this helps: https://www.mediawiki.org/wiki/Manual:Messages_API#Help_with_replacing_deprecated_wfMsg.2A_functions [00:14:07] Aha, nice [00:14:09] ty [00:14:16] (03PS4) 10Reedy: Linker::makeLinkObj() was removed [core] - 10https://gerrit.wikimedia.org/r/262021 [00:14:18] but that could be a bit trickier, because mostly, the output method doesn't fit, which means, that you need to look, if you use text() or escaped() (mostly these both) [00:14:43] (03Merged) 10jenkins-bot: Linker::makeKnownLinkObj() was removed [core] - 10https://gerrit.wikimedia.org/r/262019 (owner: 10Reedy) [00:16:30] (03PS1) 10Reedy: Replace wfMsgForContentNoTrans() [extensions/Wikilog] - 10https://gerrit.wikimedia.org/r/262046 [00:16:39] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* calls [extensions/MassEditRegex] - 10https://gerrit.wikimedia.org/r/262047 (https://phabricator.wikimedia.org/T70750) [00:16:41] InterwikiIntegration [00:21:53] (03PS1) 10Reedy: wfMsgForContentNoTrans() was removed [core] - 10https://gerrit.wikimedia.org/r/262048 [00:22:06] (03CR) 10Ricordisamoa: "What about future and third-party MessagePosters which may support HTML only? "The markup supported is implementation-specific"" [core] - 10https://gerrit.wikimedia.org/r/261794 (owner: 10Mattflaschen) [00:22:38] (03PS2) 10Reedy: wfMsgForContentNoTrans() was removed [core] - 10https://gerrit.wikimedia.org/r/262048 (https://phabricator.wikimedia.org/T70750) [00:24:02] (03CR) 10Reedy: [C: 032] Replace wfMsg* calls [extensions/MassEditRegex] - 10https://gerrit.wikimedia.org/r/262047 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:24:32] (03Merged) 10jenkins-bot: Replace wfMsg* calls [extensions/MassEditRegex] - 10https://gerrit.wikimedia.org/r/262047 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:24:47] FlorianSW: https://gerrit.wikimedia.org/r/262021 can go from core now :D [00:25:43] (03CR) 10Florianschmidtwelzow: [C: 032] Linker::makeLinkObj() was removed [core] - 10https://gerrit.wikimedia.org/r/262021 (owner: 10Reedy) [00:25:46] blub :D [00:27:38] can someone give me the url for downloading the source code for mediawiki [00:28:23] neeraj: There are many ways you can do that. But Let me look for a tutorial and give you a link [00:28:42] FlorianSW: How up to date is your grepping? [00:29:21] neeraj: https://www.mediawiki.org/wiki/Gerrit/Tutorial [00:29:38] Reedy: from Dec 29 :/ [00:29:38] neeraj: follow that tutorials well and you will find the answer to your question [00:29:45] neeraj: If you login to gerrit, you can find the mediawiki/core repo, and it will generate a git clone command for you to use. [00:29:51] thank you derik. [00:30:00] Some fixed then... Not most [00:30:40] thank you unicornisaurous. [00:31:22] (03Merged) 10jenkins-bot: Linker::makeLinkObj() was removed [core] - 10https://gerrit.wikimedia.org/r/262021 (owner: 10Reedy) [00:32:46] (03CR) 10Bartosz Dziewoński: "Third-party MessagePosters should better conform to the documented contract ;)" [core] - 10https://gerrit.wikimedia.org/r/261794 (owner: 10Mattflaschen) [00:33:07] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* and friends [extensions/InterwikiIntegration] - 10https://gerrit.wikimedia.org/r/262049 (https://phabricator.wikimedia.org/T70750) [00:33:36] (03CR) 10Florianschmidtwelzow: [C: 032] Replace wfMsgForContentNoTrans() [extensions/Wikilog] - 10https://gerrit.wikimedia.org/r/262046 (owner: 10Reedy) [00:33:39] (03CR) 10jenkins-bot: [V: 04-1] Replace wfMsg* and friends [extensions/InterwikiIntegration] - 10https://gerrit.wikimedia.org/r/262049 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:34:10] (03CR) 10Florianschmidtwelzow: "rebase needed :P" [core] - 10https://gerrit.wikimedia.org/r/262048 (https://phabricator.wikimedia.org/T70750) (owner: 10Reedy) [00:34:15] (03Merged) 10jenkins-bot: Replace wfMsgForContentNoTrans() [extensions/Wikilog] - 10https://gerrit.wikimedia.org/r/262046 (owner: 10Reedy) [00:34:46] (03PS2) 10Florianschmidtwelzow: Replace wfMsg* and friends [extensions/InterwikiIntegration] - 10https://gerrit.wikimedia.org/r/262049 (https://phabricator.wikimedia.org/T70750) [00:34:59] (03CR) 10Florianschmidtwelzow: [C: 04-1] wfMsgForContentNoTrans() was removed [core] - 10https://gerrit.wikimedia.org/r/262048 (https://phabricator.wikimedia.org/T70750) (owner: 10Reedy) [00:36:01] neeraj: what does that word mean? [00:37:07] neeraj: sorry not you [00:39:22] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* calls [extensions/LiveTranslate] - 10https://gerrit.wikimedia.org/r/262050 (https://phabricator.wikimedia.org/T70750) [00:41:43] (03PS3) 10Reedy: wfMsgForContentNoTrans() was removed [core] - 10https://gerrit.wikimedia.org/r/262048 (https://phabricator.wikimedia.org/T70750) [00:42:21] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* calls [extensions/WikivoteMapsYandex] - 10https://gerrit.wikimedia.org/r/262051 (https://phabricator.wikimedia.org/T70750) [00:42:24] FlorianSW: You're fixing commented out code now? :P [00:42:59] I thought about removing it, but there was so many commented out code, so I did this one :D [00:43:07] (03CR) 10Reedy: [C: 032] Replace wfMsg* calls [extensions/LiveTranslate] - 10https://gerrit.wikimedia.org/r/262050 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:43:36] (03Merged) 10jenkins-bot: Replace wfMsg* calls [extensions/LiveTranslate] - 10https://gerrit.wikimedia.org/r/262050 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:43:55] (03CR) 10Reedy: [C: 031] Remove Language::armourMath() and friends [core] - 10https://gerrit.wikimedia.org/r/262004 (owner: 10Florianschmidtwelzow) [00:45:09] (03CR) 10Reedy: [C: 032] Replace wfMsg* calls [extensions/WikivoteMapsYandex] - 10https://gerrit.wikimedia.org/r/262051 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:45:43] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* and friends [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262052 (https://phabricator.wikimedia.org/T70750) [00:47:35] (03CR) 10Reedy: [C: 032] Replace wfMsg* and friends [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262052 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:48:02] (03Merged) 10jenkins-bot: Replace wfMsg* calls [extensions/WikivoteMapsYandex] - 10https://gerrit.wikimedia.org/r/262051 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:48:04] (03Merged) 10jenkins-bot: Replace wfMsg* and friends [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262052 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:48:15] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* calls [extensions/CustomUserSignup] - 10https://gerrit.wikimedia.org/r/262053 (https://phabricator.wikimedia.org/T70750) [00:49:38] (03PS1) 10Reedy: Fixup authors. Remove email from wgExtensionCredits [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262054 [00:50:30] (03CR) 10Reedy: [C: 032] Replace wfMsg* calls [extensions/CustomUserSignup] - 10https://gerrit.wikimedia.org/r/262053 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:50:44] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* calls [extensions/SecurePasswords] - 10https://gerrit.wikimedia.org/r/262055 (https://phabricator.wikimedia.org/T70750) [00:50:46] (03CR) 10Reedy: [C: 032] Replace wfMsg* and friends [extensions/InterwikiIntegration] - 10https://gerrit.wikimedia.org/r/262049 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:50:59] (03Merged) 10jenkins-bot: Replace wfMsg* calls [extensions/CustomUserSignup] - 10https://gerrit.wikimedia.org/r/262053 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:51:28] (03Merged) 10jenkins-bot: Replace wfMsg* and friends [extensions/InterwikiIntegration] - 10https://gerrit.wikimedia.org/r/262049 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:51:40] Is the messageMap in ApiBase still used? If I am adding an API error message, should I use the i18n system or should I add to the messagemap? [00:51:51] *still used as in used in new things [00:52:04] (03CR) 10VolkerE: Implement skip-to links and importing mediawiki.stateful-classes.less (032 comments) [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/237870 (https://phabricator.wikimedia.org/T112308) (owner: 10VolkerE) [00:54:00] (03CR) 10Reedy: [C: 032] Replace wfMsg* calls [extensions/SecurePasswords] - 10https://gerrit.wikimedia.org/r/262055 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:54:12] Reedy: can you take a look at: https://github.com/wikimedia/mediawiki/blob/6b1642ca6b6ddb73ffbdc962aeb9e26c47569fd0/includes/OutputPage.php#L1776 this private member seems to be unused, right? [00:54:29] (03Merged) 10jenkins-bot: Replace wfMsg* calls [extensions/SecurePasswords] - 10https://gerrit.wikimedia.org/r/262055 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [00:54:43] OH-: I'd suggest asking anomie [00:55:25] FlorianSW: Looks to be, yup [00:58:20] (03PS1) 10Florianschmidtwelzow: Remove not used private member variable mParserWarnings from OutputPage [core] - 10https://gerrit.wikimedia.org/r/262056 [00:59:34] OH-: i think it is, sadly [01:00:22] OH-: i think you can write your code not to rely on it, by using ApiMessage rather than regular Message. i did something with this in TitleBlacklist once [01:04:12] 10UI-Standardization, 7Accessibility: Improve color palette to comply with WCAG 2.0 min level AA and color-deficiency-friendly. - https://phabricator.wikimedia.org/T109915#1913004 (10Volker_E) @violetto I want to add one little thing here, I'd rather prefer a clearly stated (hex) color code, implementable in p... [01:06:57] (03CR) 10Paladox: "But there were feature backports in the mw 1.25 release." [core] (REL1_26) - 10https://gerrit.wikimedia.org/r/261597 (https://phabricator.wikimedia.org/T119766) (owner: 10Paladox) [01:08:32] 10MediaWiki-extensions-OpenStackManager, 6Labs, 10Labs-Infrastructure, 5Patch-For-Review: Cannot remove all Puppet classes from a Labs instance - https://phabricator.wikimedia.org/T122733#1913009 (10yuvipanda) I've manually removed this class from LDAP to unblock Luke. [01:10:07] Interesting, I'll take a look. Thanks [01:15:41] 10MediaWiki-extensions-BlogPage, 10MediaWiki-extensions-BookManager, 10MediaWiki-extensions-CustomUserSignup, 10MediaWiki-extensions-EditUser, and 20 others: [Various] Deprecated: Use of wfMsg* was deprecated in MediaWiki 1.18 (notices in 1.21) - https://phabricator.wikimedia.org/T70750#1913018 (10Reedy) [01:15:44] 10MediaWiki-General-or-Unknown, 7Tracking: Pre MW 1.20 deprecation cleanup - https://phabricator.wikimedia.org/T122754#1913020 (10Reedy) [01:15:48] 10MediaWiki-extensions-BlogPage, 10MediaWiki-extensions-BookManager, 10MediaWiki-extensions-CustomUserSignup, 10MediaWiki-extensions-EditUser, and 20 others: [Various] Deprecated: Use of wfMsg* was deprecated in MediaWiki 1.18 (notices in 1.21) - https://phabricator.wikimedia.org/T70750#1191016 (10Reedy) [01:16:04] 10MediaWiki-General-or-Unknown, 7Tracking, 7WorkType-Maintenance: Pre MW 1.20 deprecation cleanup - https://phabricator.wikimedia.org/T122754#1913011 (10Reedy) [01:16:50] FlorianSW: I see you and Reedy are busy removing the use of the wfmsg stuff. [01:17:16] 10MediaWiki-General-or-Unknown, 7WorkType-Maintenance: Cleanup pre MW 1.20 deprecations in Wikibits.js - https://phabricator.wikimedia.org/T122755#1913024 (10Reedy) 3NEW [01:17:21] FlorianSW: Is it ok if I work on removing it in another extension? I'm waiting on Nemo to approve something, so I'm kinda bored. :) [01:17:31] MtDu: Yes please, feel free! :D [01:17:47] MtDu: We've moved onto that as we'd done pretty much the rest of the PHP pre 1.20 cleanup [01:17:58] That's (and wikibits) are the biggest issues now [01:18:01] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* and friends [extensions/Wikilog] - 10https://gerrit.wikimedia.org/r/262057 (https://phabricator.wikimedia.org/T70750) [01:18:22] Reedy: this one was hard :P ^ [01:18:29] (03PS4) 10Reedy: Remove $wgCopyrightIcon [core] - 10https://gerrit.wikimedia.org/r/262001 (https://phabricator.wikimedia.org/T122754) (owner: 10Florianschmidtwelzow) [01:18:51] (03PS2) 10Reedy: Remove $wgCopyrightIcon [extensions/Configure] - 10https://gerrit.wikimedia.org/r/262000 (https://phabricator.wikimedia.org/T122754) (owner: 10Florianschmidtwelzow) [01:19:02] MtDu: yes, yes, yes! It would be nice, if you post the name of the extension you're currently working on here in irc, so we don't end up doing it twice :P [01:19:03] (03CR) 10Reedy: [C: 032] Remove $wgCopyrightIcon [extensions/Configure] - 10https://gerrit.wikimedia.org/r/262000 (https://phabricator.wikimedia.org/T122754) (owner: 10Florianschmidtwelzow) [01:19:17] (03CR) 10jenkins-bot: [V: 04-1] Remove $wgCopyrightIcon [core] - 10https://gerrit.wikimedia.org/r/262001 (https://phabricator.wikimedia.org/T122754) (owner: 10Florianschmidtwelzow) [01:19:20] FlorianSW: Currently working on IndexFunction [01:19:29] (03CR) 10Yuvipanda: [C: 032] "<3" [extensions/OpenStackManager] - 10https://gerrit.wikimedia.org/r/261906 (https://phabricator.wikimedia.org/T122733) (owner: 10Alex Monk) [01:19:31] (03PS2) 10Florianschmidtwelzow: Fixup authors. Remove email from wgExtensionCredits [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262054 (owner: 10Reedy) [01:19:35] (03CR) 10Florianschmidtwelzow: [C: 032] Fixup authors. Remove email from wgExtensionCredits [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262054 (owner: 10Reedy) [01:19:58] ImageLink is the next one [01:20:01] FlorianSW: Quick question. Why didn't you ->text() on line 15? https://gerrit.wikimedia.org/r/#/c/262052/1/WhoIsWatching_body.php Could you explain when not to ->text()? [01:20:07] (03Merged) 10jenkins-bot: Remove $wgCopyrightIcon [extensions/Configure] - 10https://gerrit.wikimedia.org/r/262000 (https://phabricator.wikimedia.org/T122754) (owner: 10Florianschmidtwelzow) [01:20:38] MtDu: setPageTitle() takes the whole Message object :) It's documented, wait... [01:20:46] (03Merged) 10jenkins-bot: NovaPrivateHost: Ensure puppetclass/puppetvar LDAP attributes get modified [extensions/OpenStackManager] - 10https://gerrit.wikimedia.org/r/261906 (https://phabricator.wikimedia.org/T122733) (owner: 10Alex Monk) [01:20:49] https://doc.wikimedia.org/mediawiki-core/master/php/classOutputPage.html#a62445ed80350b6e3432d92d3986ddaf9 [01:21:11] (03Merged) 10jenkins-bot: Fixup authors. Remove email from wgExtensionCredits [extensions/WhoIsWatching] - 10https://gerrit.wikimedia.org/r/262054 (owner: 10Reedy) [01:21:18] it takes a string or a message object, so I simply pass the Message object and let OutputPage decide, what output method to use :) [01:21:20] MtDu: ^ [01:21:38] FlorianSW: Ok. [01:21:41] 10MediaWiki-General-or-Unknown, 7JavaScript, 7WorkType-Maintenance: Cleanup pre MW 1.20 deprecations in Wikibits.js - https://phabricator.wikimedia.org/T122755#1913041 (10Reedy) [01:22:54] FlorianSW: Is it ok if I just use wfmessage, or would it be better to use $this->msg where applicable? [01:23:34] The latter is better still [01:23:47] If it's in a special page, you should [01:24:27] (03PS1) 10Florianschmidtwelzow: Replace some wfMsg* calls [extensions/ImageLink] - 10https://gerrit.wikimedia.org/r/262058 (https://phabricator.wikimedia.org/T70750) [01:24:40] Reedy: Gotcha. Will do. [01:24:54] Reedy: Also, do you want me to replace wgout and others? [01:25:04] You don't have to :) [01:25:16] It's extra work that is nice, but unnecessary [01:25:31] We're mostly focusing on stuff so we can remove it from MW core [01:25:40] Reedy: Like Reedy said. the advantage of $this->msg is, that you get a localized message object (with the user language), not in the content language. But when I see such old extensions, I only use $this->msg() is it's obvious :P [01:25:47] Reedy: Gotcha. [01:25:51] FlorianSW: Ok. [01:26:11] MatmaRex: Any idea bout cleaning up the shit in wikibits? :P [01:26:15] FlorianSW: Will do that. I'm trying to spam some msges so I can read your whole message. This stupid windows notification is stuck there. [01:26:47] Doing SemanticMediaWiki now [01:27:00] Reedy: ori and Krinkle have some on-wiki usage logs of it somewhere [01:27:12] as in, on-wiki usage. [01:27:13] Is that most of the issues? Onwiki usages? [01:27:15] not on-wiki logs. [01:27:24] FlorianSW: I'm using wfmessage where it's static and an object isn't passed in. [01:27:30] Reedy: i think so. never really cared to check. :P [01:27:56] MatmaRex: ugh, fucking seriously [01:27:59] 52 usages of is_gecko onwiki [01:28:23] i know, right? [01:28:38] 10MediaWiki-General-or-Unknown, 7JavaScript, 7WorkType-Maintenance: Cleanup pre MW 1.20 deprecations in Wikibits.js - https://phabricator.wikimedia.org/T122755#1913045 (10Reedy) [01:28:45] BREAK IT ALL [01:28:56] MtDu: correctly (but you can use msg() only, where it is defined :)) [01:29:10] Reedy: try the latter ones. importScriptURI surely has thousands [01:29:27] Doing CongressLookup now [01:29:34] and importScript has so many, we undeprecated it :P [01:29:35] We should be able to fix a lot of these using a bot type of thing [01:29:55] 532 of importScriptURI [01:30:04] 1387 importScript [01:30:04] true. want to write one? :D [01:30:16] I'd just use AWB :P [01:30:30] Except, it doesn't do crosswiki so well [01:30:35] Reedy: AWB is a pain when you need to edit 800 different wikis [01:30:37] yeah, exactly [01:30:53] Well, scripting something to use AWB wouldn't be too bad [01:30:56] Doing it programatically [01:31:06] heh [01:31:07] Reuse it's ApiEdit stuff [01:31:27] And/or, searching for all of the stuff via mwgrep [01:31:33] Making a unique list of pages per wiki [01:31:40] And edit them using a set of defined replacements/regexs [01:31:52] AWB should follow interwiki prefixes in pages added to the list. gogogo do it [01:32:12] heh [01:32:14] great weekend project for before the dev summit [01:32:26] I'm not going to the dev summit :( [01:32:33] no? :( [01:33:48] FlorianSW: I use wfMessage in those lines right? https://dpaste.de/Eb7F#L26,39 or can I do $parser->msg? [01:33:49] (03PS1) 10Florianschmidtwelzow: Replace wfMsg* calls [extensions/CongressLookup] - 10https://gerrit.wikimedia.org/r/262060 (https://phabricator.wikimedia.org/T70750) [01:34:25] (03CR) 10Reedy: [C: 032] Replace wfMsg* calls [extensions/CongressLookup] - 10https://gerrit.wikimedia.org/r/262060 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [01:34:27] MtDu: does Parser implement a msg() method? [01:34:59] doing Push now [01:35:01] (03Merged) 10jenkins-bot: Replace wfMsg* calls [extensions/CongressLookup] - 10https://gerrit.wikimedia.org/r/262060 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [01:35:07] (03CR) 10Reedy: [C: 04-1] Replace some wfMsg* calls (031 comment) [extensions/ImageLink] - 10https://gerrit.wikimedia.org/r/262058 (https://phabricator.wikimedia.org/T70750) (owner: 10Florianschmidtwelzow) [01:35:32] FlorianSW: Nope. [01:35:46] FlorianSW: This is helpful. :) [01:35:49] MtDu: then wfMessage is fine :P [01:35:50] FlorianSW: https://doc.wikimedia.org/mediawiki-core/master/php/classParser.html [01:36:00] FlorianSW: Bookmarked. :) [01:36:18] MtDu: the whole doc.wikimedia.org is your friend :P [01:36:34] Since y’all sound like you’re having so much fun fixing deprecated i18n functions, I think I’ll have to join in. [01:38:37] What extensions are left? [01:39:48] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/IndexFunction] - 10https://gerrit.wikimedia.org/r/262061 (https://phabricator.wikimedia.org/T70750) [01:41:12] FlorianSW: ^ Please take a look. :) [01:41:18] Reedy:^ [01:41:33] unicornisaurous: There's a loooooad of them :D [01:41:49] https://phabricator.wikimedia.org/T70750 [01:42:01] Gerrit is really slow so I’m having trouble seeing which are already done [01:42:04] Look in the summary [01:42:14] I suspect we should update the greps [01:43:31] I’m going to do SemanticComments extension since it doesn’t appear to be done [01:43:34] (03PS2) 10MtDu: Replace deprecated wfMsg* functions [extensions/IndexFunction] - 10https://gerrit.wikimedia.org/r/262061 (https://phabricator.wikimedia.org/T70750) [01:43:41] (even though I have no idea what it does) [01:43:55] You're not the only one ;) [01:44:36] (03PS1) 10Florianschmidtwelzow: Replace some wfMsg* calls [extensions/Push] - 10https://gerrit.wikimedia.org/r/262062 [01:44:48] Anyone done wikilog? [01:44:55] MtDu: already done [01:45:08] Doing SecurePoll [01:45:10] 10MediaWiki-General-or-Unknown, 7Easy, 5MW-1.23-release, 5MW-1.25-release, and 3 others: Bail out early if $wgArticlePath is non-absolute - https://phabricator.wikimedia.org/T48998#1913063 (10matmarex) Yeah, probably. [01:45:10] Reedy: updating the grep takes, at least in tool labs, a looooong time :) [01:45:28] Is that hint hint for me to do it? :P [01:45:34] Actually. I'll start from bottom [01:45:43] Doing Newsignuppage [01:46:16] (03CR) 10MtDu: "recheck" [extensions/IndexFunction] - 10https://gerrit.wikimedia.org/r/262061 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [01:46:38] (03CR) 10Florianschmidtwelzow: [C: 032] Replace deprecated wfMsg* functions [extensions/IndexFunction] - 10https://gerrit.wikimedia.org/r/262061 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [01:46:58] Reedy: no, but if it's faster on your pc :P [01:47:01] do it :D [01:47:20] (but no, it wasn't, just as an information) [01:47:28] (03Merged) 10jenkins-bot: Replace deprecated wfMsg* functions [extensions/IndexFunction] - 10https://gerrit.wikimedia.org/r/262061 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [01:47:54] 10MediaWiki-General-or-Unknown, 7JavaScript, 7WorkType-Maintenance: Cleanup pre MW 1.20 deprecations in Wikibits.js - https://phabricator.wikimedia.org/T122755#1913065 (10Reedy) ``` addCheckboxClickHandlers checkboxClickHandler ff2_bugs is_chrome_mac is_chrome is_ff2 is_ff2_win is_ff2_x11 is_opera_preseven i... [01:48:10] I might do it tomorrow when we've stopped [01:48:14] (03PS2) 10Florianschmidtwelzow: Replace some wfMsg* calls [extensions/ImageLink] - 10https://gerrit.wikimedia.org/r/262058 (https://phabricator.wikimedia.org/T70750) [01:48:15] and everything merged [01:48:17] Gonna go to bed soon [01:48:39] (03CR) 10Florianschmidtwelzow: [C: 032] wfMsgForContentNoTrans() was removed [core] - 10https://gerrit.wikimedia.org/r/262048 (https://phabricator.wikimedia.org/T70750) (owner: 10Reedy) [01:49:25] * FlorianSW too [01:50:23] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/NewSignupPage] - 10https://gerrit.wikimedia.org/r/262063 (https://phabricator.wikimedia.org/T70750) [01:51:12] Doing ProofreadPage [01:51:32] (03PS1) 10Bartosz Dziewoński: Validate that $wgVariantArticlePath is absolute, too [core] - 10https://gerrit.wikimedia.org/r/262064 (https://phabricator.wikimedia.org/T48998) [01:52:16] FlorianSW: Patch above. [01:52:19] Reedy:^ [01:52:25] (03CR) 10Bartosz Dziewoński: "Untested. Somebody please check the variable-variables stuff carefully." [core] - 10https://gerrit.wikimedia.org/r/262064 (https://phabricator.wikimedia.org/T48998) (owner: 10Bartosz Dziewoński) [01:53:06] Proofreadpage seems to be done already [01:53:18] (03CR) 10Florianschmidtwelzow: [C: 032] Replace deprecated wfMsg* functions [extensions/NewSignupPage] - 10https://gerrit.wikimedia.org/r/262063 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [01:53:27] Doing premoderation [01:53:56] (03Merged) 10jenkins-bot: wfMsgForContentNoTrans() was removed [core] - 10https://gerrit.wikimedia.org/r/262048 (https://phabricator.wikimedia.org/T70750) (owner: 10Reedy) [01:53:59] (03Merged) 10jenkins-bot: Replace deprecated wfMsg* functions [extensions/NewSignupPage] - 10https://gerrit.wikimedia.org/r/262063 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [01:54:31] (03PS5) 10Florianschmidtwelzow: Remove $wgCopyrightIcon [core] - 10https://gerrit.wikimedia.org/r/262001 (https://phabricator.wikimedia.org/T122754) [01:56:42] (03CR) 10jenkins-bot: [V: 04-1] Validate that $wgVariantArticlePath is absolute, too [core] - 10https://gerrit.wikimedia.org/r/262064 (https://phabricator.wikimedia.org/T48998) (owner: 10Bartosz Dziewoński) [02:01:20] MatmaRex: Ugh, one has over 20k usages :( [02:01:33] think it was write [02:01:45] dang [02:02:35] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/Premoderation] - 10https://gerrit.wikimedia.org/r/262065 (https://phabricator.wikimedia.org/T70750) [02:03:15] FlorianSW: Patch. [02:03:17] Reedy: ^ [02:04:41] Doing createredirect now [02:05:31] Createredirect seems to be done [02:05:42] Doing favorites [02:08:39] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 5Patch-For-Review, 7Wikimedia-log-errors: Call to a member function getSiteLinkList() on a non-object (NULL) in ClientParserOutputDataUpdater - https://phabricator.wikimedia.org/T122747#1913086 (10aude) [02:08:42] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 7Wikimedia-log-errors: [Bug] BadMethodCallException from line 131 of Wikibase/client/includes/ParserOutputDataUpdater.php: Call to a member function getSiteLinkList() on a non-object (NULL) - https://phabricator.wikimedia.org/T118580#1913087 (10aude) [02:08:53] I think I did it right: https://gerrit.wikimedia.org/r/#/c/262066/ [02:09:26] ^ FlorianSW and Reedy [02:10:05] gerrit-wm didn’t say anything I guess [02:10:25] (03CR) 10Florianschmidtwelzow: [C: 04-1] "Some work needed :)" (0321 comments) [extensions/Premoderation] - 10https://gerrit.wikimedia.org/r/262065 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:11:21] oh crap I forgot all about $this->msg [02:13:13] FlorianSW: Sorry. I'm getting careless. I guess this is too exciting. :) [02:13:28] :) [02:17:52] 10MediaWiki-General-or-Unknown, 5Patch-For-Review, 7Tracking, 7WorkType-Maintenance: Pre MW 1.20 deprecation cleanup - https://phabricator.wikimedia.org/T122754#1913089 (10Florian) [02:18:09] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/Favorites] - 10https://gerrit.wikimedia.org/r/262067 (https://phabricator.wikimedia.org/T70750) [02:18:14] FlorianSW: I finished Favorites first. There it is. [02:18:27] 10MediaWiki-General-or-Unknown, 7JavaScript, 7WorkType-Maintenance: Cleanup pre MW 1.20 deprecations in Wikibits.js - https://phabricator.wikimedia.org/T122755#1913091 (10Reedy) {F3196199} {F3196200} {F3196201} {F3196202} {F3196203} {F3196204} {F3196205} {F3196206} {F3196207} {F3196208} {F3196209}... [02:19:16] FlorianSW: So on https://gerrit.wikimedia.org/r/#/c/262065/1/SpecialPremoderation.php, you want me to use escaped() on 133, 134, 135, and 136 as well? [02:19:17] (03CR) 10Florianschmidtwelzow: [C: 04-1] Replace deprecated wfMsg* functions (032 comments) [extensions/Favorites] - 10https://gerrit.wikimedia.org/r/262067 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:19:33] MtDu: yep [02:20:26] FlorianSW: New changeset. I guess this repo isn’t on gerrit-wm [02:21:09] unicornisaurous, it's probably redirected to some other IRC channel [02:23:12] * FlorianSW fall into bed tired [02:23:17] * falls [02:23:23] Can you approve my task first? :P [02:24:02] Thanks! :) [02:27:56] Reedy: You still there? [02:28:08] Yup, doing some Python :P [02:29:32] (03PS2) 10MtDu: Replace deprecated wfMsg* functions [extensions/Premoderation] - 10https://gerrit.wikimedia.org/r/262065 (https://phabricator.wikimedia.org/T70750) [02:29:50] Reedy: Can you still do code reviews? [02:33:24] (03PS2) 10MtDu: Replace deprecated wfMsg* functions [extensions/Favorites] - 10https://gerrit.wikimedia.org/r/262067 (https://phabricator.wikimedia.org/T70750) [02:33:29] Yeah [02:33:33] I need to go to bed soon though [02:33:48] Reedy: Great. Well, let's finish these two, and we can both get some rest. :) [02:34:04] Reedy: The two patches are above. [02:35:15] If I'm not mistaken, all that's left is ReplaceSet and the ones in core, assuming we did all the ones before the one Unicornisaurus did. :) [02:35:52] I can't believe they've all been done tonight ;) [02:37:20] Reedy: Go team! :) [02:37:31] Reedy: I'll go ahead and do replaceset [02:38:22] (03CR) 10Reedy: [C: 032] Replace deprecated wfMsg* functions [extensions/Favorites] - 10https://gerrit.wikimedia.org/r/262067 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:38:37] Reedy: What do you want me to do about this? https://dpaste.de/sCNs [02:38:50] (03Merged) 10jenkins-bot: Replace deprecated wfMsg* functions [extensions/Favorites] - 10https://gerrit.wikimedia.org/r/262067 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:38:56] MtDu: unicornisaurous If we need to do something to give you guys credit for these commits for GCI, poke us tomorrow ;) [02:39:31] Reedy: Florian took care of mine while falling into bed [02:39:37] heh [02:39:45] Thanks though :) [02:40:02] I’m going to do more later probably [02:40:18] Reedy: I've already done this task before. Just wanted to help you guys out. If I get credit for it, well that's great. :) [02:40:31] If we can give you credit, I don't mind doing so [02:40:46] unicornisaurous: I think all we have left is a few of them in core. :) [02:41:20] It should just be the functions in core [02:41:23] wow [02:41:26] The callers mostly should've been fixed [02:41:30] I thought somebody said theres a “bunch” [02:41:47] There was :P [02:41:50] unicornisaurous: Well, we reduced that bunch. :) [02:41:58] MtDu: https://gerrit.wikimedia.org/r/#/c/262059/1/includes/datavalues/SMW_DataValue.php,unified is what FlorianSW|away did for another like that [02:42:21] Reedy: Ok. And Reedy: Ok. That's great. Do I need to poke you tomorrow or what? On the GCI page, it just says I've completed that one. Not sure if it'll let me do it again. [02:42:32] will it not? [02:42:37] That's a bit silly for this sort of task [02:42:43] where it's discretely different [02:43:26] (03CR) 10Reedy: [C: 032] Replace deprecated wfMsg* functions [extensions/Premoderation] - 10https://gerrit.wikimedia.org/r/262065 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:43:39] Reedy: Yeah. I did a task, and Nemo told me that I could do it again if I wanted since there was a lot more left, and I tried to, but it just said completed and gave me my history of submission. I did another task, and tried to go back, but it didn't work. [02:43:54] Reedy: But if it doesn't work out, that's fine. I just wanted to help you guys out. :) [02:43:55] (03Merged) 10jenkins-bot: Replace deprecated wfMsg* functions [extensions/Premoderation] - 10https://gerrit.wikimedia.org/r/262065 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:43:55] Hmm. Maybe we ask him about it tomorrow then :) [02:44:29] (03CR) 10Reedy: [C: 04-1] "Needs rebase..." [extensions/LiveTranslate] - 10https://gerrit.wikimedia.org/r/261761 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [02:46:06] Reedy: For the one I mentioned earlier, do you want me to use escaped()? Look at the Premoderation ones, Florian told me to use escaped. [02:46:17] Sounds fine :) [02:46:38] Gets a little confusing with all the similarilty [02:47:56] Gonna run an grep and see waht state we're in [02:48:01] Know there's a few more unmerged [02:48:45] (03CR) 10Reedy: [C: 032] Replace some wfMsg* calls [extensions/Push] - 10https://gerrit.wikimedia.org/r/262062 (owner: 10Florianschmidtwelzow) [02:49:15] (03Merged) 10jenkins-bot: Replace some wfMsg* calls [extensions/Push] - 10https://gerrit.wikimedia.org/r/262062 (owner: 10Florianschmidtwelzow) [02:49:36] MtDu: unicornisaurous There's quite a few more to go :P [02:50:06] Reedy: Is there a list somewhere? [02:50:11] I'm just making one [02:50:14] Will update the task [02:50:37] Note there's a few false positives from the unmerged tasks [02:50:41] *changes [02:51:24] Maybe nemo will make a “series” of tasks like he did for the translation tasks [02:51:52] unicornisaurus: Lol. Can I like chain submit then? [02:52:06] unicornisaurous: Lol. Can we like chain submit then? [02:52:28] There's possibly a thousand to fix :/ [02:53:13] Wellllllll, he’d probably make the first task only one patch, the second one 5 patches, the third 25, the fourth 125, or something like that [02:53:17] why does it look like some are new [02:53:55] unicornisaurous: Well. Better get busy. :p [02:54:16] Reedy: Really? Dang. [02:54:24] There's maybe 10% or so less on that [02:54:30] Wikilog (which has a patch) has over 100 [02:55:00] Reedy: Ok. [02:55:08] Reedy: And how do I rebase? I've never done that before. [02:55:21] git review -d 12345 [02:55:24] git rebase origin [02:55:27] See what complains :) [02:55:36] git rebase gerrit/master [02:55:56] 10MediaWiki-extensions-BlogPage, 10MediaWiki-extensions-BookManager, 10MediaWiki-extensions-CustomUserSignup, 10MediaWiki-extensions-EditUser, and 20 others: [Various] Deprecated: Use of wfMsg* was deprecated in MediaWiki 1.18 (notices in 1.21) - https://phabricator.wikimedia.org/T70750#1913098 (10Reedy) [02:56:48] Something will be broken if gerrit didn’t do it for you. So then its like resolving a merge conflict. Then just git rebase - -continue [02:57:08] 10MediaWiki-extensions-BlogPage, 10MediaWiki-extensions-BookManager, 10MediaWiki-extensions-CustomUserSignup, 10MediaWiki-extensions-EditUser, and 20 others: [Various] Deprecated: Use of wfMsg* was deprecated in MediaWiki 1.18 (notices in 1.21) - https://phabricator.wikimedia.org/T70750#1692282 (10Reedy) `... [02:57:24] https://phabricator.wikimedia.org/T70750#1913098 [02:57:28] I just added a lot more :P [02:58:15] Reedy: It says Current branch review/mtdu/261761 is up to date. [02:58:27] git checkout master [02:58:28] git pull [02:58:29] then try again [02:58:30] Reedy: When I rebase. [02:59:12] If you’ve already rebased then you’re done [02:59:14] try git review [02:59:24] His master isn't going to be up to date [02:59:29] gerrit is complaing about it can't rebase [02:59:48] I thought he meant after he resolved stuff [03:00:03] Reedy: Justin@Du MINGW64 ~/vagrant/mediawiki/extensions/LiveTranslate (review/mtdu/261761) [03:00:03] $ git rebase origin [03:00:03] Current branch review/mtdu/261761 is up to date. [03:00:04] Justin@Du MINGW64 ~/vagrant/mediawiki/extensions/LiveTranslate (review/mtdu/261761) [03:00:04] $ git checkout master [03:00:04] Switched to branch 'master' [03:00:06] Your branch is ahead of 'origin/master' by 1 commit. [03:00:08] (use "git push" to publish your local commits) [03:00:10] Justin@Du MINGW64 ~/vagrant/mediawiki/extensions/LiveTranslate (master) [03:00:12] $ git pull [03:00:16] remote: Counting objects: 247, done [03:00:18] remote: Finding sources: 100% (7/7) [03:00:20] remote: Total 7 (delta 5), reused 7 (delta 5) [03:00:22] Unpacking objects: 100% (7/7), done. [03:00:24] From https://gerrit.wikimedia.org/r/mediawiki/extensions/LiveTranslate [03:00:26] 672c08a..e78a40c master -> origin/master [03:00:27] paste.de pls [03:00:28] Auto-merging api/ApiImportTranslationMemories.php [03:00:30] Auto-merging LiveTranslate.hooks.php [03:00:32] CONFLICT (content): Merge conflict in LiveTranslate.hooks.php [03:00:34] Automatic merge failed; fix conflicts and then commit the result. [03:00:36] Justin@Du MINGW64 ~/vagrant/mediawiki/extensions/LiveTranslate (master|MERGING) [03:01:35] Need to fix those then :) [03:01:46] I think he pasted too much :) [03:02:01] lol [03:02:16] !pastebin | MtDu [03:02:16] MtDu: To avoid overflowing the channel with inane amounts of text, use https://phabricator.wikimedia.org/paste/create/ or other awesome pastebin sites to share code, errors, and other large texts. [03:02:47] Reedy: Sorry about that text. I forgot to copy the pastebin link. [03:02:55] Did you get the link at the end? [03:02:59] So now its just like resolving merge conflicts [03:03:10] no we did not get the list [03:03:11] *link [03:03:17] https://dpaste.de/NfmP [03:03:53] LiveTranslate.hooks.php: needs merge [03:03:58] Open the file, look for conflict markers [03:05:45] Does he really want to do that? IIRC, he should do a git fetch - -all and then checkout his review branch again for the rebase to gerrit/master. Now he will be dealing with merge conflicts twice I think. (Once to pull into master, then again to rebase) [03:05:50] Reedy ^ [03:06:18] Are the commits ontop of master? [03:06:35] Your branch is ahead of 'origin/master' by 1 commit. [03:06:37] looks like it [03:06:42] git reset HEAD~5 --hard [03:06:43] git pull [03:06:45] git review -d 1234 [03:06:48] git rebase origin [03:07:04] I think a git review from master will suffie [03:07:06] *suffice [03:07:29] Sometimes easier to keep it cleaner [03:07:41] If I commit ontop of master, I reset it away after to save divergence [03:07:46] Reedy: One sec. I restarted machine. [03:07:50] Well usually after I do a rebase I end up resetting everything [03:07:59] (since i break stuff) [03:08:08] Reedy: 4gb ram is not enough sometimes. [03:09:19] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/ReplaceSet] - 10https://gerrit.wikimedia.org/r/262069 (https://phabricator.wikimedia.org/T70750) [03:09:23] * unicornisaurous looks at his 34 Chrome tabs and wonders how MtDu lives with 4GB. [03:09:48] The only time I have anything less than 4GB is headless stuff... [03:09:51] Reedy: Alright. Now that is outta the way. [03:10:05] Reedy: It's ok. I got an ssd. :) [03:10:27] my mediawiki vagrant instance probably has 4 by itself [03:10:29] Saying that, my NAS has 16GB :P [03:10:56] Reedy: So I'm starting by git review -d that patch right? [03:11:10] I'd suggest tidying up your master first [03:11:12] Reedy: Then run those commands you sent me starting at 21:06:35? [03:11:16] Reedy: Ok. [03:11:21] Reedy: How so? [03:11:32] git checkout master [03:11:37] git reset HEAD~5 --hard [03:11:37] git pull [03:12:01] Then fix merge conflicts! Yay! [03:12:02] Reedy: Then download the change? [03:12:08] 10MediaWiki-Export-or-Import, 3Google-Code-In-2015: [Import/export compatibility] Special:Import and importDump.php should ignore unrecognized tags - https://phabricator.wikimedia.org/T43938#1913102 (10TTO) https://codein.withgoogle.com/dashboard/tasks/5520672822394880/ [03:12:10] yeah [03:13:13] 10MediaWiki-Export-or-Import, 7Testme: Export function doesn't work with apache module mod_gzip - https://phabricator.wikimedia.org/T11997#1913105 (10TTO) p:5Low>3Lowest [03:14:04] Reedy: Then I just commit --amend and git review? [03:14:19] did you git review -d 1234 your patch? [03:14:28] Reedy: yep [03:16:26] Reedy:^ and btw, https://gerrit.wikimedia.org/r/#/c/262069 [03:18:53] Did you rebase and sort the conflicts too? [03:19:40] Reedy: No. Can you guide me through it? [03:20:11] Reedy: All I did was checkout master, and then git review -d my patch. [03:20:22] is your master up to date? [03:20:30] (See also: https://www.mediawiki.org/wiki/Gerrit/resolve_conflict) [03:22:46] Reedy: I think so. This is my git bash log. https://dpaste.de/FEXa [03:23:07] so yeah [03:23:08] git rebase origin [03:23:12] then it should complain about conflicts [03:23:27] I think the conflict will be the change Florian did earlier that crashes with yours [03:25:18] Reedy: It does. https://dpaste.de/VuzV [03:25:30] git diff? [03:27:20] Reedy: It doesn't show anything. Want me to git show HEAD? [03:27:46] Why are you on ConventionExtension? [03:28:11] Reedy: Woops, wrong tab [03:28:21] (03CR) 10Reedy: [C: 032] Replace deprecated wfMsg* functions [extensions/ReplaceSet] - 10https://gerrit.wikimedia.org/r/262069 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [03:28:21] Reedy: Was working on that one earlier. [03:28:51] (03Merged) 10jenkins-bot: Replace deprecated wfMsg* functions [extensions/ReplaceSet] - 10https://gerrit.wikimedia.org/r/262069 (https://phabricator.wikimedia.org/T70750) (owner: 10MtDu) [03:29:21] Reedy: Speaking of which, I'm done with actualyl. [03:30:01] 03:30 [03:31:09] Reedy: Here is the dpaste. https://dpaste.de/795O [03:32:04] git rebase origin [03:32:56] Reedy: There we go. https://dpaste.de/BG3R [03:33:05] CONFLICT (content): Merge conflict in LiveTranslate.hooks.php [03:33:11] Yay [03:33:12] Have a look in that file for <<< or >>> [03:34:19] Reedy: Ok. Hold on. Let me push that patch for ConventionExtension before I forget. [03:34:42] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/ConventionExtension] - 10https://gerrit.wikimedia.org/r/262070 (https://phabricator.wikimedia.org/T70750) [03:35:49] 10MediaWiki-Search, 10CirrusSearch, 6Discovery: English Wikipedia search results are incomplete - https://phabricator.wikimedia.org/T122566#1913127 (10MZMcBride) @Deskana: I don't pay much attention to workboards, but regarding "Deskana moved this task to Advanced search syntax on the CirrusSearch workboard.... [03:36:05] Reedy: Can you send me that link again to what Florian did? [03:39:17] Reedy: I'm not sure what to do. https://dpaste.de/1q0F My brain's getting fuzzy. [03:41:58] Reedy: Ok. So I find the <<< and >>>. Then what do I do? [03:42:06] You pick one of the changes to use [03:42:22] The bottom one is yours, the top one is what Florian changed it to [03:43:38] Then delete the other lines [03:43:42] Reedy: Which one do you think? I think Florian might have told me to use the $article method. [03:43:51] Reedy: But I think I should go with what Florian did. [03:43:53] Either is valid [03:44:15] Is there one you prefer? [03:44:39] I think I'd probably go with yours [03:44:53] As you're deglobali[sz]ing the file at the same time [03:45:53] 10MediaWiki-extensions-AbuseFilter, 10MobileFrontend: Abusefilter not tagging mobile edits - https://phabricator.wikimedia.org/T122639#1913129 (10Techman224) Now it seems to work with this edit, https://commons.wikimedia.org/wiki/Special:AbuseLog/1143562. user_mobile is 1 [03:46:02] https://gerrit.wikimedia.org/r/#/c/262059/1/includes/datavalues/SMW_DataValue.php,unified is that change that Florian did [03:46:08] for the call_user_func [03:46:14] Reedy: Ok. So I get rid of these lines? https://dpaste.de/1q0F#L130,131,132,134,146,147,148,150 [03:46:34] Yup, exactly [03:47:20] Reedy: Then I do git rebase --continue? [03:47:51] I think you need to git add the file first iirc [03:48:13] What's iirc? [03:48:32] if i remember correctly [03:49:06] Reedy: Ok. I did that. Now I run the git rebase --continue? [03:49:09] yup [03:50:02] Reedy: Success! Now what do I do? [03:50:07] git review [03:50:27] No git commit first? [03:50:31] Or remote update? [03:50:56] Reedy:^ [03:50:57] nope [03:51:22] (03CR) 10Jforrester: [C: 032] build: Enable and configure jscs, fix some errors [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/237917 (owner: 10Ricordisamoa) [03:51:37] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Remove sitelinks and subscriptions+sitelinks lookup modes in change dispatcher - https://phabricator.wikimedia.org/T118265#1913130 (10hoo) 5Open>3Resolved [03:51:39] (03CR) 10jenkins-bot: [V: 04-1] build: Enable and configure jscs, fix some errors [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/237917 (owner: 10Ricordisamoa) [03:51:44] (03PS11) 10MtDu: Replace deprecated wfMsg*() and wg*() functions [extensions/LiveTranslate] - 10https://gerrit.wikimedia.org/r/261761 (https://phabricator.wikimedia.org/T70750) [03:52:07] tada [03:52:22] Reedy: Yay! Thanks for the help. Also take a look at https://gerrit.wikimedia.org/r/262070 [03:52:33] I really need to go to bed :P [03:52:35] It's nearly 4am here [03:52:39] Reedy: Should I do any more? Or are you heading to bed after this? [03:52:45] Reedy: Ok. [03:53:08] As long as it's not something Florian has already done (he's only got a couple of commits left of these to review), you can feel free to do any others you want [03:53:15] One or both of us will look at it after we've slept [03:53:33] Reedy: Ok. So if I just do some from the list you posted, it'll be ok? [03:53:42] Yup, should be fine :) [03:53:54] Go to sleep! [03:54:05] Reedy: Well, thanks for the help and thanks for staying up! Get some rest. You deserve it. :) [03:54:11] It's not as bad as New Years eve/day [03:54:20] I got home at 5am, and had to clean up dog poop [03:54:21] :D [03:54:42] I've done over 190 commits today [03:54:45] Pretty productive [03:54:57] Reedy: Nice job! [03:55:11] https://github.com/wikimedia/mediawiki/blob/master/RELEASE-NOTES-1.27#L201-L214 [03:55:15] And that lot has been removed from core [03:56:41] 7Design, 6Multimedia, 10VisualEditor, 10VisualEditor-MediaWiki, and 4 others: Give guidance in the new upload form, to prevent the most common errors - e.g. bad file titles - https://phabricator.wikimedia.org/T116086#1913134 (10Quiddity) I have now tested this at http://en.wikipedia.beta.wmflabs.org/wiki/U... [03:58:54] Working on TweetaNew [04:00:24] unicornisaurous: Do you recognize these extensions? I don't. [04:00:42] (03CR) 10Bartosz Dziewoński: [C: 032] "Looks sensible." [core] - 10https://gerrit.wikimedia.org/r/258116 (owner: 10Thiemo Mättig (WMDE)) [04:01:24] We have a lot of extensions in our git repo that have almost been abandoned [04:01:30] Most aren't used by the WMF [04:02:08] Reedy: I see. [04:02:22] So we use this to guage whether we can delete stuff [04:02:22] The one I was fixing i18n functions on was marked “unmaintained” on MediaWiki [04:02:36] And try to be good custodians and not break stuff [04:03:10] Go to bed! :P [04:05:43] (03Merged) 10jenkins-bot: Fix positioning of jQuery.tipsy tooltip arrows [core] - 10https://gerrit.wikimedia.org/r/258116 (owner: 10Thiemo Mättig (WMDE)) [04:07:24] (03PS6) 10Ricordisamoa: build: Enable and configure jscs, fix some errors [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/237917 [04:10:41] 10MediaWiki-Export-or-Import: WikiImporter should not open the XML stream in the constructor. - https://phabricator.wikimedia.org/T89268#1913137 (10TTO) 5Open>3stalled @daniel: Can you confirm that this is no longer an issue for you? [04:11:56] (03CR) 10Ricordisamoa: "PS6 is rebase only" [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/237917 (owner: 10Ricordisamoa) [04:12:07] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/TweetANew] - 10https://gerrit.wikimedia.org/r/262071 (https://phabricator.wikimedia.org/T70750) [04:15:28] Working on smoothgallery [04:18:25] (03CR) 10Anomie: Update for SessionManager (034 comments) [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/248395 (owner: 10Anomie) [04:18:32] (03PS13) 10Anomie: Update for SessionManager [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/248395 [04:18:34] (03PS7) 10Anomie: Refactor session- and cookie-related code [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/248394 [04:23:17] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/SmoothGallery] - 10https://gerrit.wikimedia.org/r/262072 (https://phabricator.wikimedia.org/T70750) [04:25:34] Last one for tonight. Working on SolrStore [04:26:57] (03CR) 10TTO: "Dereckson: Do you still plan to work on this? If not, I can give it to a Google Code-In student to work on." [core] - 10https://gerrit.wikimedia.org/r/198215 (https://phabricator.wikimedia.org/T45426) (owner: 10Dereckson) [04:34:14] 10Commons-App-Android-Upload: Implement recursive calls to the MediaWiki API until a certain number of categories are obtained - https://phabricator.wikimedia.org/T122568#1913142 (10josephine_l) [04:35:36] 10Commons-App-Android-Upload: Implement recursive calls to the MediaWiki API until a certain number of categories are obtained - https://phabricator.wikimedia.org/T122568#1907764 (10josephine_l) [04:44:11] 10Commons-App-Android-Upload: Implement recursive calls to the MediaWiki API until a certain number of categories are obtained - https://phabricator.wikimedia.org/T122568#1913144 (10josephine_l) [04:48:08] 10Commons-App-Android-Upload: Daily reports - https://phabricator.wikimedia.org/T119164#1913145 (10josephine_l) [04:49:05] (03CR) 10Legoktm: [C: 031] Convert SubpageSortkey to use extension registration (031 comment) [extensions/SubpageSortkey] - 10https://gerrit.wikimedia.org/r/261879 (https://phabricator.wikimedia.org/T87975) (owner: 10PranavK) [04:53:53] 10MediaWiki-Configuration, 5MW-1.27-release: Support $wgAction in extension registration - https://phabricator.wikimedia.org/T117197#1913147 (10Legoktm) It's already there? https://github.com/wikimedia/mediawiki/blob/master/docs/extension.schema.json#L675 [04:55:13] 6Education-Collab: Mentorship an online ambassador - https://phabricator.wikimedia.org/T122344#1913148 (10Walaa) [05:02:58] 10Commons-App-Android-Upload: Implement recursive calls to the MediaWiki API until a certain number of categories are obtained - https://phabricator.wikimedia.org/T122568#1913151 (10josephine_l) [05:06:06] (03PS2) 10MtDu: Replace deprecated wfMsg* functions [extensions/SmoothGallery] - 10https://gerrit.wikimedia.org/r/262072 (https://phabricator.wikimedia.org/T70750) [05:07:18] 10MediaWiki-Search, 10CirrusSearch, 6Discovery: English Wikipedia search results are incomplete - https://phabricator.wikimedia.org/T122566#1913152 (10Deskana) >>! In T122566#1913127, @MZMcBride wrote: > @Deskana: I don't pay much attention to workboards, but regarding "Deskana moved this task to Advanced se... [05:11:44] (03PS2) 10MtDu: Replace deprecated wfMsg* functions [extensions/ConventionExtension] - 10https://gerrit.wikimedia.org/r/262070 (https://phabricator.wikimedia.org/T70750) [05:16:35] (03PS1) 10MtDu: Replace deprecated wfMsg* functions [extensions/CreditTab] - 10https://gerrit.wikimedia.org/r/262074 (https://phabricator.wikimedia.org/T70750) [05:44:51] hi all im futurefad new here [05:50:50] 7Design, 10MediaWiki-Special-pages, 3Google-Code-In-2015, 5Patch-For-Review: Should Special:Contributions be converted to a table - https://phabricator.wikimedia.org/T121389#1913157 (10Volker_E) @Tgr @Reedy @PauGiner @Jdlrobson I have to agree with Pau on his general feedback on the task. For Reedy's ques... [06:05:19] 7Design, 10Front-end-Standards-Group, 10Wikimedia-Developer-Summit-2016-Organization: WikiDev 16 working area: User interface presentation - https://phabricator.wikimedia.org/T119162#1913164 (10RobLa-WMF) Per @cscott's edit from earlier today ([schedule edit](https://www.mediawiki.org/w/index.php?title=Wikim... [06:09:56] (03CR) 10Jforrester: [C: 032] Mark as unmaintained. [extensions/VectorBeta] - 10https://gerrit.wikimedia.org/r/259598 (owner: 10Mattflaschen) [06:10:32] (03Merged) 10jenkins-bot: Mark as unmaintained. [extensions/VectorBeta] - 10https://gerrit.wikimedia.org/r/259598 (owner: 10Mattflaschen) [06:10:59] (03CR) 10Jforrester: [C: 032] build: Enable and configure jscs, fix some errors [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/237917 (owner: 10Ricordisamoa) [06:14:32] (03Merged) 10jenkins-bot: build: Enable and configure jscs, fix some errors [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/237917 (owner: 10Ricordisamoa) [06:18:07] 10MediaWiki-Export-or-Import, 10Wikimedia-Site-Requests, 6Wikisource: Broken revisions with rev_page=0 - https://phabricator.wikimedia.org/T18674#1913165 (10TTO) Many of the revisions on knwiki with `rev_page = 0` occurred when a user ran a simultaneous transwiki and upload import of the same pages. The tran... [06:22:43] 7Design, 10Front-end-Standards-Group, 10Wikimedia-Developer-Summit-2016-Organization: WikiDev 16 working area: User interface presentation - https://phabricator.wikimedia.org/T119162#1913167 (10RobLa-WMF) [06:31:07] 10Wikimedia-Developer-Summit-2016-Organization: Define the list of "must have" sessions for WikiDev '16 - https://phabricator.wikimedia.org/T119593#1913168 (10cscott) Since the dev summit is nigh upon us, I've gone ahead and implemented the schedule change above (descheduling the T114072 session, and moving T112... [06:40:43] Anyone around with access to the wfDebug logs on production? [06:40:43] I'd like to see if there are recent entries for testwiki from the wfDebug on line 684 of Import.php: [06:40:44] "Page ID " . $this->mPage . " mismatches the ID " . $this->mTitle->getArticleID() . " provided by the Title object." [06:43:10] hi tto [06:43:42] hi legoktm :) [06:44:21] tto: I grepped for 'provided by the' and no results [06:44:36] how recent? [06:45:00] Hmmm interesting [06:45:02] it's about 3 hours of logs, I can look at older stuff if you want [06:45:12] Something I did a few minutes ago ought to have triggered that [06:45:27] (well, within the last hour) [06:46:29] Thanks anyway! [06:46:40] anytime [06:48:11] (03PS5) 10Legoktm: Convert SubpageSortkey to use extension registration [extensions/SubpageSortkey] - 10https://gerrit.wikimedia.org/r/261879 (https://phabricator.wikimedia.org/T87975) (owner: 10PranavK) [06:49:39] (03CR) 10Legoktm: [C: 032] Convert SubpageSortkey to use extension registration [extensions/SubpageSortkey] - 10https://gerrit.wikimedia.org/r/261879 (https://phabricator.wikimedia.org/T87975) (owner: 10PranavK) [06:50:09] (03Merged) 10jenkins-bot: Convert SubpageSortkey to use extension registration [extensions/SubpageSortkey] - 10https://gerrit.wikimedia.org/r/261879 (https://phabricator.wikimedia.org/T87975) (owner: 10PranavK) [06:51:04] 10MediaWiki-extensions-SubpageSortkey, 3Google-Code-In-2015, 5Patch-For-Review: Convert SubpageSortkey to use extension registration - https://phabricator.wikimedia.org/T87975#1913173 (10Legoktm) 5Open>3Resolved a:3Pranavmk98 [06:51:09] 10MediaWiki-Configuration, 5Patch-For-Review, 7Tracking: Convert all Wikimedia-deployed extensions and skins to use extension registration - https://phabricator.wikimedia.org/T87875#1913175 (10Legoktm) [07:07:29] (03CR) 10Jack Phoenix: [C: 032] Use valid spdx license identifier [extensions/ShoutWikiAPI] - 10https://gerrit.wikimedia.org/r/261885 (owner: 10Legoktm) [07:07:58] (03Merged) 10jenkins-bot: Use valid spdx license identifier [extensions/ShoutWikiAPI] - 10https://gerrit.wikimedia.org/r/261885 (owner: 10Legoktm) [07:14:20] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Need-volunteer, 7Story: [Story] Direct link to download the raw JSON data of each Entity - https://phabricator.wikimedia.org/T66515#1913184 (10hoo) [07:14:23] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: It should be possible to get entity id from page title - https://phabricator.wikimedia.org/T67507#1913181 (10hoo) 5Open>3Resolved a:3hoo `Wikibase\Store\EntityIdLookup::getEntityIdForTitle` can do this. [07:15:31] I don't understand what to translate :( [07:15:37] Every thing is pre translated [07:20:18] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: InvalidArgumentException from line 34 of PageEntityUsages.php: $pageId must be an integer > 0 - https://phabricator.wikimedia.org/T114352#1913186 (10hoo) 5Open>3Resolved a:3hoo No more log entries regarding this since at least October 10 (all logs we kee... [07:25:11] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Implement change replication for all entities - https://phabricator.wikimedia.org/T42721#1913191 (10hoo) 5Open>3Invalid a:3hoo No longer relevant. [07:28:03] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 7Need-volunteer: Do not show language section on talk pages - https://phabricator.wikimedia.org/T47582#1913195 (10hoo) 5Open>3declined a:3hoo We no longer force that AFAIK, but ULS does nowadays (but that's not what this bug is about). [07:29:19] (03CR) 10Gergő Tisza: Update for SessionManager (031 comment) [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/248395 (owner: 10Anomie) [07:39:31] (03CR) 10Legoktm: "Leaving it totally up to implementations isn't very useful, as the point of this abstraction is so things that want to post don't have to " [core] - 10https://gerrit.wikimedia.org/r/261794 (owner: 10Mattflaschen) [08:28:23] tto: hi! [08:28:47] victorbarbu_: Hi Victor! Did you manage to get in touch with Ankita? [08:28:56] nope :( [08:30:21] How long since you last heard from her? [08:30:55] more than 24 hours [08:31:04] on the 1st actually [08:32:03] victorbarbu_: Our guidelines state that "It could take your mentor(s) up to 36 hours to receive a review of the work that you have submitted." [08:45:59] Hi tto! [08:46:14] Georggi: G'day! [08:48:01] tto, lets go into private to not spam here:) [08:51:27] I still don't understand what to do [08:51:42] for the Translation task [08:53:03] (03CR) 10TTO: ""-1"" (031 comment) [core] - 10https://gerrit.wikimedia.org/r/261311 (https://phabricator.wikimedia.org/T122587) (owner: 10Unicornisaurous) [08:54:51] ananayarora: I feel you ;) [08:56:34] Nemo_bis might be able to help you, if he is here [09:01:45] tto, have you had the time to look at my patch? [09:01:52] if yes, what do you think? [09:02:07] should I re-implement those JS files as classes? [09:02:28] victorbarbu_, I'll take another look - can you link me it? [09:02:42] of course https://gerrit.wikimedia.org/r/261911 [09:06:57] tto, sorry, internet went down:( [09:07:47] (03PS1) 10TTO: Import: Try to stop revisions getting created with rev_page = 0 [core] - 10https://gerrit.wikimedia.org/r/262078 (https://phabricator.wikimedia.org/T18674) [09:07:54] Sorry guys, I was just submitting that patch ^ [09:10:40] victorbarbu_, to test that, do I have to install both UniversalLanguageSelector and SpellingDictionary? [09:10:59] yes [09:11:24] and you should have two test steps: 1) JS disabled 2) JS enabled [09:11:29] 10MediaWiki-Page-editing: Wrap 4-tilde and 5-tilde timestamps in