Background
Current the flickr field asks for 3 different user inputs:
Item Type
Id
and User Id.

Motivation

  • All of this information can be supplied from a simplified, unique identifier - URL
  • Even when I know the values I want in the fields, it's cumbersome to copy and paste them from the URL, or god forbid I have to go search for my nsid.
  • The flickr URL contains enough information for flickr API to retrieve all the necessary info for the asset without any additional input from the end user
  • Regardless of whether these 3 fields are stored in the database, exposing the user to the underlying schema is confusing.

Solution
Only collect the asset URL from the user (set URL or photo URL), parse the photo or set id from it.

In this patch

  • in flickr.inc: added flickr_get_info_by_url(). Given the URL to a flickr asset (set or photo), return info about that asset.
  • in flickrfield.module: Update flickrfield_process() to provide a single text field for user input.
  • in flickrfield.module: Add flickrfield_field() with validate and presave operations to populate the existing columns

Note that flickrfield schema is not changed by this patch.

CommentFileSizeAuthor
#12 flickrfield_by_url_for_D7-1170010-12.patch6.13 KBlolandese
#11 flickrfield_by_url_for_D7-1170010-11.patch11.42 KBlolandese
#7 flickrfield_by_url-1170010-7.patch8.78 KBlolandese
#3 flickrfield-by-url-1170010-3.patch5.95 KBaaronbauman
#2 flickrfield-by-url-1170010-dev.patch5.96 KBaaronbauman
#1 flickrfield-by-url-1170010.patch5.6 KBaaronbauman

Comments

aaronbauman’s picture

aaronbauman
he/him
commented
StatusFileSize
new flickrfield-by-url-1170010.patch5.6 KB
aaronbauman’s picture

aaronbauman
he/him
commented
StatusFileSize
new flickrfield-by-url-1170010-dev.patch5.96 KB

Re-rolled at dev.

aaronbauman’s picture

aaronbauman
he/him
commented
StatusFileSize
new flickrfield-by-url-1170010-3.patch5.95 KB

mis-typed array indexes - values were being lost on node edit.

chaloum’s picture

chaloum commented

Do you have this patch of the D7 version of flickr?

aaronbauman’s picture

aaronbauman
he/him
commented

No, I switched to the filefield_sources, which is more suited to my simple use case.

lolandese’s picture

lolandese commented
Issue summary: View changes
StatusFileSize
new flickrfield_by_url-1170010-7.patch8.78 KB

Thanks for this. Apologies for the late reaction.

Rerolled against the current dev. With some minor changes it seems to work as expected.

lolandese’s picture

lolandese commented
Status: Needs review » Fixed

Committed with attribution. See http://drupalcode.org/project/flickr.git/commitdiff/ce9c349.

Thanks.

lolandese’s picture

lolandese commented
Status: Fixed » Patch (to be ported)

To port to D7.

lolandese’s picture

lolandese commented
Title: Make it more user-friendly » Parse the user id (nsid) and photo/set id from the Flickr page URL
Version: 6.x-1.x-dev » 7.x-1.x-dev

Better title and changed version to D7.

lolandese’s picture

lolandese commented
Status: Patch (to be ported) » Needs work
StatusFileSize
new flickrfield_by_url_for_D7-1170010-11.patch11.42 KB

Attached patch for now only adds the unified URL field and validates it. No form submission processing yet.

Thanks.

lolandese’s picture

lolandese commented
StatusFileSize
new flickrfield_by_url_for_D7-1170010-12.patch6.13 KB

Plain reroll of previous patch against latest dev. Nothing else.

The field does not seem to get validated nor saved.

lolandese’s picture

lolandese commented
Component: flickr_field » flickrfield
lolandese’s picture

lolandese commented

Meanwhile the provided example at https://www.drupal.org/node/2416565#comment-9578885 can be used by those that are looking to simplify the Flickr Field.

lolandese’s picture

lolandese commented
Issue tags: +advanced