Task #92524

Remove empty array from site configuration

Added by Georg Ringer 15 days ago. Updated 15 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
-
Start date:
2020-10-08
Due date:
% Done:

100%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Associated revisions

Revision 8b373735 (diff)
Added by Georg Ringer 15 days ago

[TASK] Remove empty array from site configuration before persisting

Having empty array like `routes: { }` make it hard to lint the site
configuration yaml files. To have a cleaner state, those empty arrays
are removed before they are persisted.

Resolves: #92524
Releases: master, 10.4
Change-Id: Id4f9cae7deb5903bf735cc1076731a5105b494ec
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/66091
Tested-by: TYPO3com <>
Tested-by: Benni Mack <>
Tested-by: Josef Glatz <>
Tested-by: Markus Klein <>
Tested-by: Oliver Bartsch <>
Tested-by: Georg Ringer <>
Reviewed-by: Benni Mack <>
Reviewed-by: Josef Glatz <>
Reviewed-by: Markus Klein <>
Reviewed-by: Oliver Bartsch <>
Reviewed-by: Georg Ringer <>

Revision be076b6a (diff)
Added by Georg Ringer 14 days ago

[TASK] Remove empty array from site configuration before persisting

Having empty array like `routes: { }` make it hard to lint the site
configuration yaml files. To have a cleaner state, those empty arrays
are removed before they are persisted.

Resolves: #92524
Releases: master, 10.4
Change-Id: Id4f9cae7deb5903bf735cc1076731a5105b494ec
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/66050
Tested-by: TYPO3com <>
Tested-by: Josef Glatz <>
Tested-by: Georg Ringer <>
Reviewed-by: Josef Glatz <>
Reviewed-by: Georg Ringer <>

History

#1 Updated by Georg Ringer 15 days ago

Empty lists make it hard for nice linting, therefore those should be removed. Examples are

routes: {  }

#2 Updated by Gerrit Code Review 15 days ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66091

#3 Updated by Gerrit Code Review 15 days ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66091

#4 Updated by Gerrit Code Review 15 days ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66091

#5 Updated by Gerrit Code Review 15 days ago

Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66050

#6 Updated by Georg Ringer 15 days ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF