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
- DrupalWebTestCase::assertFieldByName() produces confusing error text when the
$value
parameter has the default value ofNULL
, e.g.:
Found field with name logo_path and value "
Proposed resolution
- Display a different message when the value is not set.
Original report by sun
Extracted from #1376166: Custom logo and favicon functionality inanely tries to support absolute local file paths
Comment | File | Size | Author |
---|---|---|---|
#3 | 0001-Issue-1397890-by-sun-Fixed-DrupalWebTestCase-assertF.patch | 1.64 KB | sun |
drupal8.simpletest-assertfieldbyname.0.patch | 1.23 KB | sun | |
Comments
Comment #0.0
xjmUpdated issue summary.
Comment #1
xjmStraightforward fix (plus it makes the code more readable!)
Comment #1.0
xjmUpdated issue summary.
Comment #2
catchCommitted/pushed to 8.x. Thanks!
Comment #3
sunComment #3.0
sunUpdated issue summary.
Comment #4
xjmComment #5
webchickCommitted and pushed to 7.x. Thanks!
Comment #7
cweagansUpdating tags per http://drupal.org/node/1517250
Comment #7.0
cweagansUpdated issue summary.