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.
Updated: Comment 0
Problem/Motivation
As realized during working on #2053519: Broken PHPUnit test with notices enabled for AJAX fails on View with "AJAX: Yes" it came clear that there needs to be a method on the AjaxResponse object to retrieve all commands,
so they can be changed before released into the wild.
Proposed resolution
Add an getCommands method which returns a list of commands by reference.
Remaining tasks
User interface changes
API changes
Related Issues
Comment | File | Size | Author |
---|---|---|---|
#10 | vdc-2071145.patch | 4.03 KB | dawehner |
#1 | ajax-2071145-1.patch | 4.04 KB | dawehner |
Comments
Comment #1
dawehnerHere it is together with a test.
Comment #2
Crell CreditAttribution: Crell commentedThere is no API change here, only an API addition.
Comment #3
dawehnerDoes that mean we can get this in?
Comment #4
Crell CreditAttribution: Crell commented1: ajax-2071145-1.patch queued for re-testing.
Comment #5
dawehnerLet's use the regression tool for this issue.
Comment #6
Crell CreditAttribution: Crell commentedJust wanted to see if the bot was still OK with it.
Comment #7
dawehnerJust wanted to justify to move it to major.
Comment #8
catch1: ajax-2071145-1.patch queued for re-testing.
Comment #10
dawehnerJust a rerole.
Comment #12
dawehner10: vdc-2071145.patch queued for re-testing.
Comment #13
tim.plunkettRTBC if green.
Comment #14
catchCommitted/pushed to 8.x, thanks!