Merge pull request #38 from redthor/fix/configuration-mess-report

Fix/configuration mess report

Author: caciobanu

src/AntiMattr/MongoDB/Migrations/Collection/Statistics.php

@@ -118,23 +118,18 @@ protected function getCollectionStats()
         $database = $this->collection->getDatabase();
         $name = $this->collection->getName();
 
-        try {
-            if (!$data = $database->command(['collStats' => $name])) {
-                $message = sprintf(
-                    'Statistics not found for collection %s',
-                    $name
-                );
-                throw new Exception($message);
-            }
-            if (isset($data['errmsg'])) {
-                throw new Exception($data['errmsg']);
-            }
+        if (!$data = $database->command(['collStats' => $name])) {
+            $message = sprintf(
+                'Statistics not found for collection %s',
+                $name
+            );
+            throw new Exception($message);
+        }
 
-            return $data;
-        } catch (Exception $e) {
-            throw $e;
+        if (isset($data['errmsg'])) {
+            throw new Exception($data['errmsg']);
         }
 
-        return [];
+        return $data;
     }
 }

src/AntiMattr/MongoDB/Migrations/Configuration/AbstractFileConfiguration.php

@@ -103,6 +103,11 @@ class Configuration
      */
     private $outputWriter;
 
+    /**
+     * @var string
+     */
+    private $file;
+
     /**
      * @param \Doctrine\MongoDB\Connection               $connection
      * @param \AntiMattr\MongoDB\Migrations\OutputWriter $outputWriter
@@ -202,6 +207,8 @@ public function getMigrations()
     public function setMigrationsDatabaseName($databaseName)
     {
         $this->migrationsDatabaseName = $databaseName;
+
+        return $this;
     }
 
     /**
@@ -218,6 +225,8 @@ public function getMigrationsDatabaseName()
     public function setMigrationsCollectionName($collectionName)
     {
         $this->migrationsCollectionName = $collectionName;
+
+        return $this;
     }
 
     /**
@@ -234,6 +243,8 @@ public function getMigrationsCollectionName()
     public function setMigrationsDirectory($migrationsDirectory)
     {
         $this->migrationsDirectory = $migrationsDirectory;
+
+        return $this;
     }
 
     /**
@@ -252,6 +263,8 @@ public function getMigrationsDirectory()
     public function setMigrationsNamespace($migrationsNamespace)
     {
         $this->migrationsNamespace = $migrationsNamespace;
+
+        return $this;
     }
 
     /**
@@ -268,6 +281,8 @@ public function getMigrationsNamespace()
     public function setMigrationsScriptDirectory($scriptsDirectory)
     {
         $this->migrationsScriptDirectory = $scriptsDirectory;
+
+        return $this;
     }
 
     /**
@@ -278,6 +293,24 @@ public function getMigrationsScriptDirectory()
         return $this->migrationsScriptDirectory;
     }
 
+    /**
+     * @param string $file
+     */
+    public function setFile($file)
+    {
+        $this->file = $file;
+
+        return $this;
+    }
+
+    /**
+     * @return string|null
+     */
+    public function getFile(): ?string
+    {
+        return $this->file;
+    }
+
     /**
      * Returns all migrated versions from the versions collection, in an array.
      *
@@ -348,6 +381,8 @@ public function getUnavailableMigratedVersions()
     public function setName($name)
     {
         $this->name = $name;
+
+        return $this;
     }
 
     /**