diff options
| author | Aaron Patterson <aaron.patterson@gmail.com> | 2016-02-09 13:48:03 -0800 |
|---|---|---|
| committer | Aaron Patterson <aaron.patterson@gmail.com> | 2016-02-09 13:48:03 -0800 |
| commit | b5eb2423b6e431ba53e3836d58449e7e810096b4 (patch) | |
| tree | f7ef1862aea0b3c9087bf7dbbb7963a84fe56ab6 /actionpack | |
| parent | b2a9047558e20e35681eae8d47ad7a95c3e1d19e (diff) | |
| download | rails-b5eb2423b6e431ba53e3836d58449e7e810096b4.tar.gz rails-b5eb2423b6e431ba53e3836d58449e7e810096b4.tar.bz2 rails-b5eb2423b6e431ba53e3836d58449e7e810096b4.zip | |
`log_process_action` will return an array, so use `empty?`
We don't need to use active support in this case because we know the
type that will be returned.
Diffstat (limited to 'actionpack')
| -rw-r--r-- | actionpack/lib/action_controller/log_subscriber.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/actionpack/lib/action_controller/log_subscriber.rb b/actionpack/lib/action_controller/log_subscriber.rb index 450a04779f..a0917b4fdb 100644 --- a/actionpack/lib/action_controller/log_subscriber.rb +++ b/actionpack/lib/action_controller/log_subscriber.rb @@ -25,7 +25,7 @@ module ActionController status = ActionDispatch::ExceptionWrapper.status_code_for_exception(exception_class_name) end message = "Completed #{status} #{Rack::Utils::HTTP_STATUS_CODES[status]} in #{event.duration.round}ms" - message << " (#{additions.join(" | ".freeze)})" unless additions.blank? + message << " (#{additions.join(" | ".freeze)})" unless additions.empty? message << "\n\n" if defined?(Rails.env) && Rails.env.development? message |