Opened 11 years ago
Closed 10 years ago
#26732 closed defect (bug) (fixed)
Twenty Fourteen "featured" tag not hidden on frontend as expected
Reported by: | Chouby | Owned by: | lancewillett |
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Bundled Theme | Keywords: | has-patch commit fixed-major |
Focuses: | Cc: |
Description
To reproduce, just activate Twenty Fourteen on a fresh install (optionally visit the customizer to check that the featured tag is "featured" and is not displayed on frontend as default. You are happy with that so do not make any changes).
Create a new post with tag "featured". The "featured" tag appears in the list of tags as well as in the tag cloud.
Attachments (3)
Change History (18)
#2
@
11 years ago
- Keywords has-patch added
I can confirm that the "featured" tag is not hidden unless you change the option to a different tag, then back to "featured".
#3
follow-up:
↓ 4
@
11 years ago
- Keywords has-patch removed
26732.patch isn't good as it fires a fatal error
Call to undefined function wp_create_tag() in wp-content/themes/twentyfourteen/inc/featured-content.php on line 517
if the featured tag does not exist
#4
in reply to:
↑ 3
@
11 years ago
- Keywords has-patch added
26732.patch isn't good as it fires a fatal error
Should be fixed by 26732.2.patch
NB: I also noticed that it is impossible to delete the "featured" tag (as it is immediately re-created by the function Featured_Content::validate_settings(). Don't know if it is a wanted feature or a bug (as might suggest the comment for Featured_Content::delete_post_tag() :
Reset tag option when the saved tag is delete
#8
@
11 years ago
Patch adds a check for the tag name, in case we're using default values and the default id of 0 does not match the term id of the featured
tag.
#9
@
11 years ago
- Owner set to lancewillett
- Resolution set to fixed
- Status changed from new to closed
In 27118:
#10
@
11 years ago
- Keywords fixed-major added
- Resolution fixed deleted
- Status changed from closed to reopened
#12
@
11 years ago
Testing note, for Obenland and Jetpack plugin team -- with Jetpack enabled the "Hide tag from tag clouds and post meta" doesn't seem to be working correctly. Works fine with trunk and Twenty Fourteen without Jetpack active.
#14
in reply to:
↑ 11
@
11 years ago
Replying to lancewillett:
This isn't major. :)
We use "fixed-major" keyword for tickets fixed in the major release and reopened for minor release consideration :) http://make.wordpress.org/core/2011/04/06/the-keywords-fixed-major-and-fixed/
The patch just proposes to fill featured content settings at theme activation to deal with the case when the featured tag is existing before theme activation.