diff options
| author | Rafael França <rafaelmfranca@gmail.com> | 2016-12-29 12:50:36 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-12-29 12:50:36 -0500 |
| commit | c491bf012948383632e53f874c552041a6e23b36 (patch) | |
| tree | dfd008a1de576b048fc2a207400f7062ef409349 /actionpack/lib | |
| parent | 415e17d0b54681545d36a0f43d4cd8761de77bee (diff) | |
| parent | 0713265fd15f5ce0d6e86e620a2de254ff291f81 (diff) | |
| download | rails-c491bf012948383632e53f874c552041a6e23b36.tar.gz rails-c491bf012948383632e53f874c552041a6e23b36.tar.bz2 rails-c491bf012948383632e53f874c552041a6e23b36.zip | |
Merge pull request #27499 from maclover7/jm-fix-27454
Use `next` instead of `break`; avoid terminating whole loop
Diffstat (limited to 'actionpack/lib')
| -rw-r--r-- | actionpack/lib/action_dispatch/journey/formatter.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/actionpack/lib/action_dispatch/journey/formatter.rb b/actionpack/lib/action_dispatch/journey/formatter.rb index 1d239addf8..f3b8e82d32 100644 --- a/actionpack/lib/action_dispatch/journey/formatter.rb +++ b/actionpack/lib/action_dispatch/journey/formatter.rb @@ -36,7 +36,7 @@ module ActionDispatch route.parts.reverse_each do |key| break if defaults[key].nil? && parameterized_parts[key].present? - break if parameterized_parts[key].to_s != defaults[key].to_s + next if parameterized_parts[key].to_s != defaults[key].to_s break if required_parts.include?(key) parameterized_parts.delete(key) |