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 #3062757-22: Remove deprecated legacy include files from Drupal 9 (2) discovered that comment references to non-existing method
Proposed resolution
Clean-up comment
Remaining tasks
review/commit
User interface changes
no
API changes
no
Data model changes
no
Release notes snippet
no
Comment | File | Size | Author |
---|---|---|---|
#5 | 3092791-5.patch | 717 bytes | andypost |
#5 | interdiff.txt | 694 bytes | andypost |
Comments
Comment #2
andypostComment #3
andypostComment #4
BerdirThe method is called testHtmlEscapedText(), so it doesn't reference a removed method, just the wrong name..
Comment #5
andypost@Berdir thank you, missed the name, btw in 9.0 one of them should be removed as test marked as legacy
Anyway here's a proper patch
Comment #6
BerdirOk, then I'd say lets commit this to 8.8/8.9 and the other issue can remove it in 9.0.x, and also doesn't need to be blocked on this anymore?
Comment #7
alexpottCommitted and pushed d507e67789 to 8.9.x and d29f4f107a to 8.8.x. Thanks!
Backported to 8.8.x since this is a test-only documentation and not committed to 9.0.x because this test will be removed there.