Opened 9 years ago
Closed 9 years ago
#33875 closed defect (bug) (fixed)
Comment caching could be improved
Reported by: | wonderboymusic | Owned by: | |
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Comments | Keywords: | has-patch |
Focuses: | Cc: |
Description
There are some caches that are never updated properly, etc.
Attachments (1)
Change History (8)
#5
in reply to:
↑ 1
@
9 years ago
Replying to wonderboymusic:
In 34131:
I thought that these counts were non-persistent and that was why we didn't both invalidating the cache because it only lasted for the current page load.
#6
@
9 years ago
The List Table for comments is one huge race condition and passes the total back and forth via AJAX without ever checking that it is correct. Even if it's not persistent, it can be invalidated on destructive actions. The use-case here is the person looking at 20 pending comments and approving them one-by-one, or sending a list of comments from spam to trash one-at-a-time
Note: See
TracTickets for help on using
tickets.
In 34131: