Split Create out to its own internal endpoint for simplicity

Author: polyfractal

src/Elasticsearch/Client.php

@@ -604,13 +604,12 @@ public function create($params)
         /** @var callback $endpointBuilder */
         $endpointBuilder = $this->endpoints;
 
-        /** @var \Elasticsearch\Endpoints\Index $endpoint */
-        $endpoint = $endpointBuilder('Index');
+        /** @var \Elasticsearch\Endpoints\Create $endpoint */
+        $endpoint = $endpointBuilder('Create');
         $endpoint->setID($id)
                  ->setIndex($index)
                  ->setType($type)
-                 ->setBody($body)
-                 ->createIfAbsent();
+                 ->setBody($body);
         $endpoint->setParams($params);
 
         return $this->performRequest($endpoint);

src/Elasticsearch/Endpoints/Create.php

@@ -0,0 +1,107 @@
+<?php
+
+namespace Elasticsearch\Endpoints;
+
+use Elasticsearch\Common\Exceptions;
+
+/**
+ * Class Create
+ *
+ * @category Elasticsearch
+ * @package  Elasticsearch\Endpoints
+ * @author   Zachary Tong <zach@elastic.co>
+ * @license  http://www.apache.org/licenses/LICENSE-2.0 Apache2
+ * @link     http://elastic.co
+ */
+class Create extends AbstractEndpoint
+{
+    /**
+     * @param array $body
+     *
+     * @throws \Elasticsearch\Common\Exceptions\InvalidArgumentException
+     * @return $this
+     */
+    public function setBody($body)
+    {
+        if (isset($body) !== true) {
+            return $this;
+        }
+
+        $this->body = $body;
+
+        return $this;
+    }
+
+    /**
+     * @throws \Elasticsearch\Common\Exceptions\RuntimeException
+     * @return string
+     */
+    public function getURI()
+    {
+        if (isset($this->index) !== true) {
+            throw new Exceptions\RuntimeException(
+                'index is required for Create'
+            );
+        }
+
+        if (isset($this->type) !== true) {
+            throw new Exceptions\RuntimeException(
+                'type is required for Create'
+            );
+        }
+
+        if (isset($this->id) !== true) {
+            throw new Exceptions\RuntimeException(
+                'id is required for Create'
+            );
+        }
+
+        $id    = $this->id;
+        $index = $this->index;
+        $type  = $this->type;
+        return "/$index/$type/$id/_create";
+    }
+
+    /**
+     * @return string[]
+     */
+    public function getParamWhitelist()
+    {
+        return array(
+            'consistency',
+            'op_type',
+            'parent',
+            'percolate',
+            'refresh',
+            'replication',
+            'routing',
+            'timeout',
+            'timestamp',
+            'ttl',
+            'version',
+            'version_type',
+            'pipeline'
+        );
+    }
+
+    /**
+     * @return string
+     */
+    public function getMethod()
+    {
+        return 'PUT';
+    }
+
+    /**
+     * @return array
+     * @throws \Elasticsearch\Common\Exceptions\RuntimeException
+     */
+    public function getBody()
+    {
+        if (isset($this->body) !== true) {
+            throw new Exceptions\RuntimeException('Document body must be set for create request');
+        } else {
+            return $this->body;
+        }
+    }
+}

src/Elasticsearch/Endpoints/Index.php

@@ -71,11 +71,6 @@ public function getURI()
         if (isset($id) === true) {
             $uri = "/$index/$type/$id";
         }
-
-        if ($this->createIfAbsent === true) {
-            $uri .= $this->addCreateFlag();
-        }
-
         return $uri;
     }
 
@@ -125,15 +120,4 @@ public function getBody()
             return $this->body;
         }
     }
-
-    private function addCreateFlag()
-    {
-        if (isset($this->id) === true) {
-            return '/_create';
-        } else {
-            $this->params['op_type'] = 'create';
-
-            return "";
-        }
-    }
 }