Fixes dustin10#1228, metadata loading from config/vich_uploader direc…

…tory (dustin10#1230)

Fix dustin10#1228, metadata loading from config/vich_uploader directory

docs/mapping/xml.md

@@ -13,8 +13,19 @@ format and comes with the following syntax to declare your uploadable fields:
 </vich_uploader>
 ```
 
-To be automatically found, the mapping configuration MUST be in the `config/vich_uploader`
-directory of the bundle containing the entity you want to describe.
+To be automatically found, the mapping configuration MUST be in the `config/vich_uploader` directory
+of your symfony application, and the root namespace MUST be the de-facto standard `App` namespace.
+Third-party bundles providing configuration must follow the same rule in their own directory.
+
+Turning off auto-detection will disable this discovery method:
+
+```yaml
+# config/packages/vich_uploader.yaml or app/config/config.yml
+vich_uploader:
+    # ...
+    metadata:
+        auto_detection: false
+```
 
 If you need the mapping elsewhere, you need to add some configuration.
 In the following example, the configuration is placed in the `config/acme` directory:
@@ -24,7 +35,7 @@ In the following example, the configuration is placed in the `config/acme` direc
 vich_uploader:
     # ...
     metadata:
-        auto_detection: false
+        auto_detection: false # omit this line if the previously described auto-discovery is still needed
         directories:
             - {path: '%kernel.project_dir%/config/acme', namespace_prefix: 'Acme'}
 ```

docs/mapping/yaml.md

@@ -16,8 +16,19 @@ Acme\DemoBundle\Entity\Product:
         dimensions: imageDimensions
 ```
 
-To be automatically found, the mapping configuration MUST be in the `config/vich_uploader`
-directory of the bundle containing the entity you want to describe.
+To be automatically found, the mapping configuration MUST be in the `config/vich_uploader` directory 
+of your symfony application, and the root namespace MUST be the de-facto standard `App` namespace.
+Third-party bundles providing configuration must follow the same rule in their own directory.
+
+Turning off auto-detection will disable this discovery method:
+
+```yaml
+# config/packages/vich_uploader.yaml or app/config/config.yml
+vich_uploader:
+    # ...
+    metadata:
+        auto_detection: false
+```
 
 If you need the mapping elsewhere, you need to add some configuration.
 In the following example, the configuration is placed in the `config/acme` directory:
@@ -27,7 +38,7 @@ In the following example, the configuration is placed in the `config/acme` direc
 vich_uploader:
     # ...
     metadata:
-        auto_detection: false
+        auto_detection: false # omit this line if the previously described auto-discovery is still needed
         directories:
             - {path: '%kernel.project_dir%/config/acme', namespace_prefix: 'Acme'}
 ```

src/DependencyInjection/VichUploaderExtension.php

@@ -91,6 +91,14 @@ protected function registerMetadataDirectories(ContainerBuilder $container, arra
         // directories
         $directories = [];
         if ($config['metadata']['auto_detection']) {
+            $projectDir = $container->getParameter('kernel.project_dir');
+            if (\is_string($projectDir)) {
+                $appConfigDirectory = $projectDir.'/config/vich_uploader';
+                if (\is_dir($appConfigDirectory)) {
+                    $directories['App'] = $appConfigDirectory;
+                }
+            }
+
             foreach ($bundles as $class) {
                 $ref = new \ReflectionClass($class);
                 $directory = \dirname($ref->getFileName()).'/../config/vich_uploader';