#35610 closed enhancement (fixed)
clean_comment_cache needs a hook
Reported by: | spacedmonkey | Owned by: | ericlewis |
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | 2.3 |
Component: | Cache API | Keywords: | |
Focuses: | Cc: |
Description
Unlike other cache clear functions, clean_comment_cache doesn't have a hook.
Attachments (3)
Change History (11)
#1
follow-up:
↓ 5
@
9 years ago
- Keywords has-patch added
- Milestone changed from Awaiting Review to Future Release
- Version set to 2.3
#2
@
9 years ago
@spacedmonkey Looks like there is some copypasta in the hook docs
Fires once after each taxonomy's term cache has been cleaned.
#5
in reply to:
↑ 1
@
9 years ago
- Owner set to ericlewis
- Status changed from new to assigned
Replying to ericlewis:
Makes sense to me.
Do we want unit tests for this?
Don't need unit tests for an action in most cases.
35610.2.patch should have extraneous spaces in new @param
documentation removed, and I don't think we need a function-level @since
addition for this (the info is in the hook docblock).
Note: See
TracTickets for help on using
tickets.
Makes sense to me.
Do we want unit tests for this?