Remove only routing parameters from @params hash

Changes to the @params hash were being lost if routing params were supplied because the whole hash was being reset after processing the route.
sinatra · Oct 6, 2019 · 26c2919 · 26c2919
1 parent 6ee7fc1
commit 26c2919
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/sinatra/base.rb b/lib/sinatra/base.rb
@@ -1019,7 +1019,7 @@ def process_route(pattern, conditions, block = nil, values = [])
 
       params.delete("ignore") # TODO: better params handling, maybe turn it into "smart" object or detect changes
       force_encoding(params)
-      original, @params = @params, @params.merge(params) if params.any?
+      @params = @params.merge(params) if params.any?
 
       regexp_exists = pattern.is_a?(Mustermann::Regular) || (pattern.respond_to?(:patterns) && pattern.patterns.any? {|subpattern| subpattern.is_a?(Mustermann::Regular)} )
       if regexp_exists
@@ -1038,7 +1038,7 @@ def process_route(pattern, conditions, block = nil, values = [])
       @env['sinatra.error.params'] = @params
       raise
     ensure
-      @params = original if original
+      @params = @params.reject { |k, _| params.key? k } if params && params.any?
     end
 
     # No matching route was found or all routes passed. The default