Fixing script get and delete by removing lang from endpoint url

Author: feydan

src/Elasticsearch/Client.php

@@ -1213,15 +1213,13 @@ public function updateByQuery($params = array())
     public function getScript($params)
     {
         $id = $this->extractArgument($params, 'id');
-        $lang = $this->extractArgument($params, 'lang');
 
         /** @var callback $endpointBuilder */
         $endpointBuilder = $this->endpoints;
 
         /** @var \Elasticsearch\Endpoints\Script\Get $endpoint */
         $endpoint = $endpointBuilder('Script\Get');
-        $endpoint->setID($id)
-                 ->setLang($lang);
+        $endpoint->setID($id);
         $endpoint->setParams($params);
 
         return $this->performRequest($endpoint);
@@ -1238,15 +1236,13 @@ public function getScript($params)
     public function deleteScript($params)
     {
         $id = $this->extractArgument($params, 'id');
-        $lang = $this->extractArgument($params, 'lang');
 
         /** @var callback $endpointBuilder */
         $endpointBuilder = $this->endpoints;
 
         /** @var \Elasticsearch\Endpoints\Script\Delete $endpoint */
         $endpoint = $endpointBuilder('Script\Delete');
-        $endpoint->setID($id)
-                 ->setLang($lang);
+        $endpoint->setID($id);
         $endpoint->setParams($params);
 
         return $this->performRequest($endpoint);

src/Elasticsearch/Endpoints/Script/Delete.php

@@ -18,44 +18,19 @@
  */
 class Delete extends AbstractEndpoint
 {
-    /** @var string */
-    private $lang;
-
-    /**
-     * @param string $lang
-     *
-     * @return $this
-     */
-    public function setLang($lang)
-    {
-        if (isset($lang) !== true) {
-            return $this;
-        }
-
-        $this->lang = $lang;
-
-        return $this;
-    }
-
     /**
      * @throws \Elasticsearch\Common\Exceptions\RuntimeException
      * @return string
      */
     public function getURI()
     {
-        if (isset($this->lang) !== true) {
-            throw new Exceptions\RuntimeException(
-                'lang is required for Put'
-            );
-        }
         if (isset($this->id) !== true) {
             throw new Exceptions\RuntimeException(
                 'id is required for put'
             );
         }
         $id   = $this->id;
-        $lang = $this->lang;
-        $uri  = "/_scripts/$lang/$id";
+        $uri  = "/_scripts/$id";
 
         return $uri;
     }

src/Elasticsearch/Endpoints/Script/Get.php

@@ -18,44 +18,19 @@
  */
 class Get extends AbstractEndpoint
 {
-    /** @var string */
-    private $lang;
-
-    /**
-     * @param string $lang
-     *
-     * @return $this
-     */
-    public function setLang($lang)
-    {
-        if (isset($lang) !== true) {
-            return $this;
-        }
-
-        $this->lang = $lang;
-
-        return $this;
-    }
-
     /**
      * @throws \Elasticsearch\Common\Exceptions\RuntimeException
      * @return string
      */
     public function getURI()
     {
-        if (isset($this->lang) !== true) {
-            throw new Exceptions\RuntimeException(
-                'lang is required for Put'
-            );
-        }
         if (isset($this->id) !== true) {
             throw new Exceptions\RuntimeException(
                 'id is required for put'
             );
         }
         $id   = $this->id;
-        $lang = $this->lang;
-        $uri  = "/_scripts/$lang/$id";
+        $uri  = "/_scripts/$id";
 
         return $uri;
     }