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
The DB dump contains 'node' instead of '/node', so drupalGet('') on update path tests returns in a 404.
Proposed resolution
Fix it
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#2 | 2550615-2.patch | 145.58 KB | dawehner |
Comments
Comment #2
dawehnerComment #3
dawehner.
Comment #4
dawehnerThis is blocking another critical
Comment #5
catchComment #6
dawehnerNote: Sites at least could have used #2511962: Upgrade path for 2509300
Comment #7
alexpottCommitted 580f335 and pushed to 8.0.x. Thanks!
Comment #9
effulgentsia CreditAttribution: effulgentsia at Acquia commentedIs the db dump now based on a known commit hash? If so, what is it? If not, can we update to one (maybe beta-12?)?
Comment #10
catch@effulgentsia it's not as far as I know. The database was committed, and then there have been small updates like this since as with the blocked issue, sometimes tests for issues providing update paths have required data that would have been upgraded for ones that didn't (because the dump pre-dates requiring hook_update_N()).
A completely fresh dump with beta 12 sounds good - although we'd want to explicitly check that any committed/pending updates in patches still run and are tested, otherwise might as well remove both them and the test coverage.
Comment #11
jibranThis broke #2464427-152: Replace CacheablePluginInterface with CacheableDependencyInterface :(