The database schema description for the expire column is wrong with regards to permanent cache entries, the constant is "-1" not "0".

Let's use the actual Cache::PERMANENT constant in the description.

CommentFileSizeAuthor
#1 2451655.patch701 bytesklausi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi commented
Status: Active » Needs review
FileSize
2451655.patch701 bytes

klausi opened a new pull request for this issue.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Seems fine!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Issue tags: +Quickfix
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented
Status: Reviewed & tested by the community » Fixed

Nice catch.

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 497d333 on 8.0.x 
    Issue #2451655 by klausi: Cache database schema description for expire...

Status: Fixed » Closed (fixed)

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