Changeset 23590 for branches

Timestamp:

2014/08/14 11:37:36 (10 years ago)

Author:

kimoto

Message:

recursiveMkDirのPHP4サポートは不要
Windows環境の場合はmkdirのmodeは効かないのでテストを変更する
isAbsoluteRealPathの修正

Location:

branches/version-2_13-dev

Files:

• data/class/util/SC_Utils.php (modified) (2 diffs)
• tests/class/util/SC_Utils/SC_Utils_recursiveMkDirTest.php (modified) (1 diff)

branches/version-2_13-dev/data/class/util/SC_Utils.php

@@ -1795 +1795 @@
      * この関数は, パスの存在チェックを行なわないため注意すること.
      *
-     * @param string $realpath チェック対象のパス
+     * use File_Util::isAbsolute
+     * http://pear.php.net/package/File_Util/
+     *
+     * @param string $path チェック対象のパス
      * @return boolean 絶対パスの場合 true
      */
-    public static function isAbsoluteRealPath($realpath)
-    {
-        if (strpos(PHP_OS, 'WIN') === false) {
-            return substr($realpath, 0, 1) == '/';
-        } else {
-            return preg_match('/^[a-zA-Z]:(\\\|\/)/', $realpath) ? true : false;
-        }
+    public static function isAbsoluteRealPath($path)
+    {
+        if (preg_match('/(?:\/|\\\)\.\.(?=\/|$)/', $path)) {
+            return false;
+        }
+        if (!strncasecmp(PHP_OS, 'win', 3)) {
+            return (($path{0} == '/') ||  preg_match('/^[a-zA-Z]:(\\\|\/)/', $path));
+        }
+        return ($path{0} == '/') || ($path{0} == '~');
     }
 
     /**
      * ディレクトリを再帰的に作成する.
-     *
-     * mkdir 関数の $recursive パラメーターを PHP4 でサポートする.
      *
      * @param  string  $pathname ディレクトリのパス
@@ -1817 +1820 @@
      * @see http://jp.php.net/mkdir
      */
-    public static function recursiveMkdir($pathname, $mode = 0777)
-    {
-        /*
-         * SC_Utils_Ex への再帰は無限ループやメモリリークの懸念
-         * 自クラスへ再帰する.
-         */
-        is_dir(dirname($pathname)) || SC_Utils::recursiveMkdir(dirname($pathname), $mode);
-
-        return is_dir($pathname) || @mkdir($pathname, $mode);
+    public static function recursiveMkdir($path, $mode = 0777)
+    {
+        // Windows環境ではmodeは効かない
+        return mkdir($path, $mode, true);
     }
 

branches/version-2_13-dev/tests/class/util/SC_Utils/SC_Utils_recursiveMkDirTest.php

@@ -35 +35 @@
 {
 
-  static $TMP_DIR;
+    static $TMP_DIR;
 
-  protected function setUp()
-  {
-    self::$TMP_DIR = realpath(dirname(__FILE__)) . "/../../../tmp";
-    SC_Helper_FileManager::deleteFile(self::$TMP_DIR);
-    mkdir(self::$TMP_DIR, 0777, true);
-// parent::setUp();
-  }
+    protected function setUp()
+    {
+        self::$TMP_DIR = realpath(dirname(__FILE__)) . "/../../../tmp";
+        SC_Helper_FileManager::deleteFile(self::$TMP_DIR);
+        mkdir(self::$TMP_DIR, 0777, true);
+        // parent::setUp();
+    }
 
-  protected function tearDown()
-  {
-    // parent::tearDown();
-  }
+    protected function tearDown()
+    {
+        // parent::tearDown();
+    }
 
-  /////////////////////////////////////////
-  public function testRecursiveMkdir_パーミッションを指定した場合_指定のパーミッションでディレクトリが作られる()
-  {
-    $path = realpath(dirname(__FILE__)) . "/../../../tmp/dir1/dir2/dir3/";
-    $mode = 0755;
+    /////////////////////////////////////////
+    public function testRecursiveMkdir_パーミッションを指定した場合_指定のパーミッションでディレクトリが作られる()
+    {
+        $path = realpath(dirname(__FILE__)) . "/../../../tmp/dir1/dir2/dir3/";
+        $mode = 0755;
 
-    $result = SC_Utils::recursiveMkdir($path, $mode);
-    $this->expected = '0755';
-    $this->actual = substr(sprintf('%o', fileperms($path)), -4);
+        $result = SC_Utils::recursiveMkdir($path, $mode);
+        if (DIRECTORY_SEPARATOR == '\\') {
+            // Windows環境ではパーミッションを指定したディレクトリ作成が出来ない
+            $this->expected = true;
+            $this->actual = file_exists($path);
+            $this->verify('作成したディレクトリがあるかどうか');
+        } else {
+            $this->expected = '0755';
+            $this->actual = substr(sprintf('%o', fileperms($path)), -4);
+            $this->verify('作成したディレクトリのパーミッション');
+        }
 
-    $this->verify('作成したディレクトリのパーミッション');
-  }
+    }
 
-  public function testRecursiveMkdir_パーミッションを指定しない場合_0777でディレクトリが作られる()
-  {
-    $path = realpath(dirname(__FILE__)) . "/../../../tmp/dir1/dir2/dir3/";
+    public function testRecursiveMkdir_パーミッションを指定しない場合_0777でディレクトリが作られる()
+    {
+        $path = realpath(dirname(__FILE__)) . "/../../../tmp/dir1/dir2/dir3/";
 
-    $result = SC_Utils::recursiveMkdir($path);
-    $this->expected = '0777';
-    $this->actual = substr(sprintf('%o', fileperms($path)), -4);
+        $result = SC_Utils::recursiveMkdir($path);
+        if (DIRECTORY_SEPARATOR == '\\') {
+            // Windows環境ではパーミッションを指定したディレクトリ作成が出来ない
+            $this->expected = true;
+            $this->actual = file_exists($path);
+            $this->verify('作成したディレクトリがあるかどうか');
+        } else {
+            $this->expected = '0777';
+            $this->actual = substr(sprintf('%o', fileperms($path)), -4);
+            $this->verify('作成したディレクトリのパーミッション');
+        }
 
-    $this->verify('作成したディレクトリのパーミッション');
-  }
+    }
 
-  //////////////////////////////////////////
+    //////////////////////////////////////////
 }