From 3e3e23f0484b4891a70eecd89f30b8ad81852e8b Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Wed, 12 Aug 2015 17:04:32 -0700 Subject: remove unnecessary deletes These three options are stored in the `scope` chain outside of the options hash. If they are in the options hash, then someone passed them in to `match` and they don't really do anything. So lets remove the code. --- actionpack/lib/action_dispatch/routing/mapper.rb | 4 ---- 1 file changed, 4 deletions(-) diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb index 29daf39aec..f4f34e7924 100644 --- a/actionpack/lib/action_dispatch/routing/mapper.rb +++ b/actionpack/lib/action_dispatch/routing/mapper.rb @@ -64,10 +64,6 @@ module ActionDispatch def self.build(scope, set, path, as, controller, default_action, to, via, formatted, options) options = scope[:options].merge(options) if scope[:options] - options.delete :shallow_path - options.delete :shallow_prefix - options.delete :shallow - defaults = (scope[:defaults] || {}).dup scope_constraints = scope[:constraints] || {} -- cgit v1.2.3