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 testGetArgument() is run with the first data set multiple times.
Proposed resolution
Fix providerGetArgument().
Remaining tasks
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#1 | QueryParameterTest-2501635-1.patch | 1.94 KB | Lendude |
QueryParameterTest.patch | 1.4 KB | olli | |
Comments
Comment #1
LendudeNice catch. And it's not just that the tests are never run, the tests are bugged when you do run them.
But instead of re-using $single, wouldn't it be easier to just add it straight to $data? Less chance of old data being reused. Added a patch that goes that route.
Comment #2
dawehnerUps :)
RC eligible because there are just test changes.
Comment #3
alexpottCommitted ec438eb and pushed to 8.0.x. Thanks!
This looks much better.. nice catch.