[02:17:03] tgr, how should I detect if GuidedTour is installed or not, now that ext.guidedTour.launcher is no longer a dependency? [06:25:45] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985189 (10Steinsplitter) @Magnus: Lovely! Your work is highly appropriated :) [06:35:52] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985190 (10Steinsplitter) >>! In T86483#984971, @Magnus wrote: > The new code doesn't do file replacement or category replacement yet, so I suggest we keep the ol... [06:36:25] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985191 (10MichaelMaggs) Yes, thanks Magnus. You've made a lot of people very happy! [07:36:31] unicodesnowman: you can detect it server-side and set a variable via the ResourceLoaderGetConfigVars hook [07:39:02] unicodesnowman: that's the general solution, or you can check if there is something specific to the extensions that's always loaded/set in JS [07:39:43] check the same hook in GuidedTourHooks.php for example [07:43:04] thanks! [07:49:50] 3MediaWiki-extensions-GWToolset, Multimedia: GWT seems to be currently broken - https://phabricator.wikimedia.org/T87040#985198 (10Steinsplitter) [07:57:36] (03PS2) 10Unicodesnowman: Add interactive tour of Media Viewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185670 (https://phabricator.wikimedia.org/T77726) [08:02:25] 3MediaWiki-extensions-GWToolset, Multimedia: GWT seems to be currently broken - https://phabricator.wikimedia.org/T87040#985204 (10Swiss-National-Library) We used the following file {F29666} [08:15:02] (03CR) 10Gergő Tisza: [C: 04-1] Add interactive tour of Media Viewer (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185670 (https://phabricator.wikimedia.org/T77726) (owner: 10Unicodesnowman) [08:25:22] (03PS3) 10Unicodesnowman: Add interactive tour of Media Viewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185670 (https://phabricator.wikimedia.org/T77726) [08:51:30] (03CR) 10Gergő Tisza: "Great job, thanks!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185670 (https://phabricator.wikimedia.org/T77726) (owner: 10Unicodesnowman) [08:51:39] (03CR) 10Gergő Tisza: [C: 031] Add interactive tour of Media Viewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185670 (https://phabricator.wikimedia.org/T77726) (owner: 10Unicodesnowman) [09:58:48] 3MediaWiki-File-management, Multimedia: Include at least some EXIF metadata in resized pictures - https://phabricator.wikimedia.org/T20871#985274 (10El_Grafo) >>! In T20871#241270, @brion wrote: > PHP's exif support doesn't appear to include writing metadata, though, so we might need to add new code to copy the... [10:00:43] (03CR) 10Thiemo Mättig (WMDE): [C: 031] "Can we please merge this? The reason that the removal of the readOnly blockages does not solve _all_ issues is not a good reason to stick " [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/177786 (owner: 10Thiemo Mättig (WMDE)) [10:05:57] (03CR) 10Thiemo Mättig (WMDE): [C: 031] "What's a possible way forward here? Guys, come on. This is only one of the many reasons why trying to actually "work" with MV is so painfu" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/178219 (owner: 10Gergő Tisza) [10:22:22] re task, thanks! it was pretty fun :) [12:09:31] 3MediaWiki-extensions-UploadWizard, Multimedia: remove "Brasilian Portugese" from UploadWizard as the template has been merged - https://phabricator.wikimedia.org/T67204#985398 (10Steinsplitter) [12:09:57] 3MediaWiki-extensions-UploadWizard, Multimedia: remove "Brasilian Portugese" from UploadWizard as the template has been merged - https://phabricator.wikimedia.org/T67204#702999 (10Steinsplitter) [12:10:16] 3MediaWiki-extensions-UploadWizard, Multimedia: remove "Brasilian Portugese" from UploadWizard as the template has been merged - https://phabricator.wikimedia.org/T67204#702999 (10Steinsplitter) [12:11:42] 3MediaWiki-extensions-UploadWizard, Multimedia: remove "Brasilian Portugese" from UploadWizard as the template has been merged - https://phabricator.wikimedia.org/T67204#985403 (10Steinsplitter) Pt-br is still a valid user interface language setting users may choose in their preferences. So maybe this schould be... [12:37:12] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985414 (10Magnus) File replacement in the new code has been tested and appears to work. I suggest the old Delinker to be turned off now. It might also help to p... [13:33:32] (03CR) 10Gilles: [C: 032] Fix date field name [analytics/multimedia] - 10https://gerrit.wikimedia.org/r/185600 (https://phabricator.wikimedia.org/T78189) (owner: 10Gergő Tisza) [13:33:41] (03Merged) 10jenkins-bot: Fix date field name [analytics/multimedia] - 10https://gerrit.wikimedia.org/r/185600 (https://phabricator.wikimedia.org/T78189) (owner: 10Gergő Tisza) [13:45:34] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Pass extra GET parameter for Media Viewer preload requests - https://phabricator.wikimedia.org/T86914#985440 (10Gilles) You're preaching to the choir, this is a stopgap measure, as mentioned in the task description. EventLogging has just started using sendBeaco... [14:00:20] 3MediaWiki-extensions-PdfHandler, Multimedia: "Error generating thumbnail" for a specific PDF file (gs OOM'ing?) - https://phabricator.wikimedia.org/T61668#985463 (10Gilles) I can't reproduce the warnings on a stock mediawiki vagrant VM, sadly, but avoiding the pipe sounds like a sensible thing to do anyway. [14:25:04] (03PS1) 10Thiemo Mättig (WMDE): Drop distracting, extra-blurry blur effect [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185830 [14:36:47] (03CR) 10Gilles: "Did you mean to do that in relationship to this task: https://phabricator.wikimedia.org/T71609 ? You can link to the task in the commit me" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185830 (owner: 10Thiemo Mättig (WMDE)) [15:28:14] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Pass extra GET parameter for Media Viewer preload requests - https://phabricator.wikimedia.org/T86914#985563 (10ezachte) Hi Tgr, I'm really hoping we can go forward with this imperfect solution, albeit as a stopgap measure, to be improved on later. These data h... [15:31:34] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985564 (10Steinsplitter) >>! In T86483#985414, @Magnus wrote: > I suggest the old Delinker to be turned off now. It might also help to prominently link from the... [15:46:40] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985568 (10Magnus) @Steinsplitter thanks, you're in! [15:46:55] 3MediaWiki-File-management, Multimedia: Include at least some EXIF metadata in resized pictures - https://phabricator.wikimedia.org/T20871#985569 (10Bawolff) We use an option to image magick to strip all metadata (except colour profiles). In sone images the metadata can be larger than the thumb itself. [15:49:59] 3MediaWiki-File-management, Multimedia: Include at least some EXIF metadata in resized pictures - https://phabricator.wikimedia.org/T20871#985572 (10Bawolff) Addendum: Issue specificly being that image magick doesnt offer a lot of control over which fields. Although perhaps it offers enough to fix the meat of t... [15:55:04] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985586 (10Steinsplitter) @Magnus: A last question: https://commons.wikimedia.org/w/index.php?title=User:CommonsDelinker/commands&diff=147110803&oldid=147110544... [16:02:32] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985588 (10Magnus) Well, the page is protected, and CommonsDelinker can edit it, so I assume it is already +sysop. Is there a need for CommonsDelinkerHelper anymore? [16:09:16] (03CR) 10Gilles: [C: 032] Move DeedPreview to the deed step controller [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/183654 (https://phabricator.wikimedia.org/T86227) (owner: 10MarkTraceur) [16:10:12] (03Merged) 10jenkins-bot: Move DeedPreview to the deed step controller [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/183654 (https://phabricator.wikimedia.org/T86227) (owner: 10MarkTraceur) [16:10:14] Ooh. [16:11:15] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985592 (10Steinsplitter) @Magnus: A last question: https://commons.wikimedia.org/w/index.php?title=User:CommonsDelinker/commands&diff=147110803&oldid=147110544... [16:14:34] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#985600 (10Steinsplitter) 5Open>3Resolved a:3Steinsplitter [16:14:47] 3MediaWiki-extensions-Extensions-requests, Multimedia: Re-implement functionality of old CommonsDelinker bot - https://phabricator.wikimedia.org/T86483#969276 (10Steinsplitter) a:5Steinsplitter>3Magnus [16:43:24] (03CR) 10Gilles: [C: 04-1] "I now get a blank UI after clicking on "upload more files": https://www.dropbox.com/s/0nsb06toch3oc4e/Screenshot%202015-01-19%2017.43.05.p" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/184739 (https://phabricator.wikimedia.org/T86679) (owner: 10MarkTraceur) [20:23:25] (03CR) 10Gergő Tisza: "Regardless of what we think of the blur effect, I think it is a fair point that the final animation makes the loading objectively longer. " [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185830 (owner: 10Thiemo Mättig (WMDE)) [20:29:07] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Test if replacing the blur effect with pixelation makes loading feel faster - https://phabricator.wikimedia.org/T71609#985786 (10thiemowmde) https://gerrit.wikimedia.org/r/185830 [20:29:43] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Test if replacing the blur effect with pixelation makes loading feel faster - https://phabricator.wikimedia.org/T71609#985788 (10Tgr) Thinking a bit about this, the only practical way of doing such a test is to create a video showing each loading experience, an... [20:30:17] (03PS2) 10Gergő Tisza: Drop distracting, extra-blurry blur effect [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/185830 (https://phabricator.wikimedia.org/T71609) (owner: 10Thiemo Mättig (WMDE)) [21:18:21] 3MediaWiki-File-management, Multimedia: special characters in wgDBname lead to "could not create directory" error during file uploads - https://phabricator.wikimedia.org/T87201#985839 (10fgiunchedi) 3NEW [22:05:15] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Test if replacing the blur effect with pixelation makes loading feel faster - https://phabricator.wikimedia.org/T71609#985886 (10Tgr) [22:08:33] 3MediaWiki-File-management, Multimedia: special characters in wgDBname lead to "could not create directory" error during file uploads - https://phabricator.wikimedia.org/T87201#985890 (10Tgr) [22:08:35] 3Multimedia, MediaWiki-Uploading: Files cannot be uploaded when database name contains certain characters due to regex check in FileBackendStore.php - https://phabricator.wikimedia.org/T46066#985891 (10Tgr) [22:37:06] 3Multimedia, MediaWiki-Uploading: Files cannot be uploaded when database name contains certain characters due to regex check in FileBackendStore.php - https://phabricator.wikimedia.org/T46066#985939 (10fgiunchedi) FWIW due to how misleading the resulting error message is I think at least that should be improved,... [22:40:50] 3MediaWiki-File-management, Multimedia, MediaWiki-Uploading: Non-alphanumeric characters in wgDBname break file uploads - https://phabricator.wikimedia.org/T46066#985947 (10Tgr) [22:41:13] 3MediaWiki-File-management, Multimedia, MediaWiki-Uploading: Non-alphanumeric characters in wgDBname break file uploads - https://phabricator.wikimedia.org/T46066#495516 (10Tgr) [22:41:14] 3Multimedia, MediaWiki-Uploading: no upload possible when mysql database contains a "." in its name, due to 'Swift and S3 restrictions' - https://phabricator.wikimedia.org/T46949#985951 (10Tgr) [22:43:29] 3MediaWiki-File-management, Multimedia, MediaWiki-Uploading: Non-alphanumeric characters in wgDBname break file uploads - https://phabricator.wikimedia.org/T46066#985957 (10Tgr) Replacing dots with underscores would be an easy solution, as long as we can ensure uniqueness. [22:53:18] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Pass extra GET parameter for Media Viewer preload requests - https://phabricator.wikimedia.org/T86914#985964 (10Tgr) The main problem is not that this is imperfect, it's that it is going to hurt performance. If you use different URLs for the same image in diffe... [22:55:25] 3MediaWiki-extensions-MultimediaViewer, Multimedia: Pass extra GET parameter for Media Viewer preload requests - https://phabricator.wikimedia.org/T86914#985966 (10Tgr) [22:55:26] 3Analytics, Multimedia: Per-file view stats - https://phabricator.wikimedia.org/T77541#985965 (10Tgr) [23:09:25] 3MediaWiki-File-management, Multimedia: Include at least some EXIF metadata in resized pictures - https://phabricator.wikimedia.org/T20871#985970 (10Hasenlaeufer) >>! In T20871#985569, @Bawolff wrote: > We use an option to image magick to strip all metadata (except colour profiles). In sone images the metadata c...