[07:57:12] 3MediaWiki / 3Uploading: Uppercase picture extensions like .PNG are not allowed - 10https://bugzilla.wikimedia.org/66667#c7 (10Tiziano Müller) Can confirm bug in mediawiki 1.23.3 and yes, the logic behind that preg_match is definitely false. [12:27:43] 3MediaWiki extensions / 3CommonsMetadata: '\n' are added to various elements in CommonsMetadata output - 10https://bugzilla.wikimedia.org/57458#c16 (10Tisza Gergő) *** Bug 66652 has been marked as a duplicate of this bug. *** [12:27:44] 3MediaWiki extensions / 3CommonsMetadata: CommonsMetadata outputs values with trailing « » - 10https://bugzilla.wikimedia.org/66652#c1 (10Tisza Gergő) 5NEW>3RESO/DUP Fixed by a purge. Other instances should be fixed on their own 30 days after deploying the patch for bug 57458, so in ~3 weeks. *** T... [12:34:21] 3MediaWiki extensions / 3CommonsMetadata: Read "other_field" and add the value of "Attribution" to extmetadata - 10https://bugzilla.wikimedia.org/57197#c4 (10Tisza Gergő) CommonsMetadata parses id="fileinfotpl_credit" and class="licensetpl_attr" now, as specified by [[commons:COM:MRD]]. Raimond, is it OK to... [12:37:56] 3MediaWiki extensions / 3CommonsMetadata: Rename key "Credit" to "FileSource" - 10https://bugzilla.wikimedia.org/57189 (10Tisza Gergő) [12:37:58] 3MediaWiki extensions / 3CommonsMetadata: data labels in imageinfo API don't make sense - 10https://bugzilla.wikimedia.org/69914 (10Tisza Gergő) [12:37:58] 3MediaWiki extensions / 3CommonsMetadata: UsageTerms should be called LicenseLongName - 10https://bugzilla.wikimedia.org/57847 (10Tisza Gergő) [12:38:12] 3MediaWiki extensions / 3CommonsMetadata: data labels in imageinfo API don't make sense - 10https://bugzilla.wikimedia.org/69914 (10Tisza Gergő) [12:41:56] 3MediaWiki extensions / 3CommonsMetadata: Description not parsed - 10https://bugzilla.wikimedia.org/57311#c1 (10Tisza Gergő) 5ASSI>3RESO/WOR That was the long time ago; seems to work now. Assuming whatever caused this has been fixed. [12:49:54] (03PS1) 10Gilles: Track more link clicks [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158614 [12:50:29] (03CR) 10jenkins-bot: [V: 04-1] Track more link clicks [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158614 (owner: 10Gilles) [12:53:08] hey gi11es do you happen to have time to look at https://gerrit.wikimedia.org/r/#/c/156243/ today? [12:53:48] dan-nl: maybe... aren't you waiting on legoktm to approve? [12:54:02] errr, please don't wait on me [12:54:07] that was just a drive-by review [12:54:21] oh, if he can approve it then that would be great. i didn't know he could +2 it [12:57:52] (03PS2) 10Gilles: Track more link clicks [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158614 [12:57:53] (03CR) 10jenkins-bot: [V: 04-1] Track more link clicks [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158614 (owner: 10Gilles) [12:57:54] (03PS3) 10Gilles: Track more link clicks [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158614 [12:57:54] alright I have exactly 30 minutes to review it https://www.youtube.com/watch?v=ar0xLps7WSY [13:01:27] 3MediaWiki extensions / 3TimedMediaHandler: TimedMediaHandler making bad action=raw requests - 10https://bugzilla.wikimedia.org/69453#c9 (10Kunal Mehta (Legoktm)) I looked at the logs again today, and there's at least one new broken account since bawolff's patch was deployed. The log entries we have for it... [13:04:06] thanks gi11es i appreciate it. if there's something i might be able to help you out with let me know [13:04:40] dan-nl: do you have a simple xml sample file handy? preferably not one pointing to gigabytes of images :) I've lost the one I used to use for testing [13:06:08] gi11es: just emailed you 3 different sets [13:06:38] thanks [13:09:46] gi11es: also sent you links to the corresponding metadata mappings [13:12:40] dan-nl: hmmm the rijksmuseum.json seems to contain the html of the page hosting the json [13:12:54] instead of the json itself [13:13:25] ah, that's because I saved your link from the email, duh... [13:13:27] hmm, lookign at it now [13:13:32] oh [13:13:45] I just saved the page expecting it to be a link to a json file :) [13:14:07] ah :) [13:16:40] dan-nl: I'm going through the setup again since my last vm died, https://www.mediawiki.org/wiki/Extension:GWToolset makes no mention of $wgGWTFileBackend [13:16:44] and I think it needs to be defined, otherwise I run into an exception when I try to import something [13:16:54] it should be mentioned in the setup steps, if it's mandatory [13:17:19] it's in the INSTALL file, but doesn't end up on the wiki page [13:17:49] k, i'll look at the install file and add those required items to the extension page [13:18:54] short of that, you could as well define the defaults mentioned in the INSTALL file as actual defaults [13:20:19] the wiki does say "go look at the install file" but I visually ignored it, usually the required globals are shown right there on the wiki page [13:20:27] anyway, back to testing... [13:21:56] dan-nl: which template does the rijks data expect? [13:23:02] I tried Artwork, copied from Commons beta, and it was unhappy [13:27:32] it can go in any, but i usually use artwork [13:27:50] hmm, try export/import from commons regular [13:31:49] it whined about lack of templatedata block [13:34:59] never used export/import before, it failed miserably on the import side: "Import failed: Expected tag, got" [13:35:28] and I ran out of time, I have a medical appointment. I'll have to continue this on monday [13:36:34] (I was trying to import Template:Art_Photo from Commons proper) [14:02:03] gi11es: sorry about the flakey internet connection. i ended up placing a link to the install instructions in the git repo. do you think that would be enough? was an export/import of the commons artwork template enough to get the tool working? [14:37:42] (03PS2) 10Gergő Tisza: Log whether attribution could be presented by MediaViewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158560 [14:41:02] (03CR) 10Gergő Tisza: [C: 032] "Spacing issues fixed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158560 (owner: 10Gergő Tisza) [14:41:29] gi11es: https://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals#Activate_Flickr_option_in_current_UploadWizard_to_all_users [14:41:37] you'r tecnical comment is welcome. [14:41:40] Ooh. [14:41:40] (03Merged) 10jenkins-bot: Log whether attribution could be presented by MediaViewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158560 (owner: 10Gergő Tisza) [14:43:35] I'm sure I should have said that as (WMF) [14:43:46] It's so hard to keep my personalities straight [14:46:35] marktraceur: you can resign it if needed ;) [14:54:01] Nah. [14:56:13] Oh man, look at all of the meetings I don't have today [14:56:21] I don't even know what to do with myself [15:00:40] ^^ [15:48:18] (03CR) 10MarkTraceur: "...weird." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/151699 (owner: 10MarkTraceur) [16:35:29] tgr: hey; About your last email in that Metadata taskforce thread, about parser cache etc.: Do we really need categories to track progress? Wouldn't the transclusion count of {{author missing}} etc. be enough? [17:14:16] (03PS10) 10MarkTraceur: Add controllers for each step - mostly stubs [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/151699 [17:19:11] (03PS4) 10MarkTraceur: Go to the original image on image clicks [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158518 [17:20:37] tgr, gi11es - didn't someone already write a patch for https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/830 ? [18:04:27] 3MediaWiki extensions / 3CommonsMetadata: Rename key "Credit" to "FileSource" - 10https://bugzilla.wikimedia.org/57189#c5 (10Ryan Kaldari) While I appreciate the idea of adhering to standards, it doesn't make any sense for us to try to follow XMP (which is designed for very different use cases and not adequa... [18:09:33] (03PS2) 10Kaldari: Renamed 'Credit' to 'FileSource' [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/107368 (https://bugzilla.wikimedia.org/57189) (owner: 10Amruthasangeeth) [18:09:41] (03CR) 10jenkins-bot: [V: 04-1] Renamed 'Credit' to 'FileSource' [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/107368 (https://bugzilla.wikimedia.org/57189) (owner: 10Amruthasangeeth) [18:20:51] (03PS3) 10Kaldari: Renamed 'Credit' to 'FileSource' [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/107368 (https://bugzilla.wikimedia.org/57189) (owner: 10Amruthasangeeth) [18:27:26] (03CR) 10Kaldari: [C: 031] "Gergő Tisza: The main problem with using 'Credit' is that it sounds like it should be used for attribution. When we built the mobile Media" [extensions/CommonsMetadata] - 10https://gerrit.wikimedia.org/r/107368 (https://bugzilla.wikimedia.org/57189) (owner: 10Amruthasangeeth) [18:51:05] (03PS1) 10MarkTraceur: Make "more details" button bigger 'n' prettier [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158660 [18:52:49] (03CR) 10MarkTraceur: "The "more details" button changes have been ported over to I81a370a910c0cec269a000c9dcaac8461d3421b0" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/150585 (owner: 10Gilles) [18:53:39] (03CR) 10MarkTraceur: [C: 04-1] "Need to fix the "use this file" button before this can be merged - it's too big now!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158660 (owner: 10MarkTraceur) [21:54:56] (03CR) 10MarkTraceur: "Am I reading this wrong, or is DSC not blacklisted?" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/158337 (https://bugzilla.wikimedia.org/65624) (owner: 10Gilles) [21:55:16] (03CR) 10MarkTraceur: "(IMG, sorry, but DSC also doesn't return a failed check)" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/158337 (https://bugzilla.wikimedia.org/65624) (owner: 10Gilles) [21:55:38] (03CR) 10MarkTraceur: "Oh, my bad, tbaction=create instead of upload" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/158337 (https://bugzilla.wikimedia.org/65624) (owner: 10Gilles) [21:57:25] (03CR) 10MarkTraceur: [C: 04-1] "The function documentation for the preprocess method suggests that the file: prefix should probably be added there." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/158337 (https://bugzilla.wikimedia.org/65624) (owner: 10Gilles) [22:35:25] (03CR) 10MarkTraceur: "Something makes me wonder whether we should just make this configurable, and leave it in for now...I'm loathe to get rid of so much code!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158586 (owner: 10Gilles) [22:38:38] (03CR) 10MarkTraceur: "Otherwise it looks fine to me." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158586 (owner: 10Gilles) [22:53:31] (03CR) 10MarkTraceur: "Same question as the parent commit - I'd sort of like to keep the code around, but this looks fine to me otherwise." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/158587 (owner: 10Gilles) [22:56:14] loath*