fixed type ambiguity nin Container

michael-donat · Apr 21, 2014 · 0d788e8 · 0d788e8
1 parent 4d60640
commit 0d788e8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/src/VirtualFileSystem/Container.php b/src/VirtualFileSystem/Container.php
@@ -214,6 +214,10 @@ public function move($from, $to)
             $nodeAtToPath = null;
         }
 
+        if(!$toNode instanceof Directory) {
+            throw new NotDirectoryException('Destination not a directory');
+        }
+
         $fromNode->setBasename($newNodeName);
 
         if ($nodeAtToPath instanceof File) {

diff --git a/src/VirtualFileSystem/Structure/File.php b/src/VirtualFileSystem/Structure/File.php
@@ -38,7 +38,7 @@ public function size()
     /**
      * Returns contents.
      *
-     * @return null
+     * @return null|string
      */
     public function data()
     {

diff --git a/src/VirtualFileSystem/Wrapper.php b/src/VirtualFileSystem/Wrapper.php
@@ -15,7 +15,6 @@
 use VirtualFileSystem\Structure\Root;
 use VirtualFileSystem\Wrapper\FileHandler;
 use VirtualFileSystem\Wrapper\DirectoryHandler;
-use VirtualFileSystem\Wrapper\PermissionHelper;
 
 /**
  * Stream wrapper class. This is the class that PHP uses as the stream operations handler.