Last modified: 2014-01-10 03:42:18 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T44964, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 42964 - Add support for more types of Flickr URLs to UploadWizard
Add support for more types of Flickr URLs to UploadWizard
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Mateusz Maćkowski
http://www.google-melange.com/gci/tas...
gci2013
: easy
Depends on:
Blocks: 43450
  Show dependency treegraph
 
Reported: 2012-12-11 22:25 UTC by Ryan Kaldari
Modified: 2014-01-10 03:42 UTC (History)
10 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Ryan Kaldari 2012-12-11 22:25:23 UTC
Right now UploadWizard supports individual photo URLs and photoset URLs. It would be good to add support for photostream URLs, group URLs, etc. Each of these use a different API method in Flickr, so we would need to customize the code for each.
Comment 1 Nischay Nahata 2012-12-12 16:50:40 UTC
Related comment https://bugzilla.wikimedia.org/show_bug.cgi?id=28731#c13
Comment 2 Waldir 2012-12-12 18:20:48 UTC
(In reply to comment #1)
> Related comment https://bugzilla.wikimedia.org/show_bug.cgi?id=28731#c13

Adding them here directly for convenience:

* user collections (groups of albums (sets), potentially nested)
  http://www.flickr.com/photos/USER_ID/collections/COLLECTION_ID/
* user photostreams
  http://www.flickr.com/photos/USERID/
* group pools
  http://www.flickr.com/groups/FOO/pool/
  and
  http://www.flickr.com/groups/FOO/pool/USERID/
* user galleries (albums of photos by many people, curated by a user)
  http://www.flickr.com/photos/USER_ID/galleries/GALLERY_ID/
* user favorites
  http://www.flickr.com/photos/USERID/favorites/
Comment 3 Mark Holmquist 2013-01-14 18:49:41 UTC
Would it be fair to call this an easy bug? It seems like it would be pretty simple to copy the handling code that exists and adapt it to each endpoint. Confirm, deny, or whatever, it would be cool to have extra easy things for new developers (we only have two or three right now).
Comment 4 Nischay Nahata 2013-01-15 04:55:12 UTC
I think this is going to be easy, anyways I am also ready to help with this.
Comment 5 Waldir 2013-01-15 11:41:34 UTC
I would like to give it a try. Where's the existing Flickr-related code (i.e. what files need to be edited)?
Comment 6 Andre Klapper 2013-01-21 08:59:53 UTC
(In reply to comment #5)
> I would like to give it a try. Where's the existing Flickr-related code (i.e.
> what files need to be edited)?

Mostly under mediawiki/extensions/UploadWizard/resources/mw.FlickrChecker.js in gerrit.wikimedia.org (see http://www.mediawiki.org/wiki/Gerrit ) plus general settings in mediawiki/extensions/UploadWizard/UploadWizard.config.php
Comment 7 richa jain 2013-03-20 19:56:28 UTC
just tried out adding support for gallery urls, will add other urls soon 
https://gerrit.wikimedia.org/r/#/c/54897/
Comment 8 Strainu 2013-09-25 19:39:04 UTC
Tags would also be useful:
https://secure.flickr.com/photos/USER/tags/TAG/
Comment 9 Nemo 2013-12-26 11:02:51 UTC
(In reply to comment #3)
> It seems like it would be pretty
> simple to copy the handling code that exists and adapt it to each endpoint.

(In reply to comment #4)
> I think this is going to be easy, anyways I am also ready to help with this.

Thanks Mark and Nischay, let's add it as GCI task. :)
Comment 10 Quim Gil 2013-12-27 06:24:41 UTC
Task created. Thank you for proposing GCI tasks!
Comment 11 Gerrit Notification Bot 2013-12-27 07:20:04 UTC
Change 103967 had a related patch set uploaded by Mayankmadan:
Adding support for more types of flickr URLs

https://gerrit.wikimedia.org/r/103967
Comment 12 Gerrit Notification Bot 2013-12-30 17:19:09 UTC
Change 103967 abandoned by Mayankmadan:
Adding support for more types of flickr URLs

Reason:
Not working on this anymore

https://gerrit.wikimedia.org/r/103967
Comment 13 Gerrit Notification Bot 2014-01-03 22:20:42 UTC
Change 105393 had a related patch set uploaded by M4tx:
Add support for more Flickr URLs

https://gerrit.wikimedia.org/r/105393
Comment 14 Mateusz Maćkowski 2014-01-03 22:23:11 UTC
I uploaded the patch. It wasn't so simple to implement the feature, though :) I wouldn't call it "easy" task; but maybe I just have insufficient knowledge...
Comment 15 Nemo 2014-01-06 19:29:05 UTC
(In reply to comment #14)
> I uploaded the patch. It wasn't so simple to implement the feature, though
> :) I
> wouldn't call it "easy" task; but maybe I just have insufficient knowledge...

Wow, we've seen some serious work here indeed, great! Probably the URL types were considered normally "easy" to add each by itself, I agree that doing them all is another matter.
Comment 16 Gerrit Notification Bot 2014-01-09 01:43:35 UTC
Change 105393 merged by jenkins-bot:
Add support for more Flickr URLs

https://gerrit.wikimedia.org/r/105393
Comment 17 Nischay Nahata 2014-01-10 03:42:18 UTC
Man, all of this in one GCI task? that is amazing. Yes, I thought that adding individual types wouldn't be hard (with help of course) Anyways closing this as RF

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links