function cache_set_expire() {
    if ($lifespan = $this->options[$type . '_lifespan']) {
      return time() + $lifespan;
    }
    else {
      return CACHE_PERMANENT;
    }
  }

The function expects a $type variable but it's missing here.

CommentFileSizeAuthor
#1 964266-cache_set_expire.patch456 bytesdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
FileSize
456 bytes

Here is a patch

dawehner’s picture

pmorel’s picture

Status: Needs review » Reviewed & tested by the community

I was going to post exactly the same patch but did remember to search the issue queue before submitting the same issue. The patch is working for me and does make sense

merlinofchaos’s picture

Status: Reviewed & tested by the community » Fixed

Committed to all branches.

Status: Fixed » Closed (fixed)

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