Follow-up from #1653026: [META] Use properly typed values in module configuration.

Problem/Motivation

All integers, Booleans, and even octal numbers in config object files are converted to strings.

Proposed resolution

#1653026: [META] Use properly typed values in module configuration has fixed core, so no need to convert all data types to string anymore.

Remaining tasks

Fix and issue patch for below config files:
views.view.test_entity_reference.yml

User interface changes

NO

API changes

NO

Parent: #1653026: [META] Use properly typed values in module configuration

CommentFileSizeAuthor
#4 2105947-yml-type-casting-4.patch1.89 KBjaperry
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amateescu’s picture

amateescu CreditAttribution: amateescu commented

entity_reference.routing.yml is *not* a config file...

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Title: Make sure all YML files in Entity Reference module has no type-casting to string. » Make sure all config yml files in Entity Reference module has no type-casting to string.

yeah, just removed and updated title more specific to this issue.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Assigned: Unassigned » japerry
japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Status: Active » Needs review
FileSize
2105947-yml-type-casting-4.patch1.89 KB

First shot at patch for the config. Basically is for a view on the test.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Issue summary: View changes

Updated issue summary.

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Looks good to me.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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