Page MenuHomePhabricator

License Info for Media Viewer
Closed, ResolvedPublic

Description

Media-Viewer-Lightbox-View-Screenshot-Alpha-10-24-2013.png

If a file is licensed under a Creative Commons license, we want the Media Viewer to display its 'CC' label (e.g. 'CC-BY-SA-3.0'), as described in our Mingle feature requirement (1).

But this doesn't seem to be working as intended on our multimedia alpha prototype (2). For example, the aquarium photo in the attached screenshot (3) is licensed under 'CC-BY-SA-3.0' (hidden category), but that label is not displayed on the alpha prototype.

Instead, it just says 'Licensing Info'. Even if it were not CC, the recommended copy for generic links is 'See license info', as requested on Mingle.

More info about Media Viewer on our project page (3).

(1) https://mingle.corp.wikimedia.org/projects/multimedia/cards/34

(2) https://commons.wikimedia.org/wiki/File:Media-Viewer-Lightbox-View-Screenshot-Alpha-10-24-2013.png

(3) https://commons.wikimedia.org/wiki/File:Wikimedia_Foundation_2013_All_Hands_Offsite_-_Day_2_-_Photo_24.jpg

(4) https://upload.wikimedia.org/wikipedia/commons/f/fe/MediaViewer_Sept9.png

(5) https://www.mediawiki.org/wiki/Multimedia/Media_Viewer


Version: unspecified
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=56587

Attached:

Media-Viewer-Lightbox-View-Screenshot-Alpha-10-24-2013.png (888×1 px, 630 KB)

Details

Reference
bz56143

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:34 AM
bzimport added a project: MediaViewer.
bzimport set Reference to bz56143.

Actually a bug in CommonsMetadata - it's returning "LicenseUrl" instead of "License" on Commons images. We either need to translate LicenseUrls into licenses, or use the existing solution we have based on categories...maybe CommonsMetadata can't see hidden categories on a remote site? Investigate.

Assigning Gergo.

CommonsMetadata is intended to run on the central repository, but in our test setup it actually runs on the frontend wiki. It still kind of works, but some things are not accessible for a remote file, and remote categories are one of them. So License (which is based on categories) is not set, only LicenseUrl (which is based on the description page).

So wait.

Will this work once the central repo has CMD on it? Should I try to handle LicenseUrl too?

OK, we now fall back to the file page in _all_ cases. Try the new patch out for size. I'll close this bug since it never actually hit master.

https://gerrit.wikimedia.org/r/90267

Reopening this bug, because it is still happening on beta for all the photos I uploaded and featured on this page (Hong Kong, Aquarium, Kerala beach, Wikimedia team):

http://en.wikipedia.beta.wmflabs.org/wiki/Lightbox_demo

All these photos should display a 'CC-BY-SA-3.0' label in light box view, but are saying 'License info' instead.

Unfortunately beta sites use the real Commons, not Commons beta.
So this is more of a bug in our test setup than in MediaViewer/CommonsMetadata.

I can reproduce this problem on MediaWiki.org as well, which I don't believe is Beta.

None of the CC images I posted on this MW.org are showing the correct CC-BY-SA label: (aquarium, nautilus, Kerala beach)

https://www.mediawiki.org/wiki/Lightbox_demo

To be clear, I and other user expect the 'Licensing information' label to be replaced by 'CC-BY-SA-3.0' or whatever license we put in when we uploaded a file, as specified here:

https://mingle.corp.wikimedia.org/projects/multimedia/cards/34

This is an important request, which is going to be echoed over and over again by other users until we fulfill it.

Apparently Commons is a ForeignDBViaLBRepo, not a ForeignAPIRepo, and CommonsMetadata cannot handle that. This will be complicated...

  • Bug 56587 has been marked as a duplicate of this bug. ***

Change 93889 had a related patch set uploaded by Gergő Tisza:
Use description HTML structure instead of categories to identify license

https://gerrit.wikimedia.org/r/93889

Change 93889 merged by jenkins-bot:
Use description HTML structure instead of categories to identify license

https://gerrit.wikimedia.org/r/93889

This mostly works now; closing it in favor of more specific bugs.

Gilles raised the priority of this task from High to Unbreak Now!.Dec 4 2014, 10:11 AM
Gilles moved this task from Untriaged to Done on the Multimedia board.
Gilles lowered the priority of this task from Unbreak Now! to High.Dec 4 2014, 11:20 AM