Fixes #30: Check directory is writable before moving downloaded file.…

… Also check that is exists afterwards.
meanbee · Mar 14, 2016 · c7c41ef · c7c41ef
1 parent 637ec7e
commit c7c41ef
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/Meanbee/Magedbm/Command/GetCommand.php b/src/Meanbee/Magedbm/Command/GetCommand.php
@@ -218,8 +218,16 @@ protected function backupMove($file)
     {
         $filename = $this->getFilePath($file);
         $newFilename = getcwd() . '/' . $file;
-        if (rename($filename, $newFilename)) {
-            $this->getOutput()->writeln("<info>Downloaded to $newFilename.</info>");
+
+        if (!is_writable(getcwd())) {
+            $this->getOutput()->writeln("<info>Unable to write to current working directory. Check $filename</info>");
+            return;
+        }
+
+        $result = rename($filename, $newFilename);
+
+        if ($result && file_exists($newFilename)) {
+            $this->getOutput()->writeln("<info>Downloaded to $newFilename</info>");
         } else {
             $this->getOutput()->writeln("<error>Failed to move backup to current working directory. Check $filename</error>");
         }