Bugfix: update config with given value for loop, not value of duration. (#3)

Author: qrazi

src/GifCreate.php

@@ -109,15 +109,17 @@ private function configure(array $config = [])
      * @param int  $durations
      * @param null $loop
      *
-     * @return \pomirleanu\GifCreate\GifCreate
+     * @return \Pomirleanu\GifCreate\GifCreate
      * @throws \Exception
      */
     public function create($frames, $durations = 10, $loop = null)
     {
         if (count($frames) < 2) {
             throw new \Exception(sprintf($this->errors[ 'ERR06' ]));
         }
-        $this->setIfSet($durations, $loop);
+
+        // Update loop value if passed in.
+        $this->mergeConfigIfSet('loop', $loop);
 
         // Check if $frames is a dir; get all files in ascending order if yes (else die):
         if (! is_array($frames)) {
@@ -146,15 +148,19 @@ public function create($frames, $durations = 10, $loop = null)
         return $this->buildFrameSources($frames, $durations);
     }
 
-
-    private function setIfSet($loop)
+    /**
+     * If value is non-null value, overwrite or add to internal config.
+     *
+     * @param $key
+     * @param $value
+     */
+    private function mergeConfigIfSet($key, $value)
     {
-        if ($loop !== null) {
-            $this->config[ 'loop' ] = $loop;
+        if ($value !== null) {
+            $this->config[$key] = $value;
         }
     }
 
-
     /**
      * Building the frame sources for the given images
      *