Tickets :: Comment on [#9219] display_tasklists is saved on every page access

* Your Email Address
* Spam protection	Enter the letters below: . .. ..__ . ..__ \| \|\|__\|\| \\|\/\|[__) \|__\|\| \|\|__/\| \|\| \
Comment	>> Seeing this all over the place: >> >> 2010-09-22T00:39:08-06:00 DEBUG: HORDE [nag] Horde_Prefs: Storing >> preference value (display_tasklists) [pid 30516 on line 257 of >> "/disk2/src/horde/framework/Prefs/lib/Horde/Prefs.php"] > > Move it further down into the if-clause where the saving is actually > triggered. :) > >> As someone who never uses nag, I have no idea why this needs/should >> be updated every page load. Even if the value isn't changing, we >> shouldn't be wasting time recreating at all. At a minimum, this >> really needs to be cached (see, e.g., caching IMP's folder tree using >> session caching as a fallback if no Cache system is available). > > It's almost a no-op, so I wouldn't really care about that. And you > probably see this because of the Copy/Move to Tasklist feature.
Attachment
Watch this ticket