#35610 closed enhancement (fixed)
clean_comment_cache needs a hook
| Reported by: | spacedmonkey | Owned by: | ericlewis |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.5 |
| Component: | Cache API | Version: | 2.3 |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
Unlike other cache clear functions, clean_comment_cache doesn't have a hook.
Attachments (3)
Change History (11)
#1
follow-up:
↓ 5
@
11 years ago
- Keywords has-patch added
- Milestone Awaiting Review → Future Release
- Version → 2.3
#2
@
11 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
@
11 years ago
- Owner set to
- Status new → 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Makes sense to me.
Do we want unit tests for this?