changed the default directory for the upload temp dir, made the dir m…

…andatory
symfony · Apr 23, 2011 · 8cc5caf · 8cc5caf
1 parent 02c66e6
commit 8cc5caf
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/form.xml b/src/Symfony/Bundle/FrameworkBundle/Resources/config/form.xml
@@ -13,8 +13,6 @@
         <parameter key="form.validation_groups">Default</parameter>
         <parameter key="file.temporary_storage.class">Symfony\Component\HttpFoundation\File\SessionBasedTemporaryStorage</parameter>
         <parameter key="file.temporary_storage.secret">abcdef</parameter>
-        <parameter key="file.temporary_storage.directory"></parameter>
-
     </parameters>
 
     <services>
@@ -66,7 +64,7 @@
         <service id="file.temporary_storage" class="%file.temporary_storage.class%">
             <argument type="service" id="session" />
             <argument>%file.temporary_storage.secret%</argument>
-            <argument>%file.temporary_storage.directory%</argument>
+            <argument>%kernel.cache_dir%/upload</argument>
         </service>
 
         <!-- CoreExtension -->

diff --git a/src/Symfony/Component/HttpFoundation/File/SessionBasedTemporaryStorage.php b/src/Symfony/Component/HttpFoundation/File/SessionBasedTemporaryStorage.php
@@ -18,7 +18,7 @@
  */
 class SessionBasedTemporaryStorage extends TemporaryStorage
 {
-    public function __construct(Session $session, $secret, $directory = null)
+    public function __construct(Session $session, $secret, $directory)
     {
         parent::__construct($secret, $directory);
 

diff --git a/src/Symfony/Component/HttpFoundation/File/TemporaryStorage.php b/src/Symfony/Component/HttpFoundation/File/TemporaryStorage.php
@@ -22,12 +22,8 @@ class TemporaryStorage
     private $directory;
     private $secret;
 
-    public function __construct($secret, $directory = null)
+    public function __construct($secret, $directory)
     {
-        if (empty($directory)) {
-            $directory = sys_get_temp_dir();
-        }
-
         $this->directory = realpath($directory);
         $this->secret = $secret;
     }