aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/lib/active_support/buffered_logger.rb
diff options
context:
space:
mode:
authorJeremy Kemper <jeremy@bitsweat.net>2007-09-27 04:50:58 +0000
committerJeremy Kemper <jeremy@bitsweat.net>2007-09-27 04:50:58 +0000
commit6f06ac6e1a2e2b3ebd9025c3e38b867e8279d6c9 (patch)
treeade7d2a79b0e103084a4d7f65bf3fa435b9801a3 /activesupport/lib/active_support/buffered_logger.rb
parent1c68394b68507052103fc36406b140f23c348437 (diff)
downloadrails-6f06ac6e1a2e2b3ebd9025c3e38b867e8279d6c9.tar.gz
rails-6f06ac6e1a2e2b3ebd9025c3e38b867e8279d6c9.tar.bz2
rails-6f06ac6e1a2e2b3ebd9025c3e38b867e8279d6c9.zip
BufferedLogger#add doesn't modify the message argument. Closes #9702.
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@7643 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'activesupport/lib/active_support/buffered_logger.rb')
-rw-r--r--activesupport/lib/active_support/buffered_logger.rb4
1 files changed, 3 insertions, 1 deletions
diff --git a/activesupport/lib/active_support/buffered_logger.rb b/activesupport/lib/active_support/buffered_logger.rb
index c646b43d96..6025e1a6d0 100644
--- a/activesupport/lib/active_support/buffered_logger.rb
+++ b/activesupport/lib/active_support/buffered_logger.rb
@@ -52,7 +52,9 @@ module ActiveSupport
def add(severity, message = nil, progname = nil, &block)
return if @level > severity
message = message || (block && block.call) || progname
- message << "\n" unless message[-1] == ?\n
+ # If a newline is nessesary then create a new message end with a new line.
+ # Ensures that the original message is not mutated.
+ message = "#{message}\n" unless message[-1] == ?\n
@buffer << message
flush if auto_flushing
message