should photoset filter show a photoset?

Same question here.

+1 ;-)

i have the same question.

because i don't really see the point in using flicker-photoset tag when i can just link to the photoset

Hello,

I wanted to ask whether or not flickr photosets are supposed to embed the whole gallery in Drupal, or just the first photo with a link to the gallery.

If not, do you know of a way to embed a whole gallery? That's what I'm really looking to do.

Great module, thanks for maintaining it and developing it.

Thanks,
Jeremy

See the flickr_sets module

#4 - andrewlevine response is not relevant. This desired functionality is not in the module.

I like others want this functionality and there is a feature request, started a long time, with several patches that haven't been applied. I'm trying to sort through it today to see if any of the patches are usable:

http://drupal.org/node/152792

Shai

I'd like this too. Meanwhile, I'm using functionality from Flickr Slideshow Generator at http://fabiocavassini.com.ar/SlideShowGenerator.html to generate these slides. It surprises me that there isn't support for this in a filter already.

Getting the same thing in the 6x-1 Alpha1 version.
I have the blocks working, just not the sets thing.

Hi,

the problem may rely in flickr_filter.

At the end of filter/flickr_filter.module we have

function theme_flickr_filter_photoset($ps, $owner, $size, $attribs) {
  return theme('flickr_photoset', $ps, $owner, $size, $attribs);
}

and in flickr.module:

function theme_flickr_photoset($ps, $owner, $size, $attribs = NULL) {
  $img = flickr_img($ps, $size, $attribs);
  $photo_url = flickr_photoset_page_url($owner, $ps['id']);
  $title = is_array($ps['title']) ? $ps['title']['_content'] : $ps['title'];
  return l($img, $photo_url, array('attributes' => array('title' => $title), 'absolute' => TRUE, 'html' => TRUE));
}

So the functions in sets/ aren't used at all when using the [flickr-photoset:...] filter.

Can we change that please?

I'll try myself, but I don't know drupal very well yet, so it might take time.

Thanks,
Antonio

Hi,

I found a solution that *works for me* (TM) :)
This now becomes a feature request.

Attaching the patch here. The code duplicates what's also in sets/flickr_sets.module.

Thanks,
Antonio

Any updates on this?

The above patch doesn't work for me since I'm on 5 I guess.

Very sorry, but I dont know how to use a patch.

can you please make a tarball of all the files after your patch and attach here?

Thank you very much

@bgogoi: http://drupal.org/patch

so easy! Thanks Andrew :)

Does the patch re #9 work for people then. It doesn't make any difference for me (5.x).

This patch works for me, except "size" feature - clicking on image always gives link to "medium" size flickr image. Does "size" works for you?

hi naiptol,

can you test the new attached version?
It contains this change:

--- flickr-expand_photoset_rendering.patch	2008-10-14 19:32:07.000000000 +0200
+++ flickr-expand_photoset_rendering_v2.patch	2008-10-14 19:32:06.000000000 +0200
@@ -25,7 +25,7 @@ diff -bruN flickr.orig/flickr.module fli
 +  foreach ((array) $photos['photoset']['photo'] as $photo) {
 +    //insert owner into $photo because theme_flickr_photo needs it
 +    $photo['owner'] = $owner;
-+    $output .= theme('flickr_photo', $photo, 's');
++    $output .= theme('flickr_photo', $photo, $size);
 +  }
 +  $output .= '</div>';
 +  $output .= theme('pager', NULL, variable_get('flickr_photos_per_page', 20));

Hi, ao2,
Thank you for patches! :)

You can see test results in http://eminute.net/en/flickr_test
Drupal 6.5 , Lightbox2 6.x-1.8, flickr module 6.x-1.x-dev (2008-Sep-30) flickr-expand_photoset_rendering_v2.patch.
(I will upgrade versions in the future, so this is current moment)

There is the same flickr photoset with diferent sizes in filter. In previous patch version (flickr-expand_photoset_rendering_v1.patch) all of them looked like with 'size=s'. Now sizes are different.

Only one moment. There are 2 sizes actually. One is "what sizes you see in page showing all photoset", and another is "what size image you get when you click one of them".

Currently when I click on picture in photoset, I get 'medium' sized image with s, t, m, -, and 'original' with b, o. (my original pictures are 800px on longest size).
I think 'original' would be better, than 'medium', becouse 'medium' is too small, and if 'original' is too large, it is possible to manage this with Lightbox2 settings.

Also one strange effect:
I set 'Continuous Galleries' in Lightbox2, and when I watch the same gallery 2-nd time, looks like I get 'Original' size, except last image in gallery, which shows 'broken image', if I go further, 3-d time I get 'medium' again, 4-d - "original', except last image, and so on. It looks like bug somewhere.

I noticed in the code you can change how many pictures per page to display for the paging feature. Unfortunately when I load an album with more than the said amount of photos, no pagers show up. It only displays the default number of photos you have set in the code (20). You can change this to allow for more but I have albums with 50+ pictures that I didnt want to load on one page alone. Any thoughts on how to get the paging to work?

