Administrators are not aware of additional bypass permissions

- I think I like this patch. It is very helpful but it sorta puts a lot of emphasis on by-passing though. I'd recommend that we at least remove the by-pass information from the context sensitive help (i.e. "... unless they are able to ...").

- The list of permissions is not complete, is it? It does not mention Mollom's global by-pass Mollom permission?

Subscribing.

The implicit bypassing when having the "Edit any [nodetype] content" permission has bitten me.

+++ mollom.admin.inc	29 Oct 2010 02:52:48 -0000
@@ -211,6 +213,32 @@ function mollom_admin_configure_form(&$f
+        '#title' => t('Skip posts of users having the permissions'),

I'm not 100% on this description. Technically, having one of those permissions is enough. Second, we're not skipping the posts, we're skipping Mollom. To be correct we'd have to write:

"Skip Mollom for users that have any of the following permissions."

That is a long title. I think we could create a short title like "By-pass Mollom permissions" -- I'd think it is self-explanatory.

Sorry, I can't read all the comments, so I'll just give you a heuristic review of the last screenshot...

1. Protection mode is clear. I get it.
2. When a post is blocked I'm not so clear on.
A. Is this telling me "when protection mode works, do one of these things"? If so, that could be more clear.
B. Of the choices, discard means what? Is discard delete?
3. Text fields to analyze. I get this, but it feels like its out of order. Seems like step 2. That is...
A. How do you want to protect, what do you want to protect, what do you want to do when protection works.
4. Bypass access permissions. I have no idea what this is doing. It doesn't seem actionable. I think it's informational and relating to item 1. That is.. "How do you want to protect? Regardless of what you choose, these protection rules are in place. I also think it could be more clear as...

Protection is not enabled for these permissions (link permissions to Permissions):
• Bypass Mollom Protection
• Administer Nodes
• Edit any story content.

Committed to DRUPAL-6--1. Great job, sun. Let's sync up D7.