The comment posts even if any of the mandatory fields do not meet their requirements or even if they are empty.

So my case is when a user tries to submit a comment and the mandatory fields (Name, Email, etc) are not valid or even empty the comment appears to post. It gives the "Your comment has been posted" message with the comment displaying underneath it. But since it did not pass the basic validation it does not actually get put in the system at all.

Maybe I have something configured wrong, but if this is the case then I cant use this module at all. Everything else appears to be working well though.

The site is on a zen sub theme using Drupal 7

Comments

acouch’s picture

acouch commented

I have been meaning to put some more work on this. Will take a look next time I sit down.

inolen’s picture

inolen commented

Could you test with the latest?

This should be fixed with http://drupal.org/node/1264916

gratefulsk’s picture

gratefulsk commented

The mandatory fields are now working as expected. The comment does not get posted and the module asks the user to supply valid information in those fields. Great job!

I tested it out further and everything is looking awesome. I even have the captcha module installed.

There was just one thing I found that was not working correctly. For all my site(s), the configuration is always set to allow anonymous users to post comments but requires administrator approval. My testing shows when an anonymous user attempts to post a comment it displays a success message (Your comment has been posted) and the comment is displayed in the list of comments. For the user they feel there comment was posted and leave the page, but this is misleading since the comment still acquires approval to be published.

acouch’s picture

acouch commented
Status: Active » Fixed

Thanks for reporting back!

I've created a new ticket for the authorization issue #1348402: Notification for Comments that still require admin approval

Generally it is easier on the devs to keep the issues as focused as possible.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.