change fix to not rely on the order of validators

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2018-12-06 21:06:59 -0500 using RuboCop version 0.51.0.
+# on 2018-12-20 21:38:24 +0000 using RuboCop version 0.51.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -38,13 +38,13 @@ Metrics/BlockLength:
 # Offense count: 9
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
-  Max: 288
+  Max: 295
 
 # Offense count: 31
 Metrics/CyclomaticComplexity:
   Max: 14
 
-# Offense count: 1227
+# Offense count: 1234
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
 Metrics/LineLength:
@@ -55,7 +55,7 @@ Metrics/LineLength:
 Metrics/MethodLength:
   Max: 33
 
-# Offense count: 11
+# Offense count: 12
 # Configuration parameters: CountComments.
 Metrics/ModuleLength:
   Max: 220

CHANGELOG.md

@@ -8,7 +8,7 @@
 
 #### Fixes
 
-* [#1852](https://github.com/ruby-grape/grape/pull/1852): Validators checking aliased param instead of original one - [@glaucocustodio](https://github.com/glaucocustodio).
+* [#1852](https://github.com/ruby-grape/grape/pull/1852): `allow_blank` called after `as` when the original param is not blank - [@glaucocustodio](https://github.com/glaucocustodio).
 * [#1844](https://github.com/ruby-grape/grape/pull/1844): Enforce `:tempfile` to be a `Tempfile` object in `File` validator - [@Nyangawa](https://github.com/Nyangawa).
 
 ### 1.2.2 (2018/12/07)

lib/grape/endpoint.rb

@@ -258,6 +258,7 @@ def run
         else
           run_filters before_validations, :before_validation
           run_validators validations, request
+          remove_aliased_params
           run_filters after_validations, :after_validation
           response_object = @block ? @block.call(self) : nil
         end
@@ -319,7 +320,14 @@ def build_helpers
       Module.new { helpers.each { |mod_to_include| include mod_to_include } }
     end
 
-    private :build_stack, :build_helpers
+    def remove_aliased_params
+      return unless route_setting(:aliased_params)
+      route_setting(:aliased_params).flat_map(&:keys).each do |aliased_param|
+        @params.delete(aliased_param)
+      end
+    end
+
+    private :build_stack, :build_helpers, :remove_aliased_params
 
     def helpers
       lazy_initialize! && @helpers

lib/grape/validations/params_scope.rb

@@ -293,8 +293,6 @@ def validates(attrs, validations)
         # type casted values
         coerce_type validations, attrs, doc_attrs, opts
 
-        move_alias_to_end(validations)
-
         validations.each do |type, options|
           validate(type, options, attrs, doc_attrs, opts)
         end
@@ -433,11 +431,6 @@ def validate_value_coercion(coerce_type, *values_list)
         end
       end
 
-      def move_alias_to_end(validations)
-        alias_validation = validations.delete(:as)
-        validations[:as] = alias_validation if alias_validation
-      end
-
       def extract_message_option(attrs)
         return nil unless attrs.is_a?(Array)
         opts = attrs.last.is_a?(Hash) ? attrs.pop : {}

lib/grape/validations/validators/as.rb

@@ -8,7 +8,6 @@ def initialize(attrs, options, required, scope, opts = {})
 
       def validate_param!(attr_name, params)
         params[@alias] = params[attr_name]
-        params.delete(attr_name)
       end
     end
   end