Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#16879 closed defect (bug) (duplicate)

bug in function wp_count_comments in wp-includes/comment.php line 900

Reported by: tfnab's profile tfnab Owned by:
Milestone: Priority: normal
Severity: trivial Version: 3.1
Component: Comments Keywords: has-patch close
Focuses: Cc:

Description

This seems to have been around for a while and keeps being ignored, or rather pushed back, by Mark Jaquith, with phony arguments. Please re-consider just patching this!!! It's really trivial.

See tickets #16446 and #15215, both on WP3.0, for one possible solution.

I'd actually like to propose another approach: the $known_types array is not needed, because instead of inarray($x,array_keys($y)) one can use isset($y[$x]) -- see my patch.

Mark: you're basically saying, that there are other things that should keep us from using a custom comment status in a plugin. Well, if we can live with it, why do you still practically force us to edit core and make keeping our blogs up-to-date with the latest WordPress updates such a big pain?

Attachments (1)

3.1-wp_count_comments.patch (772 bytes) - added by tfnab 14 years ago.

Download all attachments as: .zip

Change History (3)

#1 @solarissmoke
14 years ago

  • Keywords close added

#15215 has been tagged for 3.2-early, which means that it will hopefully get some attention soon. I don't see why opening a duplicate ticket was necessary?

#2 @nacin
14 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.