Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
In update_calculate_project_update_status()
in update.compare.inc
An intended call to UpdateManagerInterface::NOT_SECURE
is misspelled as UpdateManagerInterfac::NOT_SECURE
if (isset($available['project_status'])) {
switch ($available['project_status']) {
case 'insecure':
$project_data['status'] = UpdateManagerInterfac::NOT_SECURE;
if (empty($project_data['extra'])) {
$project_data['extra'] = [];
}
This change happened less than a week ago in #3015810: Properly deprecate UPDATE_* constants.
Proposed resolution
Fix the typo
Open a followup to expand test coverage so this could be caught more easily: #3100448: Expand test coverage for update_calculate_project_update_status()
Remaining tasks
.
User interface changes
.
API changes
.
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#2 | 3100444-2.patch | 670 bytes | bnjmnm |
Comments
Comment #2
bnjmnmComment #3
bnjmnmComment #4
Wim Leers🚢
Comment #5
longwaveComment #6
Wim LeersOops, yes, I meant to mark this RTBC 😅
Comment #7
Gábor HojtsyComment #10
Gábor HojtsyWhoops, thanks. I agree there should be tests so we don't just accidentally bump into this. #3100448: Expand test coverage for update_calculate_project_update_status()
Comment #11
Wim Leers