Comments filtered for XHTML validity multiple times

[markjaquith]

#2714 introduced comment_text() XHTML validity forcing.

Now, if the balanceTags option is on, WP is filtering comments on their way into the database as well as on the way out. pre_comment_content and comment_save_pre both run the comment content through balanceTags().

I think that comments, as third-party data whose XHTML invalidity can ruin your blog or your WP admin, should always be filtered for XHTML validity.

The best solution IMO would be to have comments always XHTML filtered on the way into the database (that is, run force_balance_tags() on pre_comment_content and comment_save_pre and remove the call to force_balance_tags() on comment_text() )

That's how content is filtered, and that'd be the least processor intensive.

[markjaquith]

Change of heart. Other filters could be messing around, and we want to make absolutely sure comments have tags balanced.