Opened 2 years ago
Last modified 6 weeks ago
#15919 accepted defect (bug)
wp_count_terms() hide_empty not working
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 3.6 |
| Component: | Taxonomy | Version: | |
| Severity: | major | Keywords: | has-patch |
| Cc: |
Description
For tax category, wp_count_terms( 'category', array( 'hide_empty' => true ) ) returns all category and doesn't hide category with post count 0. Instead need to use wp_count_terms( 'category', array( 'hide_empty' => true, 'hierarchical' => false ) ).
Since hide_empty=false is the default args, wp_count_terms() should set hierarchical=false by default too.
Attachments (4)
Change History (12)
comment:1
markjaquith — 2 years ago
- Milestone changed from Awaiting Review to 3.1
- Owner set to markjaquith
- Status changed from new to accepted
comment:4
markjaquith — 2 years ago
- Keywords 3.2-early added
- Milestone changed from 3.1 to Future Release
- Priority changed from normal to high
- Severity changed from normal to major
This is not a regression from 3.0, so I'm going to do it 3.2-early.
comment:5
SergeyBiryukov — 18 months ago
Closed #19340 as a duplicate.
wonderboymusic — 4 months ago
comment:6
wonderboymusic — 4 months ago
- Milestone changed from Future Release to 3.6
Refreshed against trunk just so the line numbers match - this is from the class of 3.2-early
comment:8
SergeyBiryukov — 6 weeks ago
Generally, a refresh is only needed when the patch no longer applies cleanly.
Line numbers may be off, but as long as the content is the same, the patch should still apply cleanly. 15919.2.diff is still fine.
Also, when copying code directly from Trac, make sure to remove trailing whitespace.
Bug reproduced. Patch fixes it. In it goes.