Changes between Version 1 and Version 2 of Ticket #5733, comment 11
- Timestamp:
- 10/11/2014 07:05:24 PM (12 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #5733, comment 11
v1 v2 1 1 > But most of our caching doesn't actually use the global group - it uses object-type-specific cache groups (like 'bp_activity'). Creating a wrapper in one place suggests creating a wrapper everywhere, and I don't want to go wrapper-crazy without good argument :) 2 2 3 For sure, but we can default to one global group, which should handle the majority of spots.3 For sure, but we can default to one global group, which should handle the ~~majority of spots~~. 4 4 5 For those that need new groups, we add it in the additional parameter - eg. `bp_cache_set( 'bp_messages_unread_count', $count, 'bp_counts' );` 5 ~~For those that need new groups, we add it in the additional parameter - eg. `bp_cache_set( 'bp_messages_unread_count', $count, 'bp_counts' );`~~ 6 7 You're right about the majority of how cache calls being object-specific cache groups. Bah! Forget I said anything! 6 8 7 9 > Creating a wrapper in one place suggests creating a wrapper everywhere, and I don't want to go wrapper-crazy without good argument :)
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)