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.
This is a sub-issue of #1938580: [META] Make active config save format match the default yml file (order and quotes). Check meta issue for Steps to reproduce and Coding standards
Files fixed already
filter.settings.yml
Files need to be fixed
filter.format.plain_text.yml
Remaining tasks
- Create patch. Contributor Task doc: http://drupal.org/node/1424598
Comment | File | Size | Author |
---|---|---|---|
#3 | 1942122-filter-config-format-3.patch | 449 bytes | sandipmkhairnar |
#2 | 2013-03-16_163624.png | 27.99 KB | vijaycs85 |
#2 | 2013-03-16_163608.png | 60.56 KB | vijaycs85 |
Comments
Comment #1
sandipmkhairnar CreditAttribution: sandipmkhairnar commentedI think filter.format.plain_text.yml file looks good. No need to change.
Comment #2
vijaycs85Guess user roles should be removed for this format as there is no option we can change it? Please refer screenshots:
Comment #3
sandipmkhairnar CreditAttribution: sandipmkhairnar commentedThanks @vijaycs85 for review.
Created patch as per comment.
Comment #4
sandipmkhairnar CreditAttribution: sandipmkhairnar commentedComment #5
vijaycs85#3: 1942122-filter-config-format-3.patch queued for re-testing.
Comment #6
SwapS CreditAttribution: SwapS commented#3: 1942122-filter-config-format-3.patch queued for re-testing.
Comment #7
tim.plunkettActually, those roles values are needed. We have a test for that, if you tried to remove the roles from filter.format.filter_test.yml, this would fail:
http://drupalcode.org/project/drupal.git/blob?f=core/modules/filter/lib/...
These files are being updated as a part of the much larger #1868772: Convert filters to plugins, I'm closing this as a duplicate.
Comment #7.0
tim.plunkettUpdated issue summary.