Unnecessary query in comment render

Thanks. Re-roll for 6.

This broke comment approval tests - if a node only has one comment, and that comment is unpublished, we no longer render it. We could probably find a way around this like a separate 'has comments' flag to the count one, but for now here's a straight revert. I miss the test bot :(

Thanks Dries.

This gets more interesting.

I fixed the first bug with a user_access('administer comments') check.

However, I then found that the comment 'approve' link bypasses comment_save (as does everything in http://api.drupal.org/api/function/comment_operations/6) - this means that node_comment_statistics never gets updated - so the tests still have one fail without that being fixed.

Also, when manually testing the comment approval, I noticed that the published/unpublished radios are mislabeled. Fun.

So, here's a new patch which both adds the proper checks to avoid the query, and fixes comment_approve() so that tests continue to pass, this bug was hidden due to the lack of the $node->comment_status check but is now covered by the comment approval test.

I've opened issues for the other two bugs at #352911: comment administration form mis-labeling and #352895: comment_operations bypasses comment_save.

comment_save() should also take a properly formatted comment object instead of a fapi array, but there must be an issue for that somewhere already.

Better code comment.

re-rolled due to a whitespace change. No changes, so leaving RTBC.