Add comments_open() conditional to Twenty Eleven's comment-reply.js call

[philiparthurmoore]

The current code used to include comment-reply.js in Twenty Eleven's header.php is as follows:

    if ( is_singular() && get_option( 'thread_comments' ) )
        wp_enqueue_script( 'comment-reply' );

If a singular page is being displayed and comments are closed, the script is still loaded. The code should be adjusted as follows:

    if ( is_singular() && get_option( 'thread_comments' ) && comments_open() )
        wp_enqueue_script( 'comment-reply' );

The expected output of this code is for comment-reply.js to only be loaded on singular pages that have commenting currently open.

[ocean90]

You could also reply to a ping. Maybe

&& ( comments_open() || pings_open() )

[philiparthurmoore]

Replying to ocean90:

You could also reply to a ping. Maybe

&& ( comments_open() || pings_open() )

Isn't comment-reply.js just used for the threaded comments feature? I don't think pings_open() is important in this conditional.

Edit: Never mind. I see what you're saying. Sorry about that!

[ocean90]

Just ignore my comment too. In Twenty Eleven a reply to a ping isn't supported.

[zeo]

See #12932 and read @nacin comment: ​http://core.trac.wordpress.org/ticket/12932#comment:10

[markjaquith]

Not worth a query, IMO. Agree with Nacin's comment that zeo linked.