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.
Spotted that the setDialogTitle in the OpenDialogCommand didn't work. It is never called which is why I think this hasn't been spotted. Not sure whether this is a bug or a task, but went for bug. Provided a test.
Beta phase evaluation
Issue category | Bug |
---|---|
Unfrozen changes | Unfrozen because it only changes code that doesn't currently work as it is supposed to. |
Comment | File | Size | Author |
---|---|---|---|
#22 | interdiff.txt | 2.67 KB | Sam152 |
#22 | 2121119-set-title-dialog-22.patch | 2.43 KB | Sam152 |
#22 | 2121119-set-title-dialog-22--test-only.patch | 1.93 KB | Sam152 |
#19 | 2121119-19-do-not-commit.patch | 1.77 KB | BR0kEN |
#18 | 2121119-18.patch | 2.27 KB | BR0kEN |
Comments
Comment #1
ACF CreditAttribution: ACF commentedAdding tag.
Comment #2
larowlanCan we get a test-only patch to verify it catches the issue?
Comment #3
ACF CreditAttribution: ACF commentedJust the test patch.
Comment #6
ACF CreditAttribution: ACF commentedOops mistake in not using the TestOpenDialogCommand.
Comment #7
ACF CreditAttribution: ACF commentedComment #9
ACF CreditAttribution: ACF commentedThis patch now has the same test, but with the fix in the code as well.
Comment #10
ACF CreditAttribution: ACF commentedComment #11
jhedstromLooks right, and the test passes. I've added the beta phase evaluation to the summary.
Comment #14
jhedstromTestOpenDialogCommand
was converted to mocks in #2250165: Replace fake mocks with actual OpenDialogCommand stubs in AjaxCommandsTest.Comment #18
BR0kENComment #19
BR0kENPatch without fixes to ensure test fails.
Comment #21
BR0kENComment #22
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedSaw the slack conversation about not mocking the unit being tested. Turns out if the content is a string, the renderer isn't invoked. Also added the assert to the previous test method.
Comment #25
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedComment #28
larowlanCommitted as 916e6ba and pushed to 8.5.x.
Cherry-picked as 8b0d7f8 and pushed to 8.4.x.