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.
Problem/Motivation
Over the past few months, the SmartCache issue/patch has collected a bunch of smaller fixes that don't really make sense to commit separately (usually because they only cause test failures with SmartCache applied, or because of small bugs that are only made apparent by SmartCache). But, now that that issue was raised to critical and is approaching RTBC, it was proposed by @effulgentsia that it'd be better to still commit them separately, in this issue.
Proposed resolution
Fix those small bugs, and have test coverage where it makes sense.
Remaining tasks
None.
User interface changes
None.
API changes
None.
Data model changes
None.
Beta phase evaluation
Issue category | Bug because lots of small bugs that are blocking SmartCache. |
---|---|
Issue priority | Major because blocking a critical. |
Disruption | Zero disruption. |
Comment | File | Size | Author |
---|---|---|---|
#2 | 2562757-2.patch | 10.81 KB | Wim Leers |
Comments
Comment #2
Wim LeersComment #3
Fabianx CreditAttribution: Fabianx for Acquia commentedRTBC, that all makes a lot of sense!
Comment #4
effulgentsia CreditAttribution: effulgentsia at Acquia commentedNice! Promoting to Critical since it's a hard blocker for a critical.
Comment #5
dawehner+1
Comment #8
effulgentsia CreditAttribution: effulgentsia at Acquia commentedPushed to 8.0.x. Thanks!