This is a simple patch that adds a column for the email address of an anonymous poster (See: http://drupal.org/node/view/7444) to the comment_admin_overview.

Future Enhancements:
-Make the email address a mailto: link (or integrate with email form)
-Show email address of registered users

CommentFileSizeAuthor
#4 comment_admin_edit.patch1.55 KBdavidnunez
#2 comment_admin.patch2.93 KBAnonymous (not verified)
comment_admin_overview.patch2.53 KBdavidnunez

Comments

dries’s picture

The comment overivew page is already too wide on my screen -- adding another column would make it even worse. Also, your patch shows the e-mail address of anonymous users only. It is confusing IMO. Thoughts?

(We should also look into extending the comment edit form so we can edit one's name, e-mail address and homepage.)

Anonymous’s picture

StatusFileSize
new2.93 KB

Here is another stab at this patch (apply to unmodified cvs tree).

It does two things:

1. It adds a "Mail" link underneath author names in comment_admin_overview table. It will display an active link for both registered and unregistered users if they left an email. Otherwise, it says "No Mail"
2. It adds a "Mail" field to the comment_admin_edit form that you access by clicking "Edit this Comment"

It appears that the ability to edit email and homepages of anonymous comments should be trivial to add. However, care must be taken so that registered users' email and url can't be modified through this form... they should be handled off of the user admin pages, as usual.

davidnunez’s picture

Ugh. Sorry. The last patch, http://drupal.org/node/view/8174#comment-8360, was mine... forgot to login before posting.

davidnunez’s picture

StatusFileSize
new1.55 KB

Here's a much better idea.

This patch to a clean cvs does nothing to the comment_admin_overview table.

Instead, if the user posting a comment is unregistered:

The comment_admin_edit form has fields for url and homepage.

Registered users' comments do not have these fields available.

Simple, uncluttered, and hopefully not too confusing.

dries’s picture

This patch does no longer apply. Marking it 'active'.

davidnunez’s picture

This is no longer necessary.