[wp-trac] [WordPress Trac] #39038: Add support for non-split shared terms to singular term capabilities
WordPress Trac
noreply at wordpress.org
Thu Dec 8 16:32:08 UTC 2016
#39038: Add support for non-split shared terms to singular term capabilities
--------------------------+--------------------
Reporter: dd32 | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.7.1
Component: Taxonomy | Version:
Severity: normal | Resolution:
Keywords: needs-patch | Focuses:
--------------------------+--------------------
Changes (by boonebgorges):
* keywords: => needs-patch
Comment:
I'm with @dd32 that we are probably far enough along that we can assume,
throughout most of the taxonomy API, that terms have been split. Auth
checks are a case where we should be extra careful, but we're erring on
the side of caution here (shared terms will get 'do_not_allow').
Something like [attachment:35614.diff] seems OK, but it could result in
weird behavior due to latency or race conditions when the same cap-check
happens more than once on a page: the first time it could fail, triggering
a split, while on a subsequent call it might succeed. This could have
consequences that are difficult to predict, though it'd only be a one-time
occurrence for a given term, so maybe it's an OK consequence.
(If we want to go this direction, the patch will need to be written in a
way that fetches all terms sharing the term_id, so that we have a `tt_id`
to pass to `_split_shared_term()`.)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/39038#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list