Opened 9 years ago
Closed 9 years ago
#33938 closed defect (bug) (fixed)
Taxonomies with show_ui set to false should not be accessible via the admin area
Reported by: | johnbillion | Owned by: | johnbillion |
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | 2.3 |
Component: | Taxonomy | Keywords: | needs-patch |
Focuses: | administration | Cc: |
Description
Same as #33763 for taxonomies.
If a taxonomy is registered with show_ui
set to false, it's still possible to access the term editing screen for the taxonomy by manually navigating to the correct URL.
Change History (7)
#2
follow-up:
↓ 3
@
9 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
We're hiding UI to add our own custom metabox (combining multiple taxonomies) and admin menu. However, register_taxonomy()
doesn't offer a registration argument to disable core meta boxes. It would be nice to have the argument, so we don't have to deregister metaboxes.
#3
in reply to:
↑ 2
;
follow-up:
↓ 4
@
9 years ago
Replying to danielbachhuber:
We're hiding UI to add our own custom metabox (combining multiple taxonomies) and admin menu. However,
register_taxonomy()
doesn't offer a registration argument to disable core meta boxes. It would be nice to have the argument, so we don't have to deregister metaboxes.
register_taxonomy()
has a meta_box_cb
arg, which can be set to false
.
#4
in reply to:
↑ 3
@
9 years ago
- Resolution set to fixed
- Status changed from reopened to closed
Replying to helen:
Replying to danielbachhuber:
We're hiding UI to add our own custom metabox (combining multiple taxonomies) and admin menu. However,
register_taxonomy()
doesn't offer a registration argument to disable core meta boxes. It would be nice to have the argument, so we don't have to deregister metaboxes.
register_taxonomy()
has ameta_box_cb
arg, which can be set tofalse
.
Ah. I tried __return_false
, but ended up with an empty meta box. Explicitly setting to false
does the trick.
In 34359: