Ignore scrutinizer bugs.

triadev · Dec 3, 2018 · e727140 · e727140
1 parent ee97053
commit e727140
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/Console/Commands/ImportFileMigrations.php b/src/Console/Commands/ImportFileMigrations.php
@@ -97,8 +97,8 @@ private function importMigrationSteps(string $migration, array $migrationSteps)
             foreach ($this->getValidMigrationSteps($migrationSteps) as $validMigrationStep) {
                 $this->elasticsearchMigrationService->addMigrationStep(
                     $migration,
-                    array_get($validMigrationStep, 'type'),
-                    array_get($validMigrationStep, 'params'),
+                    /** @scrutinizer ignore-type */ array_get($validMigrationStep, 'type'),
+                    /** @scrutinizer ignore-type */ array_get($validMigrationStep, 'params'),
                     array_get($validMigrationStep, 'priority', 1),
                     array_get($validMigrationStep, 'stopOnFailure', true)
                 );
@@ -121,19 +121,19 @@ private function getValidMigrationSteps(array $migrationSteps) : array
 
         foreach ($migrationSteps as $migrationStep) {
             $step = require $migrationStep;
-
-            $validator = Validator::make($step, [
+            
+            if (Validator::make($step, [
                 'type' => 'required|string',
                 'params' => 'required|array',
                 'priority' => 'integer',
                 'stopOnFailure' => 'boolean'
-            ]);
-
-            if ($validator->fails()) {
+            ])->fails()) {
                 throw new \Exception("The migration step is invalid.");
             }
 
-            if (!$migrationTypes->isMigrationTypeValid(array_get($step, 'type'))) {
+            if (!$migrationTypes->isMigrationTypeValid(
+                /** @scrutinizer ignore-type */ array_get($step, 'type')
+            )) {
                 throw new \Exception("The migration step type is invalid.");
             }