Added yii2tech\filestorage\local\Storage::$dirPermission

Author: klimov-paul

CHANGELOG.md

@@ -1,6 +1,12 @@
 Yii 2 File Storage extension Change Log
 =======================================
 
+1.1.4 under development
+-----------------------
+
+- Enh #10: Added `yii2tech\filestorage\local\Storage::$dirPermission` allowing setup of directory permissions different from file permissions (klimov-paul)
+
+
 1.1.3, November 3, 2017
 -----------------------
 

README.md

@@ -84,7 +84,8 @@ return [
             'class' => 'yii2tech\filestorage\local\Storage',
             'basePath' => '@webroot/files',
             'baseUrl' => '@web/files',
-            'filePermission' => 0777,
+            'dirPermission' => 0775,
+            'filePermission' => 0755,
             'buckets' => [
                 'tempFiles' => [
                     'baseSubPath' => 'temp',

local/Bucket.php

@@ -149,9 +149,8 @@ protected function resolveFullFileName($fileName)
     protected function resolvePath($path)
     {
         if (!file_exists($path)) {
-            $dirPermission = $this->getStorage()->filePermission;
             $this->log("creating file path '{$path}'");
-            FileHelper::createDirectory($path, $dirPermission);
+            FileHelper::createDirectory($path, $this->getStorage()->dirPermission);
         }
         if (!is_dir($path)) {
             throw new Exception("Path '{$path}' is not a directory!");

local/Storage.php

@@ -20,7 +20,8 @@
  *     'class' => 'yii2tech\filestorage\local\Storage',
  *     'basePath' => '@webroot/files',
  *     'baseUrl' => '@web/files',
- *     'filePermission' => 0777,
+ *     'dirPermission' => 0775,
+ *     'filePermission' => 0755,
  *     'buckets' => [
  *         'tempFiles' => [
  *             'baseSubPath' => 'temp',
@@ -53,12 +54,30 @@ class Storage extends BaseStorage
      * created in the process. Defaults to 0755 (owner rwx, group rx and others rx).
      */
     public $filePermission = 0755;
+    /**
+     * @var int the chmod permission for the directories created in the process.
+     * If not set - value of [[filePermission]] will be used.
+     * @since 1.1.4
+     */
+    public $dirPermission;
     /**
      * @var string file system path, which is basic for all buckets.
      */
     private $_basePath = '';
 
 
+    /**
+     * {@inheritdoc}
+     */
+    public function init()
+    {
+        parent::init();
+
+        if ($this->dirPermission === null) {
+            $this->dirPermission = $this->filePermission;
+        }
+    }
+
     /**
      * @param string $basePath file system path, which is basic for all buckets.
      */