[wp-trac] [WordPress Trac] #31168: Comments should be turned off on pages by default

WordPress Trac noreply at wordpress.org
Thu Jul 2 15:24:12 UTC 2015


#31168: Comments should be turned off on pages by default
-------------------------------------+---------------------------
 Reporter:  melchoyce                |       Owner:  valendesigns
     Type:  enhancement              |      Status:  reopened
 Priority:  normal                   |   Milestone:  4.3
Component:  Comments                 |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  needs-patch 2nd-opinion  |     Focuses:
-------------------------------------+---------------------------

Comment (by obenland):

 Replying to [comment:46 helen]:
 > I noticed that the hook ended up with a dynamic name:
 `get_{$post_type}_default_comment_status`. Dynamic hooks are neat, but not
 always easy to find, and there's usually a case to be made for the generic
 non-dynamic version. I think it'd be better to go with
 `get_default_comment_status` and pass the post type as context.

 I was actually thinking about that in my final review. I think we should
 definitely do that.

 > Also, do we need to do a general audit of `get_option(
 'default_comment_status' )` uses?
 Yes we do.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31168#comment:47>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list