[MediaBundle] Add web_root configuration to support both web and publ…

…ic as the root dir for uploads (#2058)

src/Kunstmaan/MediaBundle/DependencyInjection/Configuration.php

@@ -2,6 +2,7 @@
 
 namespace Kunstmaan\MediaBundle\DependencyInjection;
 
+use Kunstmaan\MediaBundle\Utils\SymfonyVersion;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
 
@@ -38,6 +39,10 @@ public function getConfigTreeBuilder()
                     ->defaultValue(array('php', 'htaccess'))
                     ->prototype('scalar')->end()
                 ->end()
+                ->scalarNode('web_root')
+                    ->defaultValue(SymfonyVersion::getRootWebPath())
+                    ->cannotBeEmpty()
+                ->end()
             ->end();
 
         return $treeBuilder;

src/Kunstmaan/MediaBundle/DependencyInjection/KunstmaanMediaExtension.php

@@ -40,6 +40,7 @@ public function load(array $configs, ContainerBuilder $container)
         $container->setParameter('kunstmaan_media.remote_video', $config['remote_video']);
         $container->setParameter('kunstmaan_media.enable_pdf_preview', $config['enable_pdf_preview']);
         $container->setParameter('kunstmaan_media.blacklisted_extensions', $config['blacklisted_extensions']);
+        $container->setParameter('kunstmaan_media.full_media_path', $config['web_root'] . '%kunstmaan_media.media_path%');
 
         $loader->load('services.yml');
         $loader->load('handlers.yml');

src/Kunstmaan/MediaBundle/Resources/config/services.yml

@@ -110,7 +110,7 @@ services:
     kunstmaan_media.filesystem_adapter:
         class: Gaufrette\Adapter\Local
         arguments:
-            - '%kernel.root_dir%/../web%kunstmaan_media.media_path%'
+            - '%kunstmaan_media.full_media_path%'
             - true
 
     kunstmaan_media.filesystem:

src/Kunstmaan/MediaBundle/Tests/DependencyInjection/ConfigurationTest.php

@@ -5,6 +5,7 @@
 use Kunstmaan\MediaBundle\DependencyInjection\Configuration;
 use Matthias\SymfonyConfigTest\PhpUnit\ConfigurationTestCaseTrait;
 use PHPUnit_Framework_TestCase;
+use Symfony\Component\HttpKernel\Kernel;
 
 /**
  * Class ConfigurationTest
@@ -31,9 +32,14 @@ public function testConfigGeneratesAsExpected()
                 'dailymotion' => false,
             ],
             'enable_pdf_preview' => true,
-            'blacklisted_extensions' => []
+            'blacklisted_extensions' => [],
+            'web_root' => '%kernel.project_dir%/web'
         ];
 
+        if (Kernel::VERSION_ID >= 40000) {
+            $array['web_root'] = '%kernel.project_dir%/public';
+        }
+
         $this->assertProcessedConfigurationEquals([$array], $array);
     }
 }

src/Kunstmaan/MediaBundle/Tests/Utils/SymfonyVersionTest.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace Kunstmaan\MediaBundle\Tests\Utils;
+
+use Kunstmaan\MediaBundle\Utils\SymfonyVersion;
+use Symfony\Component\HttpKernel\Kernel;
+
+/**
+ * @covers \Kunstmaan\MediaBundle\Utils\SymfonyVersion
+ */
+class SymfonyVersionTest extends \PHPUnit_Framework_TestCase
+{
+    public function testGetRootWebPath()
+    {
+        $path = SymfonyVersion::getRootWebPath();
+        $this->assertStringStartsWith('%kernel.project_dir%/', $path);
+        $this->assertStringEndsWith(Kernel::VERSION_ID < 40000 ? 'web' : 'public', $path);
+    }
+
+    public function testIsKernelLessThan()
+    {
+        $this->assertTrue(SymfonyVersion::isKernelLessThan(100, 100, 100));
+        $this->assertFalse(SymfonyVersion::isKernelLessThan(1, 1, 1));
+    }
+}

src/Kunstmaan/MediaBundle/Utils/SymfonyVersion.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace Kunstmaan\MediaBundle\Utils;
+
+use Symfony\Component\HttpKernel\Kernel;
+
+/**
+ * @internal
+ */
+class SymfonyVersion
+{
+    /**
+     * @return string
+     */
+    public static function getRootWebPath()
+    {
+        return sprintf('%%kernel.project_dir%%/%s', self::isKernelLessThan(4) ? 'web' : 'public');
+    }
+
+    /**
+     * @return bool
+     */
+    public static function isKernelLessThan($major, $minor = null, $patch = null)
+    {
+        return static::kernelVersionCompare('<', $major, $minor, $patch);
+    }
+
+    /**
+     * @return bool
+     */
+    private static function kernelVersionCompare($operator, $major, $minor = null, $patch = null)
+    {
+        return version_compare(Kernel::VERSION_ID, sprintf("%d%'.02d%'.02d", $major, $minor ?: 0, $patch ?: 0), $operator);
+    }
+}