+++ b/core/includes/config.inc @@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf + * for all objects of this type. If it doesn't contain it, we load first the generic metadata Minor: We shouldn't abbreviate here, i.e. "doesn't" -> "does not" +++ b/core/includes/config.inc @@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf + * 1. Load metadata for image.style.* + * 2. Merge metadata for image.style.large I think both of these should be "from" not "for". In general, I think it would be clearer to put the listing in words, rather than using this actual list. That's definitely debatable, though. +++ b/core/includes/config.inc @@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf +function config_meta_data($name, $key = '') { This is missing some @return documentation. +++ b/core/includes/config.inc @@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf + else { + return $meta[$name]; + } Might as well drop the whole else, since the if already returns. +++ b/core/includes/config.inc @@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf +function config_meta_build($name, $data) { This is missing docs on @params and @return. +++ b/core/includes/config.inc @@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf +function _config_meta_element($meta, $data, $base, $index, $group = '') { Is missing @return documentation. +++ b/core/modules/system/meta/system.site.yml @@ -0,0 +1,18 @@ \ No newline at end of file So let's add one :-)