This patch will allow the current URL with query parameters to be added to the end of the cid (in the panels cache action configuration).

This makes for example views pagers work without any hassle and extra work.

Sort of Cache Actions extension of Panels Hash Cache

CommentFileSizeAuthor
#3 cache_actions-respect-query-parameters-707484-1803128-7.41.patch1.23 KBrobinmofo
cache_actions_cached_url.patch1.17 KBtcmug

Comments

RunePhilosof’s picture

RunePhilosof commented
Issue summary: View changes
Related issues: +#2151445: Add more granularity settings

I have also added that part from panels_hash_cache in my patch (in 2151445).
Although, the code that I ripped from that panels_hash_cache looks different than yours.
I wonder which is best.

I can see why it would be good to avoid too long cid by hashing.
However, I am leaning towards doing it only when the length comes above 255.
Because meaningful cids can be helpful for debugging.

robinmofo’s picture

robinmofo commented

I need one of these patches.. this one is more appealing as it's much simpler.

Tested with a view pane that needed pagination and works as expected (get params are hashed to end of cid).

Thanks

robinmofo’s picture

robinmofo commented
StatusFileSize
new cache_actions-respect-query-parameters-707484-1803128-7.41.patch1.23 KB

Re-rolled patch so it doesn't throw a notice, twice.