Follow-up to #2342045: Standard views base fields need to use same rendering as Field UI fields, for formatting, access checking, and translation consistency

The patch there introduced a YAML syntax error in the test_group_rows test view.

This is used by one test: core/modules/views_ui/src/Tests/FilterUITest.php

The test apparently/probably passes. But the API module is not able to parse the file (it must be using a somewhat stricter YAML parsing routing).

The only other files that the API module failed to parse were:
vendor/jcalderonzumba/gastonjs/mkdocs.yml
core/tests/Drupal/Tests/Core/Asset/library_test_files/invalid_file.libraries.yml

I guess we don't fix vendor and the other one presumably is meant to be invalid. ;)

Anyway, it's a one-line patch.

CommentFileSizeAuthor
fix-yaml.patch610 bytesjhodgdon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon created an issue. See original summary.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented

Is this the only one?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

Please read the issue summary. ;)

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Please read the issue summary. ;)

Maybe that late in the case :)

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Reviewed & tested by the community » Fixed

Since this only changes test code, this is indeed an rc eligible change, and clear straightforward fix. Committed and pushed to 8.0.x. Thanks!

  • xjm committed f3d6f3a on 8.0.x 
    Issue #2600686 by jhodgdon: Broken views test config YAML

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.