I would like to be able to display photos from a Flickr group's pool, rather than just by username.

Comments

markDrupal’s picture

same here

deantregenza’s picture

Yes, I agree this would very a very good feature - especially if we could also get a cloud of tags along the lines of how flickrhood/x/tags works.

Rebecca@category.greenash.net.au’s picture

Me three!

puregin’s picture

I have some alpha quality code that displays a group pool block. I'll clean it up and post a patch later today.

puregin’s picture

Version: 6.x-1.x-dev » 5.x-1.x-dev
Component: flickr_block » Code
Assigned: Unassigned » puregin
Status: Active » Needs review
StatusFileSize
new8.43 KB

I'm attaching a patch which enables a group_pool block.

Currently, there is at most one group per site (or more precisely, per theme)

There is no mechanism for filtering images from the group pool.

puregin’s picture

StatusFileSize
new8.43 KB

Hmm, let's try that again, without the hardcoded group id :)

puregin’s picture

StatusFileSize
new4.83 KB

Argh. Either not enough caffeine today, or too much. This should do it... please ignore the previous patches

Christefano-oldaccount’s picture

Hey, cool stuff. Can't wait to try this.

Anonymous’s picture

I'll test it out tomorrow. Thanks.

Christefano-oldaccount’s picture

Title: groups » patch which enables a Flickr group pool block

Changing title to make it easier to find this.

drewish’s picture

purgin, that looks pretty good the only change i'd make just based on looking at the code is renaming _flickr_block_group_pool_recent() to _flickr_block_group_recent()... i haven't looked at the api in a while but i don't think group comments are available through it.

Anonymous’s picture

I tested the latest patch on my local machine and it seems to work fine. Thanks.

puregin’s picture

drewish, i named the function after the flickr api object: 'group.pool'...

I'll happily look at this again if anyone has any issues to report or functionality they would like added.

andrewlevine’s picture

After a quick glance this looks good. How hard would it be to add multiple group functionality? Maybe accept multiple pool ID's separated by commas and explode them. Thanks for the contribution.

drewish’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev

sorry to have ignored this for so long, if someone could re-roll this for HEAD i'd appreciated it. it needs to go into the 6.x and then be backported.

charlieman’s picture

StatusFileSize
new4 KB

Hi, I've also created a patch for this (I didn't saw this thread and patch). Anyway, mine adds two group blocks (random and recent) and extends the already existing functions instead of creating a new one for the groups. Hope someone finds it helpfull.

Anonymous’s picture

Status: Needs review » Needs work

Couldn't apply the patch .

paulbookers-computer:~/Subversion/Development/drupal6/sites/all/modules/flickr paulbooker$ patch -p0 < flickr_group_blocks.patch
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/block/flickr_block.module b/block/flickr_block.module
|index 1e2e09c..c8bc63b 100644
|--- a/block/flickr_block.module
|+++ b/block/flickr_block.module
--------------------------

Artascope’s picture

anyone have the idiots version of how to do this?

am I cutting, pasting uploading or wining and dining?

thanks for being supportive of the newbies!

thanks!

Anonymous’s picture

Status: Needs work » Closed (fixed)

This has been partially implemented in #318722

Would you please open a new feature request and roll a further patch to implement new features , thanks

Paul