[00:22:15] (03CR) 10Gergő Tisza: [C: 04-1] Add gender support to Multimediaviewer-text-embed-credit-text-tbls and other messages (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) (owner: 10MtDu) [01:13:00] welcome back, MtDu [01:13:10] IRCCloud master race [01:14:01] * OH- googles origin of PC master race joke [01:14:06] nevermind, that was a bad joke [01:14:26] Hello. I'm just on my phone. About to film [01:15:35] Tgr: I saw your comments. I'll fix them when I get home. Will you still be here in like 3 hours? [01:16:19] tgr: ^ [01:16:38] not in 3 hours exactly [01:16:48] but I'm on IRC most of the time [01:18:33] tgr: so in line 146, can I just do params = [ needGenderMessages[i][0] ]; [01:19:09] tgr: or do I have to push it to params after I do params = [] [01:19:10] that's the idea [01:19:24] no, defining arrays like that works fine [01:19:33] tgr: ok [01:20:44] tgr: So in line 355, do I check if userInfo itself is null or if userInfo.gender is null? [01:21:21] if userInfo is not null, you don't get an error [01:21:36] so you need to check for that [01:22:01] (object).foo always works in javascript, (null).foo is an error [01:23:22] tgr: so make an if statement. The if clause checking if userInfo is null. If it is then imageInfo.gender = 'unknown'. Else imageInfo.gender = userInfo.gender [01:23:48] that's one way to do it [01:23:52] tgr: And for the if clause I would use ===? [01:24:13] when in doubt, always use tripe equal sign [01:24:37] tgr: ok [01:24:48] although if you are testing for null, you can simply use ( !$userInfo ) [01:25:08] that's a bit more robust and easier to read as well [01:25:51] tgr: The dollar sign shows It's a variable right? [01:27:35] tgr:^ [01:28:19] uh, sorry, ignore that [01:28:31] I was context switching from writing some PHP code [01:28:52] tgr: ok. So just !userInfo [01:28:54] simply !userInfo [01:29:09] we use $ prefix for jQuery objects, usually [01:29:53] tgr: yep. I remember from working on stuff for SmIteSpam. and so that fixes those two thing you commented on right? [01:35:26] I think so [01:36:06] tgr: And about the file permissions, I'll try those commands. Hopefully they will work [01:37:12] https://www.irccloud.com/pastebin/Fl5yfihH [01:37:42] tgr: ^ [01:41:24] indeed [01:51:52] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor, 10VisualEditor-MediaWiki-Media, 5Patch-For-Review: Show determinate progress bar for the image upload in mw.ForeignStructuredUpload.BookletLayout - https://phabricator.wikimedia.org/T115861#1899923 (10Esanders) Adding to this, I think the quest... [03:00:22] welcome back, JadeMaveric [03:09:02] Hi, Hydronium, OH- nice [03:15:02] I must have done something horrible. The Multimedia Viewer doesn't seem to be working. [03:15:27] Does anyone know how to undo the last commit? [03:24:52] Yay, it worked! [03:24:55] git checkout master [03:25:04] git reset master [03:25:18] git checkout -- * [03:25:29] But now I'm back at square one. :( [03:25:48] Or 1.5. At least I know what to do [04:46:35] Does the Multimedia Viewer plugin have any developer docs? [05:00:55] tgr: Does the Multimedia Viewer plugin have any developer docs? [05:02:11] JadeMaveric: not really [05:02:27] just the standard ones, https://www.mediawiki.org/wiki/Extension:MultimediaViewer and https://doc.wikimedia.org/MultimediaViewer/master/js/ [05:04:40] tgr: I found the first one yesterday, not really helpful. The second one seems useful. Thanks [05:05:53] yeah, the extension page is more of a webmaster documentation than developer documentation [05:05:57] It's got a nice interface too! [05:07:37] tgr: What does the 'abstract' tag mean? [05:07:57] we have such a thing? [05:08:08] usually it means that you need to subclass it [05:08:35] can't remember having any abstract classes in MediaViewer though [05:08:42] kk, that makes sense. (I was viewing the 'element' class) [05:09:04] oh yeah, the element/model/provider base classes [05:09:04] mw.mmv.ui.Element [05:09:31] (03PS21) 10MtDu: Add gender support to Multimediaviewer-text-embed-credit-text-tbls and other messages [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) [05:11:19] Is there any 'official' way to run test? Right now I've uploaded an image to the admin page on my local wiki. And I'm testing MMV from there [05:11:46] do you mean unit tests? [05:11:53] or just manual testing? [05:12:04] what are unit test? Manual I can guess [05:12:39] I want to remove the uploader's name, so I think I want manual [05:12:48] 'cause it's going to affect the end-user [05:15:35] tgr: My patch passed all tests. :) [05:15:42] tgr: I need some help with the file permissions. [05:15:51] tgr: They still don't seem to get fixed. https://dpaste.de/7Znj#L48,49 [05:15:54] JadeMaveric: in theory, unit testing is a way to test parts of the code in isolation, so relatively simple errors can be found quickly [05:17:21] in practice, some of the MediaViewer tests ended up rather awkward [05:18:04] anyway, you don't need to worry about unit tests initially, they are automated tests so the continuous integration system will complain anyway if they break [05:18:41] tgr:^ [05:19:07] ? [05:19:25] manually, just look at some images? the change is pretty simple so I don't think there are many ways to test it [05:19:48] That's what I'm doing. But I didn't find any images, so I uploaded them [05:19:56] there is a page called E2E...something with some images if you don't want to set up your own [05:20:30] mediawiki-vagrant uses https://www.mediawiki.org/wiki/InstantCommons which means you can use Commons images without uploading [05:20:46] of course if you do upload them that's fine as well [05:21:37] My local wiki doesn't have anything! Besides a few special pages [05:22:10] MtDu: right before you commit the permission change, can you run git status and git diff --cached and see what they say? [05:22:20] JadeMaveric: are you using vagrant? [05:22:43] yes, I'm using vagrant [05:23:50] tgr: Same thing. https://dpaste.de/ABMd#L42,43,45,46 [05:23:55] then you should have that page [05:24:31] http://127.0.0.1:8080/wiki/InstantCommons --> There is currently no text in this page [05:25:42] there shouldn't, that's just a documentation page on mw.org [05:25:56] there should be a page called E2Esomething though [05:26:17] you can just start typing it into the search bar and get it autocompleted [05:26:34] MtDu: looks like the checkout command is not working [05:27:00] search ( E2E ) --> There were no results matching the query. [05:27:02] tgr: Should I do git checkout master instead? [05:27:24] note how the instructions say "use git checkout -- to discard changes" [05:27:31] and you did exactly that [05:27:39] but the changes are not discarded [05:27:55] I doubt checkout out from another version makes a difference [05:28:09] some weird filesystem problem maybe? [05:28:21] tgr: Hm. Ok. I'm not sure. I am on a Windows machine [05:28:28] you can get around it by committing the index, not the working directory [05:28:42] are you familiar with git terminology? [05:28:47] tgr: So just git commit --index? [05:29:00] just git commit [05:29:12] well, git commit --amend because you don't want a new commit [05:29:39] tgr: I have to run those commands again right? [05:29:53] tgr: The git update-index --chmod..... one? [05:30:31] JadeMaveric: sorry, memory problems [05:30:37] the correct name is MediaViewerE2ETest [05:31:03] MtDu: yes, run everything again, just omit the --all from the commit command [05:32:17] tgr: It worked, but all the images have broken links. [05:32:20] tgr: It's working. :) [05:32:30] MtDu: Are you GCI student? [05:32:33] neat [05:32:40] JadeMaveric: Yes. Welcome. :) [05:32:50] (03PS22) 10MtDu: Add gender support to Multimediaviewer-text-embed-credit-text-tbls and other messages [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) [05:33:16] I think you would avoid weird filesystem problems if you used vagrant ssh instead of git bash, but it's a moot point now [05:33:41] tgr: I thought I had to run all these commands in git bash? [05:33:50] tgr: Oh. You're saying go into vagrant ssh first? [05:34:05] well, apparantly git bash is problematic when it comes to permissions [05:34:36] which makes sense, it's a simulated linux filesystem over what's actually a windows filesystem so it's bound to be fragile [05:35:23] of course, vagrant's shared folders are also a simulated linux filesystem over a windows filesystem, but the simulation is much more robust [05:35:41] tgr: Yep. Makes sense. [05:35:57] MtDu: for most things, git bash and the vagrant terminal are identical [05:35:58] tgr: Everything is working and successful. I have submitted the task on GCI. Thanks for all your help and patience! [05:36:09] tgr: Gotcha. [05:36:25] except you need to run the vagrant XXX commands on your host machine, and PHP commands on your guest machine [05:36:52] tgr: Ok. [05:40:05] ME sighs [05:40:09] * JadeMaveric sighs [05:40:19] JadeMaveric: You can do it! [05:41:16] Thanks, I know that too. I sighed because of the local wiki images not loading [05:41:54] what kind of images? [05:42:36] The images on MediaViewrE2ETest. [05:42:43] All the images have broken links. [05:45:04] huh [05:45:22] I admit I haven't looked at the for a long time [05:45:51] but the automated browser test is supposed to use that page, it should alert when something is broken [05:46:01] http://127.0.0.1:8080/wiki/MediaViewerE2ETest --> lots of red links with 'Category: Pages with broken file links' at the bottom [05:46:38] This is on my system. [05:46:49] also, the links on that page don't look wrong [05:48:09] This works: https://www.mediawiki.org/wiki/MediaViewerE2ETest [05:48:21] But this doesn't: http://127.0.0.1:8080/wiki/MediaViewerE2ETest [05:49:05] well crap [05:49:15] righto [05:49:16] InstantCommons is broken again on vagrant [05:49:27] righto again [05:50:42] Wait, instant commons doesn't even exits, I was talking about E2E [05:53:17] tgr: Here is the link. https://gerrit.wikimedia.org/r/#/c/260221/ Take a look when you get a chance. I've already submitted the task. Good night! [05:54:39] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-Vagrant: InstantCommons does not work in Vagrant (again) - https://phabricator.wikimedia.org/T122275#1900301 (10Tgr) 3NEW a:3Tgr [05:55:52] JadeMaveric: does that block your task? [05:56:30] NO. I can always manually upload and test. In fact, that's what I've been doing. [05:56:43] cool [05:56:44] But doesn't it hinder the automated test? [05:56:52] I will look into this, eventually [05:57:29] the automated tests run on en.wikipedia.beta.wmflabs.org [05:57:41] kk. [05:57:42] which is set up differently, so probably it's not broken there [05:58:03] that, or no one checks whether the tests are working anymore [05:58:18] I sure don't [05:58:46] but people in the release engineering team tend to spot when something is on fire [07:04:20] (03CR) 10Gergő Tisza: [C: 04-1] "One of the changes in EmbedFileFormatter has been corrected, but the other one is still wrong (pushing the username which does not work in" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) (owner: 10MtDu) [07:44:15] tgr|away: Does the uploaded date refer to the last/initial upload date? Should I rename it to 'Last uploaded: '? [07:48:28] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor, 10VisualEditor-MediaWiki-Media, 5Patch-For-Review: Show determinate progress bar for the image upload in mw.ForeignStructuredUpload.BookletLayout - https://phabricator.wikimedia.org/T115861#1900351 (10Pginer-WMF) I'd recommend to get some clari... [08:05:33] JadeMaveric: it falls back to the upload date (of the last version of the file I think) but it can take its value from other sources [08:05:53] EXIF metadata, file page templates [08:07:21] kk, btw I found multiple references (if that's what their called) to 'multimediaviewer-userpage-link' in extensions.json [08:07:34] I removed them and nothing bad happened. [08:08:12] I think they link to i18n/.json --> multimediaviewer-userpage-link [08:08:43] Should I remove all the entries there? I only removed the `en` one [08:09:15] no, the rest will be removed automatically [08:09:27] SO only remove 'en' [08:10:02] translation happens on translatewiki.net and they have automation to keep languages in sync [08:10:19] so only remove 'en'? [08:10:25] Gah, I've lost my train of thought [08:10:51] if you check where it is used in the code though, you'll find lots of stuff that is becoming unnecessary [08:11:54] extensions.json basically tells which javascript modules need which messages [08:12:25] since messages need to be sent to the browser as well, not just JS code [08:12:55] so some message appearing multiple times only means that it's used by multiple modules [08:14:06] So it it safe to remove those multiple declaration? [08:14:29] It sounds like they're automatically generated [08:15:09] so the declaration in say fr.json is automatically generated from en.json [08:15:37] for some value of "automatically" - the actual translation is done by a human of course [08:16:26] but you don't need to worry about them, change en.json and the translation mechanism will sort the rest out [08:17:04] if you check the git log, you can see it's full of commits by one "translation updater bot" [08:18:03] everything else is the developers' responsibility [08:18:54] so if you remove something from en.json you need to make sure it's removed from extension.json, and removed from javascript code as well [08:19:20] (and php code, but MediaViewer doesn't have much php codeÖ [08:47:06] 6Multimedia, 10MediaWiki-Special-pages: Special:MediaStatistics: Show file size per section after the section title (before the table) - https://phabricator.wikimedia.org/T122208#1900394 (10Legoktm) http://commons.wikimedia.beta.wmflabs.org/wiki/Special:MediaStatistics Also the "Total file size for all files"... [11:10:30] (03PS1) 10Mhutti1: Moved uw.controller.Details.prototype.valid to use OO.ui.confirm() [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) [11:13:20] (03CR) 10jenkins-bot: [V: 04-1] Moved uw.controller.Details.prototype.valid to use OO.ui.confirm() [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) (owner: 10Mhutti1) [11:31:26] (03PS2) 10Mhutti1: Moved uw.controller.Details.prototype.valid to use OO.ui.confirm() [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) [13:17:23] 6Multimedia, 6Commons, 10UploadWizard: Provide a copyright click-through guide in the uploadwizard - https://phabricator.wikimedia.org/T121229#1900734 (10Prtksxna) These designs were made for the upload dialog in core. That dialog allows //only// own work right now, and these instructions are for that. Uploa... [14:00:22] (03PS23) 10MtDu: Add gender support to Multimediaviewer-text-embed-credit-text-tbls and other messages [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) [14:02:01] (03PS24) 10MtDu: Add gender support to Multimediaviewer-text-embed-credit-text-tbls and other messages [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) [14:05:53] (03CR) 10MtDu: "The tests passed when I just changed info.imageInfo.lastUploader to info.imageInfo.gender, but I just put all the logic in just in case so" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) (owner: 10MtDu) [14:48:20] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor, 10VisualEditor-MediaWiki-Media, 5Patch-For-Review: Show determinate progress bar for the image upload in mw.ForeignStructuredUpload.BookletLayout - https://phabricator.wikimedia.org/T115861#1900912 (10Esanders) If you look at the actual example... [15:33:31] (03CR) 10Bartosz Dziewoński: [C: 04-1] "Thanks! But this doesn't quite work. With this patch, it's now impossible to submit the form when there are no warnings :)" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) (owner: 10Mhutti1) [15:38:54] MatmaRex: Could you review https://gerrit.wikimedia.org/r/#/c/253550/ and https://gerrit.wikimedia.org/r/#/c/247513/, please? [15:40:06] sure [15:40:27] Thanks! [15:43:42] MatmaRex: I am off to bed a little early today. No meeting \o/ [15:43:48] MatmaRex: I'll see you tomorrow. [15:43:59] MatmaRex: Stand-up tomorrow? [15:46:00] prtksxna: maybe? i'm not sure if i'll be around at this time [15:46:31] i think i'll be mostly doing code review today :) have a few other things queued [15:46:59] MatmaRex: Cool! [16:06:36] (03CR) 10Mhutti1: "I kept the actions because the names had already been localized but I guess they are general localizations so removing them would't remove" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) (owner: 10Mhutti1) [16:11:38] 6Multimedia, 6Commons, 10MediaWiki-extensions-GWToolset: "HTTP request timed out" for large files - https://phabricator.wikimedia.org/T86024#1901032 (10Kelson) [16:15:22] (03PS3) 10Mhutti1: Moved uw.controller.Details.prototype.valid to use OO.ui.confirm() [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) [16:36:32] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor, 10VisualEditor-MediaWiki-Media, 5Patch-For-Review: Show determinate progress bar for the image upload in mw.ForeignStructuredUpload.BookletLayout - https://phabricator.wikimedia.org/T115861#1901062 (10Pginer-WMF) > If you look at the actual exa... [17:02:45] 6Multimedia, 6Commons, 10MediaWiki-extensions-GWToolset: GWT - https://phabricator.wikimedia.org/T119053#1901103 (10Kelson) I have tried to reproduce that bug with an ever simplified version of the XML including only the row with "GS-GUGE-FREUDENBERGER-A-1". In my case it fails too with... [17:03:11] 6Multimedia, 6Commons, 10MediaWiki-extensions-GWToolset: "HTTP request timed out" for large files - https://phabricator.wikimedia.org/T86024#960033 (10Kelson) [17:05:52] 6Multimedia, 6Commons, 10MediaWiki-extensions-GWToolset: "HTTP request timed out" for large files - https://phabricator.wikimedia.org/T86024#1901115 (10Kelson) It looks like the current timeout is 90s, it should be more something like 1200s. Because of that bug we have GLAM partners stuck in their upload pr... [17:06:38] 6Multimedia, 6Commons, 10MediaWiki-extensions-GWToolset: "HTTP request timed out" for large files - https://phabricator.wikimedia.org/T86024#1901116 (10Kelson) p:5Low>3Unbreak! [17:19:32] MatmaRex: hold up, before you go https://imgur.com/a/p5Ikw [17:20:25] 6Multimedia, 6Commons, 10MediaWiki-extensions-GWToolset: GWT - https://phabricator.wikimedia.org/T119053#1816669 (10Kelson) We should really try to fix this in the next times because we have a whole GLAM upload/GLAM partner stuck because of that bug. [17:20:48] (03CR) 10Bartosz Dziewoński: [C: 04-1] "Thanks. But one more case remains, notice how nothing happens when there are errors in the form (e.g. a required field is empty) and you t" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) (owner: 10Mhutti1) [17:21:20] OH-: yeah? [17:21:37] progress on date filters and date selection widget [17:21:44] looks promising :D [19:08:52] (03CR) 10Bartosz Dziewoński: [C: 032] "Thank you, everything looks fine now!" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) (owner: 10Mhutti1) [19:12:52] (03Merged) 10jenkins-bot: Moved uw.controller.Details.prototype.valid to use OO.ui.confirm() [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260740 (https://phabricator.wikimedia.org/T117077) (owner: 10Mhutti1) [19:13:39] 6Multimedia, 10UploadWizard, 7Easy, 3Google-Code-In-2015, 7Technical-Debt: Use OO.ui.confirm() for the confirmation dialog in uw.controller.Details.prototype.valid - https://phabricator.wikimedia.org/T117077#1901547 (10matmarex) 5Open>3Resolved [19:28:45] (03PS1) 10Bartosz Dziewoński: mw.UploadWizard: Change order of operations to position the file input correctly [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260781 [19:29:25] (03CR) 10Bartosz Dziewoński: "Actually, this does not fix the whole issue – the button is still not updated when removing a file from the list." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260781 (owner: 10Bartosz Dziewoński) [19:31:39] (03PS1) 10Bartosz Dziewoński: Revert "Kill polling for moveFileInputToCover" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260782 [19:31:51] (03Abandoned) 10Bartosz Dziewoński: mw.UploadWizard: Change order of operations to position the file input correctly [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260781 (owner: 10Bartosz Dziewoński) [19:31:58] (03CR) 10Bartosz Dziewoński: [C: 032] Revert "Kill polling for moveFileInputToCover" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260782 (owner: 10Bartosz Dziewoński) [19:32:48] (03CR) 10Bartosz Dziewoński: "Attempted follow-up: https://gerrit.wikimedia.org/r/260781 (it didn't fix everything, so I opted to revert instead)." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260059 (https://phabricator.wikimedia.org/T121901) (owner: 10MarkTraceur) [19:37:31] (03Merged) 10jenkins-bot: Revert "Kill polling for moveFileInputToCover" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/260782 (owner: 10Bartosz Dziewoński) [19:37:31] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor, 10VisualEditor-MediaWiki-Media, 5Patch-For-Review: Show determinate progress bar for the image upload in mw.ForeignStructuredUpload.BookletLayout - https://phabricator.wikimedia.org/T115861#1901624 (10Esanders) There are two issues here: * Red... [20:01:44] tgr: When you get a chance, take a look at my patch. I posted a comment about what I changed. https://gerrit.wikimedia.org/r/260221. [21:05:04] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7JavaScript: 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#1901986 (10matmarex) [21:12:47] (03CR) 10Jdlrobson: [C: 04-1] "Looking at this patch and thanks to the work that you've done in exploring this problem it's clear that it's not quite as simple as we fir" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) (owner: 10MtDu) [21:25:25] 6Multimedia, 10MediaWiki-Special-pages: Implement 'newbies' mode for Special:NewFiles - https://phabricator.wikimedia.org/T122354#1902125 (10matmarex) 3NEW [21:25:38] 6Multimedia, 10MediaWiki-Special-pages: Implement 'newbies' mode for Special:NewFiles - https://phabricator.wikimedia.org/T122354#1902131 (10matmarex) [21:26:52] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7JavaScript: 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#1902139 (10matmarex) Back to your regularly scheduled programme: >>! In T12... [21:28:22] (03Abandoned) 10Gergő Tisza: Add gender support to Multimediaviewer-text-embed-credit-text-tbls and other messages [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260221 (https://phabricator.wikimedia.org/T85685) (owner: 10MtDu) [21:43:09] 6Multimedia, 6Commons, 10MediaWiki-File-management: CSV file downloads - https://phabricator.wikimedia.org/T122358#1902185 (10ekkis) 3NEW [21:56:09] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7JavaScript: 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#1902232 (10matmarex) And back to the ongoing A/B test: **I decided to extend... [22:12:00] 6Multimedia, 6Commons, 10MediaWiki-File-management: CSV file downloads - https://phabricator.wikimedia.org/T122358#1902300 (10matmarex) [22:20:05] 6Multimedia, 6Commons, 10Wikidata, 7Community-Wishlist-Survey: Allow tabular datasets on Commons (or some similar central repository) (CSV, TSV, JSON, XML) - https://phabricator.wikimedia.org/T120452#1902320 (10matmarex) [22:26:06] 6Multimedia, 6Commons, 10MediaWiki-File-management: Image Width and Image Height seem to be mixed up in image metadata (file Exif.php?) - https://phabricator.wikimedia.org/T107717#1902363 (10matmarex) a:3matmarex It does seem like we have width and height switched around… haha. ``` "exif-pixelydimension":... [22:29:18] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 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#1902384 (10ggellerman) [22:35:54] 6Multimedia, 6Commons, 10MediaWiki-General-or-Unknown: Links do not work in svg-long-desc, insane logic in nested message parsing - https://phabricator.wikimedia.org/T122369#1902425 (10Rillke) [23:23:45] (03PS1) 10Dduvall: WIP Testing mwext-mw-selenium job [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260875 [23:27:37] (03CR) 10jenkins-bot: [V: 04-1] WIP Testing mwext-mw-selenium job [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/260875 (owner: 10Dduvall) [23:34:22] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: MediaViewer intermittent image loading errors (2015 Oct-Dec) - https://phabricator.wikimedia.org/T115563#1902669 (10Tgr) [23:45:56] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: MediaViewer intermittent image loading errors (2015 Oct-Dec) - https://phabricator.wikimedia.org/T115563#1733031 (10Tgr) [23:51:42] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10Structured-Multimedia-Data, 10Wikidata: Provide a way to curate and fetch titles (short descriptions) for media files - https://phabricator.wikimedia.org/T122381#1902764 (10Tgr) 3NEW [23:52:15] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10Structured-Multimedia-Data, 10Wikidata: Provide a way to curate and fetch titles (short descriptions) for media files - https://phabricator.wikimedia.org/T122381#1902774 (10Tgr)