Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
This is a child task for #422380: Convert all core module to use new static caching API focusing on the aggregator module.
Comment | File | Size | Author |
---|---|---|---|
#3 | jamesan_480102-3.patch | 1.11 KB | JamesAn |
#2 | jamesan_480102-2.patch | 1.11 KB | JamesAn |
#1 | jamesan_480102-2.patch | 1.15 KB | JamesAn |
Comments
Comment #1
JamesAn CreditAttribution: JamesAn commentedThe aggregator module has two static vars, neither of which are ever reset. This patch inserts inline comments explicitly stating that these static vars don't use drupal_static() as is done elsewhere.
Comment #2
JamesAn CreditAttribution: JamesAn commentedAs mentioned in #480122-2: Update blogapi module to use drupal_static(), most static vars should be converted over to use the drupal_static() method.
Comment #3
JamesAn CreditAttribution: JamesAn commentedOops. Forgot the ampersand in front of drupal_static().
Comment #4
catchComment #5
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD.