Allow site builders to choose their own defaults.

#1 I think is a won't fix. It would effectively disable commenting until the author became aware of the feature.

#2 Is another won't fix. If you don't have approval permissions, the form isn't there. But if you do have approval permissions its essential to let the user disable e-mails getting sent to them. A site builder can turn this off with a theme function if they really want to do that.

#3 is a good point. I've opened #989962: Add option for approval notifications on login to deal with that specifically. Thanks for pointing it out!

#4 Not sure what you mean here. Can you explain further?

Ok I'm guessing you mean some kind of per content-type setting like content access module has?

See the image at: http://drupal.org/node/257453

Assuming that's the case, I'm going to add it to a very similar issue at #723080: Allow per content type admin my comments and close this one as a duplicate. Please re-open if you want to provide more input.

please see related post also at: http://drupal.org/node/1064964#comment-4208502

Okay i just found Untick "Skip page approvals" by default: http://drupal.org/node/583698 and your response:

#1 I think is a won't fix. It would effectively disable commenting until the author became aware of the feature.

am i misunderstanding this?

In my case, having this ticked disabled emails being delivered when comments were made which makes sense... however if a user doesn't know this is checked by default, they'll never know why they're not receiving emails notifications. When unticked, i got the email notification as expected. Either way, this checkbox had NO effect on allowing/disallowing commenting. Rather, it did have effect on whether or not email notifications were received, and as such, seems it should be DISABLED by default as people expect email notifications to be received with this module "out of the box" unless that's turned off by unchecking "Receive e-mail notifications"

...any thoughts...?

thanks!

For anyone else who's interested, i "hacked" this by changing line 243 of usercomment.module from

              '#default_value' => isset($edit['usercomment_approve_'. $node->type]) ? $edit['usercomment_approve_'. $node->type] : 1,

to

              '#default_value' => isset($edit['usercomment_approve_'. $node->type]) ? $edit['usercomment_approve_'. $node->type] : 0,

seems to have worked for now.

Instead of being mean and not letting you do this, I'm going to work on adding these options to the admin settings form. Stay tuned.

thanks!

Pushing to 2.x-dev. Patch as described here http://drupal.org/node/1064964#comment-4208814