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.
Based on the work that we started in #2297915: Various fixes: PSR-4, correctly set expiration time, tests.
Comment | File | Size | Author |
---|---|---|---|
#1 | interdiff.txt | 2.97 KB | slashrsm |
#1 | 2305649_1.patch | 4.12 KB | slashrsm |
Comments
Comment #1
slashrsm CreditAttribution: slashrsm commentedInterdiff is against #2297915-8: Various fixes: PSR-4, correctly set expiration time, tests. Patch includes two things: fix $expire argument handling in set to follow what PHPdoc says about CacheBackendInterface::set() and cache all items permanently and handle expiration in backend's logic.
Comment #2
damiankloip CreditAttribution: damiankloip commentedLet's get this in now, as it works, and simplifies things. Which is always good for now :) Get things working, then extend it further.
Me and catch are considering TTL support still, but we can add that back in easily. Also we may end up setting an expiration time for the memcache item. Both can be discussed and done separately though.
Thanks.