This is a sub-issue of #1938580: [META] Make active config save format match the default yml file (order and quotes).

Files need to be fixed
menu.menu.tools.yml
menu.menu.main.yml
menu.menu.admin.yml
menu.menu.account.yml

Files fixed already
menu.setttings.yml

CommentFileSizeAuthor
#5 1942152-menu-config-fix-5.patch2.12 KBrajneeshb
#1 1942152-menu-config-fix-1.patch1.71 KBvijaycs85

Comments

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Active » Needs review
StatusFileSize
new 1942152-menu-config-fix-1.patch1.71 KB

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

The last submitted patch, 1942152-menu-config-fix-1.patch, failed testing.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Needs work » Needs review
Issue tags: +Configuration system, +Config novice

#1: 1942152-menu-config-fix-1.patch queued for re-testing.

sandipmkhairnar’s picture

sandipmkhairnar commented

It looks good for me.

rajneeshb’s picture

rajneeshb commented
StatusFileSize
new 1942152-menu-config-fix-5.patch2.12 KB

I have checked 1942152-menu-config-fix-1.patch and found that single word also within single quote. I think no need to wrap single word in quotes. So I have recreated patch.

Status: Needs review » Needs work

The last submitted patch, 1942152-menu-config-fix-5.patch, failed testing.

yesct’s picture

yesct commented

@rajneeshb an interdiff is awesome when adding a new patch to an issue:
For instructions on creating an interdiff, see http://drupal.org/node/1488712 Or, http://xjm.drupalgardens.com/blog/interdiffs-how-make-them-and-why-they-...

yesct’s picture

yesct commented

failure was:
The test did not complete due to a fatal error. Completion check MinimalTest.php 30 Drupal\minimal\Tests\MinimalTest->testMinimal()

Retesting.

yesct’s picture

yesct commented
Status: Needs work » Needs review

#5: 1942152-menu-config-fix-5.patch queued for re-testing.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Needs review » Reviewed & tested by the community

This is good to go, if test comes back green.

vijaycs85’s picture

vijaycs85
Location London, UK
commented

#5: 1942152-menu-config-fix-5.patch queued for re-testing.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 130a554 and pushed to 8.x. Thanks!

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