here is another regex that needs to be added to _media_flickr_media_parse in media_flickr.utilities.inc

'@flickr\.com/photos/([0-9A-Za-z\@\&\$_-]+)/([0-9]+)/in/pool-([0-9A-Za-z\@\&\$_-]+)@i',

with added regex:

function _media_flickr_media_parse($url, $options = array()) {
// The following are single images.
$patterns = array(
'@flickr\.com/photos/([0-9A-Za-z\@\&\$_-]+)/([0-9]+)/in/photostream@i',
'@flickr\.com/photos/([0-9A-Za-z\@\&\$_-]+)/([0-9]+)/in/set-([0-9]+)@i',
'@flickr\.com/photos/([0-9A-Za-z\@\&\$_-]+)/([0-9]+)/?$@i',
'@flickr\.com/photos/([0-9A-Za-z\@\&\$_-]+)/([0-9]+)/in/pool-([0-9A-Za-z\@\&\$_-]+)@i',
);

when i get time i will create a patch to do this

Comments

ransom’s picture

subscribe

ransom’s picture

Version: 7.x-1.0-alpha1 » 7.x-1.x-dev
StatusFileSize
new683 bytes

Don't look like a normal patch but done with git diff *shrug*

Leeteq’s picture

Priority: Normal » Major

Media 7.x-1rc was just released, time to get this one in?

steinmb’s picture

Title: Unable to parse photos with /in/pool- in the url » Unable to parse photos in Flickr groups pool
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new1007 bytes

Tested OK. Adds support to grab images from a Flickr group pool. Example, adding pictures from https://www.flickr.com/groups/drupalcon_london

Patch rolled against 7.x-2.x thought should also be added to 7.x-1.x.

  • steinmb committed 9cfb7e7 on 7.x-2.x
    Issue #1236624 by ransom, steinmb: Enable module to parse photos from...

  • steinmb committed 82229ab on 7.x-1.x
    Issue #1236624 by ransom, steinmb: Enable module to parse photos from...
steinmb’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.