"View all imagecache presets" permission [#558820]

It's useful to be able to grant access to view resized images on a per-preset basis, but IMO it's more common to want to make all presets available to somebody. Having to remember to add the new preset permission every time one's added is a chore - would it be hard to add a 'view all imagecache presets' permission?

Comment	File	Size	Author
#2	imagecache-view_all_permission.patch	993 bytes	BWPanda
#1	all_presets_patch.diff	980 bytes	kingandy

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

kingandy CreditAttribution: kingandy commented 11 November 2009 at 17:15

File	Size
all_presets_patch.diff	980 bytes

Investigation suggests it could be as easy as
a) adding the permission and
b) adding an "OR" clause to the permission check inside imagecache_cache_private().

Find attached a patch which does this. Fail [y/n]?

Comment #2

BWPanda CreditAttribution: BWPanda commented 23 June 2010 at 02:39

Version:	6.x-2.0-beta10	» 6.x-2.x-dev
Status:	Active	» Needs review

File	Size
imagecache-view_all_permission.patch	993 bytes

I'd also love to see something like this committed!

Updated patch for latest dev version.

Comment #3

Bartezz CreditAttribution: Bartezz commented 3 July 2011 at 23:16

+1 I'd like this too...

I'd would also be nice if imagecache would somehow make images for which a user has no permission not being listed. Now I get a bunch of broken images in a gallery, I'd rather see an empty gallery :)