Move all configuration schema files into a schema subdirectory

Here is a patch only moving the files. The reading of the files is currently handled with regular config storage (and schema files are indeed copied to the active store, etc, which is not good/intended). So I think we'll need to revert back to the state from #1648930: Introduce configuration schema and use for translation where InstallStorage was used to look up from specific places from the file system. Someone more knowledgeable with the config system would be great to help here.

So far this will only demonstrate it fails if the schema files are moved out from where the system expects them :) So it will bounce itself to needs work.

Interdiff looks pretty good to me, taking over the needed pieces from #1648930: Introduce configuration schema and use for translation. Now the schema discovery need to use this storage and the moving of files would be accomplished. Making it read and merge all the files could be a bit harder, but we don't need to dynamically find keys in files or anything, just merge all files outright from the module AFAIS.

All the above fails are expected, since the files are moved, but discovery is not yet updated.

Nonono! It should not copy the schema files to active storage, no. That behaviour is a side effect of the current design that is in fact not desired. The schema should be kept with the module and it cannot / will not be changed. So instead of copying it over, schema discovery should change to not look at the active config store but look at the original place with the component.

@alexpott, @vijaycs85: thanks! Looking at the patch I only found one problem:

+++ b/core/lib/Drupal/Core/Config/TypedConfigManager.phpundefined
@@ -18,19 +18,19 @@ class TypedConfigManager extends TypedDataManager {
-   * @param Drupal\Core\Config\StorageInterface $storage
+   * @param \Drupal\Core\Config\StorageInterface $storage
    *   The storage controller object to use for reading schema data
    */
-  public function __construct($storage) {
-    $this->storage = $storage;
-    $this->discovery = new SchemaDiscovery($storage);
+  public function __construct(StorageInterface $configStorage, StorageInterface $schemaStorage) {

phpdoc not fully updated to current arguments.

Also, do we want to tackle reading more than 1 file from there in this issue or in another one? :)

Looks pretty good to me. (Will be committable once testbot comes back green).

Given other changes and introductions of schema, it would be great to prioritise this for commit (there are other RTBC issues for config schemas but those are easier to reroll, this one is harder). So moving back to RTBC and adding Avoid commit conflicts.

Added the array typehint to that hunk. No other items found that this would apply to. So no interdiff. Given such minimal change, this should still be RTBC.

Thanks! That should help us reroll the 30 or so patches for schemas in http://drupal.org/project/issues/search/drupal?issue_tags=Configuration%... :) Superb!

Added this to the changelog at http://drupal.org/node/1905120 and updating the docs at http://drupal.org/node/1905070 :)

http://drupal.org/node/1905070 (doc page) fully updated for new location: http://drupal.org/node/1905070/revisions/view/2571372/2573208