Merge pull request #18 from tlaverdure/patch-5

Only process if there are ids
corcel · Feb 15, 2017 · 9bc8696 · 9bc8696
2 parents a987e53 + a76ad15
commit 9bc8696
Showing 1 changed file with 12 additions and 11 deletions.
diff --git a/src/Field/Gallery.php b/src/Field/Gallery.php
@@ -28,17 +28,18 @@ class Gallery extends Image implements FieldInterface
      */
     public function process($field)
     {
-        $ids = $this->fetchValue($field);
-        $connection = $this->post->getConnectionName();
-        $attachments = Post::on($connection)->whereIn('ID', $ids)->get();
-
-        $metaDataValues = $this->fetchMultipleMetadataValues($attachments);
-
-        foreach ($attachments as $attachment) {
-            $image = new Image($this->post);
-            $image->fillFields($attachment);
-            $image->fillMetadataFields($metaDataValues[$attachment->ID]);
-            $this->images[] = $image;
+        if ($ids = $this->fetchValue($field)) {
+            $connection = $this->post->getConnectionName();
+            $attachments = Post::on($connection)->whereIn('ID', $ids)->get();
+
+            $metaDataValues = $this->fetchMultipleMetadataValues($attachments);
+
+            foreach ($attachments as $attachment) {
+                $image = new Image($this->post);
+                $image->fillFields($attachment);
+                $image->fillMetadataFields($metaDataValues[$attachment->ID]);
+                $this->images[] = $image;
+            }
         }
     }