Ok. So, I noticed that there is a flickr-photoset class div wrapped around the images. I thought that I would simply put a margin around the images using CSS to get some space between them. However, I was having trouble getting it to recognize my CSS changes. I couldn't figure out why it wasn't working. Turns out that the div class is misspelled. It actually says "fickr-photoset". This should be corrected before committing it to the dev branch.

Right now I'm using Flickr 6.x-1.0-beta4.
The problem that only one picture is being displayed in photosets on a page or in articles persists?
Are there any progresses?

Sorry for my last blank reply.

I've made a patch for flickr.admin.inc that enables more than 20 photos of a photoset in every node created with patched flickr.module thanks to ao2 (#16).

Both work well.

Instead of duplicating code in flickr.module, I have done the following in flickr_filter.module

function theme_flickr_filter_photoset($ps, $owner, $size, $attribs) {
    //return theme('flickr_photoset', $ps, $owner, $size, $attribs);
// chandrabhan
  $photo_arr = flickr_set_load($ps['id']);
$set_info['owner'] = $owner;
  $set_info['size'] = $size;
  return theme('flickr_sets_photoset', $uid, $per_page, $photo_arr, $set_info);
}

And also in flickr_sets.module

function theme_flickr_sets_photoset($uid, $per_page, $photo_arr, $set_info) {
  $output =
    theme('pager', NULL, variable_get('flickr_photos_per_page', 20));
  $output .= "<div class='fickr-photoset'>\n";
  foreach ((array) $photo_arr['photoset']['photo'] as $photo) {
    //insert owner into $photo because theme_flickr_photo_box/flickr_photo_page_url needs it
    $photo['owner'] = $set_info['owner'];
    //$output .= theme('flickr_photo_box', $photo, 'm');
	// chandrabhan
	$output .= theme('flickr_photo_box', $photo, $set_info['size']);
  }
  $output .= '</div>';
  $output .=
    theme('pager', NULL, variable_get('flickr_photos_per_page', 20));
  return $output;
}

code after // chandrabhan is added

Is this committed? I have the latest dev version of the module, but I only see one image from my photoset in my node after inputting the photoset code in the Flickr CCK field.

I'm not sure this is actually fixed. I'm having difficultly getting a photo set to display - like others in the comments before me, all I'm seeing is a single photo from the photoset.

Here's what I have:

1. Modules enabled: Flickrfield, Flickr, Flickr Sets (dev version from 2009-June-24).

2. I added a "Flickr Photo" field to the "blog" content type with all the default settings.

3. I created a new blog post and in the Flickr fieldset, I set:
- Item Type: photoset
- ID: 72157622689719750
- User ID: MichaelFlickr
(enjoy the photos of my kids!)

Upon viewing the node, I don't see the entire photoset, just the first photo from the set. The "Source: Flickr" link below the photo links to the photoset.

What am I missing in order to pull in all the images from the photoset?

Thanks in advance,
-mike

Just figured out what I was doing wrong - hopefully this will help save somebody else some time...

There are 2 types of CCK fields included with the FlickrField module - "Flickr Photo" and "Flickr photo set". If you want to be able to display an entire set in a node, you need to choose the "Flickr photo set" type (obviously). I was banging my head against the wall because I inherited a site with the wrong type selected from the start.

-mike

changing issue title back - @ultimike Thanks for helping people with your comment. However, note when you change the "Issue title" field it changes for the whole issue, not just your comment.

Is this fixed? I'm using the latest dev version. Only one photo from my photoset is displaying. I've added a photoset cck field.
Which handler should I be selecting in the display fields page? I see there's an option for embedded slideshow, but I don't want that. I want a main photo and thumbs. Can anyone help? Thanks.

I see that the flickr photo field (not photoset field) also accepts a photoset ID. So I tried that instead and now all I get is just the main photo, with the text link below Source: flickr to the set on flickr.

But the photos from the photoset are all displayed displayed when using the filter.

So is this supposed to work with the cck field, or only the filter?

Thanks

subscribing - would this issue be the same in the v5 of the Flickr module? My images (using the filter code) opens the photoset in a new browser window.

subscribing

Oh, and actually I think it should be a bug report -- the feature is there, it's just not working correctly.

I've written a patch which makes flickrfield display all photos from a photoset if the flickr_sets module is enabled. It can probably be done in a nicer way so if you can improve it, please do so. This patch also prints the photoset's description.

The missing patch...

Yes, it applies against 6.x-1.x-dev 2009-Jun-25.

I'm attaching a new version of the patch - I've changed the HTML a bit (it's now easier to theme to look like original Flickr photosets) and made it link to the original photo.

Demo: http://archefoxtrot.com/en

I am not sure why you want me to write all this. Isn't it clear what my patch does? The "expected results" were for the module to display a complete photoset when using the supplied CCK field and this patch addresses this issue. However, I am not an experienced PHP programmer so there may be a cleaner way of doing this.

OK, thanks.

I'm not sure this works with panels. When I create a custom node view and add the flickr photoset cck field to a panel I can only choose to display the "primary set photo {at different sizes}"

I'd like to display the entire gallery.