Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The ROLE cache mode does use the concatenation of the all the roles a user has. In various cases however, all that is needed is a different cash hash for anonymous and authenticated users. Patch adds this as an option for the panels submodule.
Note : this superseeds #2220445: esi_panels_url does not pass the CACHE mode in url
Comment | File | Size | Author |
---|---|---|---|
#3 | esi-authenticated_cache_mode-2232217-3.patch | 4.01 KB | ron_s |
Comments
Comment #1
bellesmanieres CreditAttribution: bellesmanieres commentedComment #2
bellesmanieres CreditAttribution: bellesmanieres commentedForgot a needed bit !
Comment #3
ron_s CreditAttribution: ron_s commentedThere is a problem with the patch in #2.
In the
esi_panels__esi_pane_prepare
function, it says, "The cache mode is always the last argument, then the url." However, in theesi_panels_url
function, the code to add the cache mode is placed before the current page URL, not after it. The attached patch changes this order so it can work properly.Also there is a typo in the
ESI_PANELS_CACHE_AUTHENTICATED
label, and I don't understand the naming ofhook_boot
as "tmw_caching_boot
"? Have changed this to beesi_panels_boot
.Please review the attached patch, thanks.
Comment #4
dstuart CreditAttribution: dstuart at Axis12 Limited commentedComment #5
dstuart CreditAttribution: dstuart at Axis12 Limited commentedPushed to 7.3