lib/elasticsearch/api/actions/delete_script.rb in elasticsearch-api-6.1.0 vs lib/elasticsearch/api/actions/delete_script.rb in elasticsearch-api-6.2.0

@@ -3,29 +3,30 @@
     module Actions
 
       # Remove an indexed script from Elasticsearch
       #
       # @option arguments [String] :id Script ID (*Required*)
-      # @option arguments [String] :lang Script language (*Required*)
+      # @option arguments [String] :lang Script language
       # @option arguments [Number] :version Explicit version number for concurrency control
       # @option arguments [String] :version_type Specific version type (options: internal, external, external_gte, force)
       #
       # @see http://www.elasticsearch.org/guide/en/elasticsearch/reference/master/modules-scripting.html
       #
       def delete_script(arguments={})
         raise ArgumentError, "Required argument 'id' missing"   unless arguments[:id]
-        raise ArgumentError, "Required argument 'lang' missing" unless arguments[:lang]
-
-        valid_params = [
-          :version,
-          :version_type ]
-
         method = HTTP_DELETE
-        path   = "_scripts/#{arguments.delete(:lang)}/#{arguments[:id]}"
-        params = Utils.__validate_and_extract_params arguments, valid_params
+        path   = "_scripts/#{arguments.has_key?(:lang) ? "#{arguments.delete(:lang)}/" : ''}#{arguments[:id]}"
+        params = Utils.__validate_and_extract_params arguments, ParamsRegistry.get(__method__)
         body   = nil
 
         perform_request(method, path, params, body).body
       end
+
+      # Register this action with its valid params when the module is loaded.
+      #
+      # @since 6.2.0
+      ParamsRegistry.register(:delete_script, [
+          :version,
+          :version_type ].freeze)
     end
   end
 end