scripting cleanup, removed ScriptFile

ruflin · Aug 30, 2017 · 0be4f71 · 0be4f71
1 parent f28072d
commit 0be4f71
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 268 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,10 @@ All notable changes to this project will be documented in this file based on the
 - In QueryString is not allowed to use fields parameters in conjunction with default_field parameter. This is not well documented, it's possibile to understand from [Elasticsearch tests :  QueryStringQueryBuilderTests.java](https://github.com/elastic/elasticsearch/blob/6.0/core/src/test/java/org/elasticsearch/index/query/QueryStringQueryBuilderTests.java#L917) [#1352](https://github.com/ruflin/Elastica/pull/1352)   
 - Index mapping field of type [*'string'*](https://www.elastic.co/guide/en/elasticsearch/reference/5.5/string.html) has been removed from Elasticsearch 6.0 codebase [#1353](https://github.com/ruflin/Elastica/pull/1353)
 - The [created and found](https://github.com/elastic/elasticsearch/pull/25516) fields in index and delete responses became obsolete after the introduction of the result field in index, update and delete responses [#1354](https://github.com/ruflin/Elastica/pull/1354) 
+- Removed file scripts [#24627](https://github.com/elastic/elasticsearch/pull/24627) [#1364](https://github.com/ruflin/Elastica/pull/1364)
+- Removed [groovy script](https://github.com/elastic/elasticsearch/pull/21607) [#1364](https://github.com/ruflin/Elastica/pull/1364)
+- Removed [native script](https://github.com/elastic/elasticsearch/pull/24726) [#1364](https://github.com/ruflin/Elastica/pull/1364)
+- Removed old / removed script language support : javascript, python, mvel [#1364](https://github.com/ruflin/Elastica/pull/1364)
 
 ### Bugfixes
 - Enforce [Content-Type requirement on the layer Rest](https://github.com/elastic/elasticsearch/pull/23146), a [PR on Elastica #1301](https://github.com/ruflin/Elastica/issues/1301) solved it (it has been implemented only in the HTTP Transport), but it was not implemented in the Guzzle Transport. [#1349](https://github.com/ruflin/Elastica/pull/1349)

diff --git a/lib/Elastica/Script/AbstractScript.php b/lib/Elastica/Script/AbstractScript.php
@@ -17,11 +17,7 @@
  */
 abstract class AbstractScript extends AbstractUpdateAction
 {
-    const LANG_MVEL = 'mvel';
-    const LANG_JS = 'js';
-    const LANG_GROOVY = 'groovy';
-    const LANG_PYTHON = 'python';
-    const LANG_NATIVE = 'native';
+    const LANG_MOUSTACHE = 'moustache';
     const LANG_EXPRESSION = 'expression';
     const LANG_PAINLESS = 'painless';
 
@@ -37,7 +33,7 @@ abstract class AbstractScript extends AbstractUpdateAction
      *
      * @throws InvalidException
      *
-     * @return Script|ScriptFile|ScriptId
+     * @return Script|ScriptId
      */
     public static function create($data)
     {
@@ -75,14 +71,6 @@ private static function _createFromArray(array $data)
             );
         }
 
-        if (isset($data['script']['file'])) {
-            return new ScriptFile(
-                $data['script']['file'],
-                $params,
-                $lang
-            );
-        }
-
         if (isset($data['script']['id'])) {
             return new ScriptId(
                 $data['script']['id'],

diff --git a/lib/Elastica/Script/ScriptFile.php b/lib/Elastica/Script/ScriptFile.php
diff --git a/test/Elastica/Index/SettingsTest.php b/test/Elastica/Index/SettingsTest.php
@@ -151,6 +151,7 @@ public function testGetDefaultNumberOfShards()
     {
         $index = $this->_createIndex();
         $index->create([], true);
+        $index->refresh();
 
         $settings = $index->getSettings();
 

diff --git a/test/Elastica/Script/ScriptFileTest.php b/test/Elastica/Script/ScriptFileTest.php
diff --git a/test/Elastica/Script/ScriptIdTest.php b/test/Elastica/Script/ScriptIdTest.php
@@ -143,8 +143,8 @@ public function testSetLang()
     {
         $script = new ScriptId(self::SCRIPT_ID, [], ScriptId::LANG_PAINLESS);
 
-        $this->assertSame($script, $script->setLang(ScriptId::LANG_GROOVY));
-        $this->assertEquals(ScriptId::LANG_GROOVY, $script->getLang());
+        $this->assertSame($script, $script->setLang(ScriptId::LANG_PAINLESS));
+        $this->assertEquals(ScriptId::LANG_PAINLESS, $script->getLang());
     }
 
     /**

diff --git a/test/Elastica/Script/ScriptTest.php b/test/Elastica/Script/ScriptTest.php
@@ -142,8 +142,8 @@ public function testSetLang()
     {
         $script = new Script(self::SCRIPT, [], Script::LANG_PAINLESS);
 
-        $this->assertSame($script, $script->setLang(Script::LANG_GROOVY));
-        $this->assertEquals(Script::LANG_GROOVY, $script->getLang());
+        $this->assertSame($script, $script->setLang(Script::LANG_PAINLESS));
+        $this->assertEquals(Script::LANG_PAINLESS, $script->getLang());
     }
 
     /**