Merge pull request #27 from iksaku/fix-implode-order

Fixed wrong implode() order of parameters
sausin · Jan 11, 2020 · 2becb57 · 2becb57
2 parents 97fb61f + 8a514a1
commit 2becb57
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.gitignore b/.gitignore
@@ -3,3 +3,4 @@ node_modules/
 laravel-ovh.sublime-project
 laravel-ovh.sublime-workspace
 composer.lock
+.idea/
diff --git a/src/OVHServiceProvider.php b/src/OVHServiceProvider.php
@@ -47,7 +47,7 @@ protected function checkConfig($config)
         }
 
         // if the configuration wasn't complete, throw an exception
-        throw new BadMethodCallException('Need following keys '.implode($needKeys, ', '));
+        throw new BadMethodCallException('Need following keys '.implode(', ', $needKeys));
     }
 
     /**
-Original file line number
+Diff line change
@@ Expand Up / @@ -47,7 +47,7 @@ protected function checkConfig($config) @@
             }
             // if the configuration wasn't complete, throw an exception
-            throw new BadMethodCallException('Need following keys '.implode($needKeys, ', '));
+            throw new BadMethodCallException('Need following keys '.implode(', ', $needKeys));
         }
         /**
@@ Expand Down @@