Problem/Motivation

As @Berdir pointed:
In EuCookieComplianceConfigForm it does a database query against the cache table. Instead, we need to use the \Drupal::cache() api and probably use deleteAll() as delete by prefix is not possible.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 query_on_eucookieform-2978326-4.patch593 bytesjohnchque
#3 query_on_eucookieform-2978326-3.patch587 bytesjohnchque

Comments

yongt9412 created an issue. See original summary.

johnchque’s picture

johnchque
Primary language Spanish
Location Bolivia
commented
Assigned: Unassigned » johnchque

Working on this.

johnchque’s picture

johnchque
Primary language Spanish
Location Bolivia
commented
Status: Active » Needs review
StatusFileSize
new query_on_eucookieform-2978326-3.patch587 bytes
berdir’s picture

berdir
Primary language German
Location Switzerland
commented

The reason is that this table doesn't exist (and if it would exist it wouldn't work) when using redis/memcache or a similar alternative cache backend.

johnchque’s picture

johnchque
Primary language Spanish
Location Bolivia
commented
StatusFileSize
new query_on_eucookieform-2978326-4.patch593 bytes

Oops my bad, thanks @Berdir for the help. :)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Looks good.

idebr’s picture

idebr commented
Related issues: +#2974074: Error submitting configuration form when using Redis as cache backend
svenryen’s picture

svenryen commented
Assigned: johnchque » Unassigned
svenryen’s picture

svenryen commented

  • svenryen committed fab6246 on 8.x-1.x authored by yongt9412 
    Issue #2978326 by yongt9412: Query on EuCookieComplianceConfigForm to...
svenryen’s picture

svenryen commented
Status: Reviewed & tested by the community » Fixed

Fixed. Thanks for the patch and review!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.