Closed (fixed)
Project:
Signup
Version:
5.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
16 Jun 2006 at 16:13 UTC
Updated:
28 Mar 2007 at 07:46 UTC
Jump to comment: Most recent file
it's kinda silly that the "Users can view signups" setting on admin/settings/signup exists as a global, site-wide setting. it should just be another permission. i'm tempted to backport this to 4.6, even.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | signup_view_perm_5x.patch_2.txt | 4.92 KB | dww |
| #4 | signup_view_perm_5x.patch_1.txt | 4.88 KB | dww |
| #3 | signup_view_perm_5x.patch.txt | 3.09 KB | dww |
Comments
Comment #1
leoklein commentedBackport! Backport! The crowds love it.
Comment #2
mdowsett commentedi don't have that setting at /admin/settings/signup...I am using 4.6 tho...could that be why?
Comment #3
dwwComment #4
dwwnow, with a DB update to convert old sites. if the old setting was enabled, this automatically adds the new perm to the "authenticated user" role and warns the admin to consider customizing this permission. either way, the old variable is deleted from the {variable} table.
Comment #5
dwwafter extensive discussion w/ webchick and add1sun in IRC, we think this new perm should be called "view all signups", not just "view signups", since we'll want to add a "view signups for own content" permission, too: http://drupal.org/node/126632
Comment #6
dwwComment #7
dwwbetter yet, see the patch in http://drupal.org/node/69283#comment-211563.
Comment #8
dwwthis change made it in with the patches i just commited for http://drupal.org/node/69283 in HEAD, DRUPAL-5, and DRUPAL-4-7.
Comment #9
(not verified) commented