Opened 9 years ago
Last modified 9 years ago
#3117 new enhancement
Make a more efficient update walker
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | 2.7 | Priority: | high |
| Severity: | normal | Version: | 2.1 |
| Component: | General - Performance | Keywords: | needs-patch |
| Cc: | contato@… |
Description
Over the years, the new/update forum/topic/reply code has accidentally become a mish-mash of approaches to address specific needs at the time. This has resulted in multiple approaches to walking & traversing the post_parent hierarchy, making it not a very straightforward process to understand or hook into.
A few things to consider:
- Our
handlerfunctions should be the only places where user input is checked - Our
walkerfunction should be the only places where that branch of the tree is updated - Our
extrasandupdatefunctions should be helpers, but not invoke a walk of the tree or handle user input - Some functions require a recalculation of the tree, like spamming & unspamming a reply. In these cases, the walkers should know how to query for and update the relevant meta-data
- Caching should be scrutinized deeply to make sure we aren't triggering multiple recalculations per iteration of the walker
Note: See
TracTickets for help on using
tickets.