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

pukku’s picture

Assigned: Unassigned » pukku

Hi! 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.

Christefano-oldaccount’s picture

Thanks 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/20 or something instead of showing the user's email address. I'll post a patch as soon as I make one.

pukku’s picture

Status: Active » Fixed

Hi! 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.

pukku’s picture

Bah. That should read:

<a href="/privatemsg/msgto/%uid">%name</a>
Christefano-oldaccount’s picture

This 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...

pukku’s picture

Hi! 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.

pukku’s picture

Status: Fixed » Closed (fixed)