Closed (fixed)
Project:
Drupal core
Component:
user.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 May 2005 at 23:47 UTC
Updated:
8 Oct 2005 at 22:20 UTC
Jump to comment: Most recent file
Because you can enable/disable user pictures in the PHPTemplate config pages, that option should not exist under admin/users. And if PHPTemplate shouldn't handle this option, it should be removed from PHPTemplate's config screens. This option should be set in one place, and one place only.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | picture_0.patch | 1.22 KB | Steven |
| #1 | picture.patch | 1.07 KB | Steven |
Comments
Comment #1
Steven commentedI disagree... the option in admin/users controls whether users can upload a picture. This image is always displayed in their profile and this is a user.module feature.
The PHPTemplate options on the other hand control whether pictures are displayed next to nodes resp. comments. This is a theme feature.
The two are different and complement each other. Sometimes you may want users to have pictures, but not have them displayed next to each post. In that case you could even let users upload larger pictures of themselves as they don't need to fit snugly next to text.
However you are partially right: it does not make sense to show the PHPTemplate options if user pictures are not allowed, so we should only make them appear in the right situation. I've attached a patch for this.
Comment #2
Steven commentedComment #3
Steven commentedAh, it seems we prefer to disable the toggles rather than hide them. It is probably for the best (I've always read you should disable options if they can be enabled by the user in this session).
I had to generalize the check for search.module a bit, the code is cleaner now.
Comment #4
factoryjoe commented+1 for the solution and to graying out options.
Comment #5
Steven commentedCommited to HEAD.
Comment #6
(not verified) commentedComment #7
(not verified) commentedComment #8
(not verified) commentedComment #9
(not verified) commentedComment #10
(not verified) commented