Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#12771 closed defect (bug) (fixed)

Comments do not work after commit 9175

Reported by: pgquiles Owned by: iandstewart
Priority: normal Milestone: WordPress.org
Component: Themes Version:
Severity: major Keywords:
Cc: Focuses:

Description

Commit 9175 implemented threaded comments and moved this line:

<input type="hidden" name="comment_post_ID" value="<?php echo $id; ?>" />

to a new generic function "comment_id_fields()" as per issue 7635:

http://core.trac.wordpress.org/changeset/9175

http://core.trac.wordpress.org/ticket/7635

Problem is it broke the default theme for me. I have been suffering this issue for months now but I never found the fix until yesterday:

http://wordpress.org/support/topic/38683
(see comment by westi)

I reversed 9175 locally and now the default theme works again in Wordpress 2.9.7 (and I guess it would fix the issue in many older versions too)

I don't know whether this issue needs fixing, or what kind of fix, now that classic and default themes have been removed, thought:

http://core.trac.wordpress.org/changeset/13879

Change History (7)

#1 @scribu
16 years ago

  • Milestone Unassigned
  • Resolutioninvalid
  • Status newclosed

This ticket missed the train.

I guess you should take it up with whoever maintains the default theme now.

#2 @nacin
16 years ago

  • Component CommentsWordPress.org
  • MilestoneWordPress.org site
  • Resolution invalid
  • Status closedreopened

#3 @nacin
16 years ago

  • Owner set to iandstewart
  • Status reopenedassigned

#4 @nacin
16 years ago

  • Component WordPress.orgThemes

#5 @iandstewart
16 years ago

@pgquiles This is working for me in 3.0 RC3. Is your comment-template.php up to date?

#6 @mrmist
16 years ago

Is this still an issue?

#7 @johnbillion
16 years ago

  • Resolutionfixed
  • Status assignedclosed

Closing as per iandstewart's comment. Re-open if it's still an issue.

Note: See TracTickets for help on using tickets.