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.
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:
aggregator.settings.yml
views.view.aggregator_rss_feed.yml
views.view.test_aggregator_items.yml
User interface changes
NO
API changes
NO
Related Issues
Parent: #1653026: [META] Use properly typed values in module configuration
Comment | File | Size | Author |
---|---|---|---|
#7 | 2105913-aggregator-config-cleanup-7.patch | 5.3 KB | vijaycs85 |
#4 | 2105913-aggregator-yml-type-casting-4.patch | 1.99 KB | krishnan.n |
#3 | 2105913-aggregator-yml-type-casting-3.patch | 11.44 KB | vijaycs85 |
Comments
Comment #1
vijaycs85Comment #2
vijaycs85Comment #3
vijaycs85Initial patch...
Comment #4
krishnan.n CreditAttribution: krishnan.n commentedPl see script here : 21059
Comment #5
twistor CreditAttribution: twistor commented@krishnan.n, your script isn't working quite right, since keys like items_per_page should become integers, not booleans.
Comment #6
mtift#5 means that this needs work.
Rather than running scripts we should be saving the view in the ui and copying the saved .yml back to the aggregator's config directory.
Comment #7
vijaycs85Re-rolling...
Comment #8
vijaycs85The patch on this issue has been updated as part of #2167623: Add test for all default configuration to ensure schema exists and is correct. As this issue doesn't have any test to confirm/validate the schema, making this change and closing this issue as duplicate of #2167623: Add test for all default configuration to ensure schema exists and is correct. The contributors of this issue (in commit message) is copied to #2167623: Add test for all default configuration to ensure schema exists and is correct.