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:
image.settings.yml
image.style.large.yml
image.style.medium.yml
image.style.thumbnail.yml

User interface changes

NO

API changes

NO

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

CommentFileSizeAuthor
#1 yml-2105965-1.patch2.21 KBclaudiu.cristea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

claudiu.cristea’s picture

Status: Active » Needs review
FileSize
2.21 KB

Here's a patch.

claudiu.cristea’s picture

Assigned: Unassigned » claudiu.cristea

Assigning to keep track.

Status: Needs review » Needs work

The last submitted patch, yml-2105965-1.patch, failed testing.

claudiu.cristea’s picture

Weird. Drupal\update\Tests\UpdateContribTest is passing locally and I see no possible influence by this patch.

claudiu.cristea’s picture

Status: Needs work » Needs review
Issue tags: -Configuration system, -YAML

#1: yml-2105965-1.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Configuration system, +YAML

The last submitted patch, yml-2105965-1.patch, failed testing.

webchick’s picture

I believe this got fixed by #2106459: Core config has everything as string which (attempted to) fixed everything at once en masse.

claudiu.cristea’s picture

Status: Needs work » Closed (duplicate)
claudiu.cristea’s picture

Assigned: claudiu.cristea » Unassigned