Page MenuHomePhabricator

VIPS scaled thumbnails don't have a comment with a link to the file description page
Open, NormalPublic


As part of T30135: Review and deploy VipsScaler extension we added the option to set the comment of a thumbnail with the file descriptionpage url that it came from, as we did for the imagemagick scalar. It seems however that the option was never actually enabled in the configuration for the sites, and that we have been generating thumbnails without 'backtrack' urls ever since then...

That's not really what was intended.

This relates to the "setcomment" options of wgVipsOptions



Event Timeline

bzimport raised the priority of this task from to Normal.Nov 22 2014, 3:34 AM
bzimport set Reference to bz69336.
bzimport added a subscriber: Unknown Object (MLST).
TheDJ created this task.Aug 9 2014, 1:19 PM

Hmm. Is exiv2 on the servers? I think the option depends on that.

Jdforrester-WMF moved this task from Untriaged to Backlog on the Multimedia board.Sep 4 2015, 6:31 PM
Restricted Application added subscribers: Matanya, Aklapper. · View Herald TranscriptSep 4 2015, 6:31 PM
matmarex updated the task description. (Show Details)Jan 26 2016, 4:04 PM
matmarex set Security to None.
matmarex removed a subscriber: wikibugs-l-list.
Restricted Application added a subscriber: JEumerus. · View Herald TranscriptJan 26 2016, 4:04 PM
TheDJ updated the task description. (Show Details)Jan 26 2016, 4:06 PM
Dereckson added a project: Operations.
Dereckson added a subscriber: Dereckson.

@Bawolff So next step is to install pyexiv2 on mediawiki::packages::multimedia?

Dereckson claimed this task.Sep 9 2016, 3:23 AM

[ Taking this bug, as I can sheperd it, but I'll mainly rely on @Bawolff and @TheDJ  support for their extension knowledge to do this. ]

Change 309505 had a related patch set uploaded (by Dereckson):
Install exiv2 to mediawiki::packages::multimedia

So the question is to know if we want to do this with exiv2, or if we want to update the VIPS extension code to support exiftool, already installed on multimedia servers.

@TheDJ Do you have an opinion about this?

@Dereckson right. I have no opinion on this. My ticket was only an observation about functionality that we lost compared to the pre-vips stages, and I have no specific preferences in how it gets resolved.

Change 309505 abandoned by Dereckson:
Install exiv2 to mediawiki::packages::multimedia

No preference for this solution

Dereckson removed Dereckson as the assignee of this task.Nov 21 2016, 11:18 AM

To summarize, it was offered to solve right now the issue installing the missing software. This solution doesn't have any enthusiasm from the current extension maintainers.

An alternative solution offered is to improve the extension code to use exiftool, already deployed on the scalers, to perform this task.

fgiunchedi added a subscriber: fgiunchedi.

Removing Operations as there isn't anything to do ATM, though Thumbor might be a good place for a wishlist in this case