Closed (fixed)
Project:
Minutes
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
25 Oct 2006 at 16:28 UTC
Updated:
8 Nov 2006 at 13:16 UTC
I love this module!
Enabling OG (organic groups) functionality is a big deal for me, so I'll be trying lenzjo's code at http://drupal.org/node/70740
In the meantime and beyond, I would really prefer not having every user's email address listed in the minutes creation form. It's a definite privacy issue on my sites with many users (not even half my users have their contact form enabled).
Comments
Comment #1
pukku commentedHi! Right now, I'm busy with a number of other things I'm working on, so I won't be able to get to this for a while (maybe in December or January). But in the short term, if you go into minutes.module, and look around line 418 (in the function _minutes_uid_to_display), you will see the query used to retrieve the user information from the database. You can edit this to suppress the email address.
To solve this problem, I will probably grab some code I wrote for my module OI that allows you to define the display string using any profile field.
Comment #2
Christefano-oldaccount commentedThanks for the response. I think that as soon as I can get to it (I have a lot I'm working on, too), I'll add a link to
/privatemsg/msgto/20or something instead of showing the user's email address. I'll post a patch as soon as I make one.Comment #3
pukku commentedHi! As I had basically just written the code needed for this for a different module, and I had an hour-long conference call to be on, I went ahead and applied the fix. Update to the new version (minutes.module,v 1.2.2.2) (there are no database schema changes, so you don't need to run update.php), and clear your cache (or some other way of updating the menu cache — submitting the modules page might work?). There will then be a settings page for the module, where you can set what the display should be. The default is "%name" — which just displays the user name. You can use "%uid" and "%mail", as well as "%fieldname" to pick up any field in the user's profile. So in your case, you might want to do this as '%name'.
A future modification may be to separate out the user name on the edit form from the user name when displayed, as I can see how you might want these to be different.
Comment #4
pukku commentedBah. That should read:
Comment #5
Christefano-oldaccount commentedThis is great, thanks!
I hope I'm not asking for too much all at once, but I'd love to see your new code in 1.2.2.3 combined with lenzjo's code at 70740.
If only all conference calls were only an hour...
Comment #6
pukku commentedHi! There are some issues with the code that lenzjo gave me — I'm about to post more info to that issue, so you can read about that there.
I will get around to it, but I'm not sure how / when.
Comment #7
pukku commented