It would be nice if the Flickr Settings section of a user profile was removed if the user did not have permission to view their own Flickr photos. If they cannot view their own photos, why do they need the option to add their Flickr ID?

CommentFileSizeAuthor
#1 flickr-settings-permissions.patch534 bytestms8707056

Comments

tms8707056’s picture

tms8707056 commented
Version: 6.x-1.0-alpha1 » 6.x-1.x-dev
Assigned: Unassigned » tms8707056
Status: Active » Needs review
StatusFileSize
new flickr-settings-permissions.patch534 bytes

This is my first patch... so I'm not sure if I did it right. I have added a permissions check for displaying the Flickr Settings section on user profiles. If a user does not have view own photos permission then the option to add their Flickr ID disappears from their profile edit page.

Grumot’s picture

Grumot commented

I installed it at mines.... It seems to work fine ;-)
Thanks!

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Closed (duplicate)

This patch looks to be already incooporated in the latest development release on drupal.org


function flickr_user($op, &$edit, &$account, $category = NULL) {
if ($op == 'form' && (user_access('view own flickr photos') || user_access('administer flickr'))) {

Best, Paul

hnln’s picture

hnln commented

Does this also apply for the drupal 5 release (since the latest dev release for drupal 5 doesn't match the dates of these comments).

If not, will this be incorporated ?

Tnx,

HnLn