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.
Updated: Comment #N
Problem/Motivation
We are still using cache() and cache_invalidate_tags() function all over the place. We need to convert all of these to \Drupal::cache() and Cache::invalidateTags(), respectively. We can then remove cache.inc in #2198339: Remove cache.inc
Proposed resolution
Convert all the things.
Remaining tasks
Passing patch
User interface changes
None
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#11 | interdiff-2198343-11.txt | 1.33 KB | damiankloip |
#11 | 2198343-11.patch | 57.03 KB | damiankloip |
Comments
Comment #1
damiankloip CreditAttribution: damiankloip commentedLet's see what happens.
Comment #3
damiankloip CreditAttribution: damiankloip commentedDidn't rebase local branch.
Comment #5
damiankloip CreditAttribution: damiankloip commentedOops, was going to use an array for the callable, then changed my mind, then didn't remove the array :)
Comment #6
longwaveRead through the patch and all looks good, also confirmed that there are no uses of cache() or cache_invalidate_tags() left after applying.
Comment #7
alexpott2198343-5.patch no longer applies.
Comment #8
longwaveRerolled.
Comment #9
damiankloip CreditAttribution: damiankloip commentedAnd back, just a reroll.
Comment #10
alexpott2198343-cache-8.patch no longer applies.
Comment #11
damiankloip CreditAttribution: damiankloip commentedReroll with these conflicts. Got to love patch based workflow.
Comment #12
damiankloip CreditAttribution: damiankloip commentedComment #13
longwaveComment #14
alexpottCommitted a1a5e15 and pushed to 8.x. Thanks!
Used the new format for deprecated messages.