Closed (fixed)
Project:
Views navigation
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
5 Oct 2015 at 10:57 UTC
Updated:
3 Jan 2017 at 18:05 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
lunatic commentedForgot to add hook_flush_caches (our use case is never clear them)
Attaching updated patch
Comment #3
lunatic commentedWe have this on production for about a week and seems to work fine
Moving it to RTBC
Comment #4
heine commentedChanges compared to the earlier patch:
- hook_flush_caches moved to the module file, or it will never be invoked.
- hook_update_N cannot use current schema information, changed to hardcoded schema.
We need this because we use memcache and views_navigation cache entries are quickly evicted. Users then see a "Sorry, could not find the requested page." message. We also don't want these cache items in memcache.
Using a separate cachebin allows us to store the cache items in the database.
Comment #5
BarisW commentedWe could use the static VIEWS_NAVIGATION_CACHE_BIN here.
I'll fix this when committing the patch.
Comment #7
BarisW commentedPushed to DEV. Thanks both!