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 | 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?
#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?