Added ActiveRecord::Base.colorize_logging to control whether to use colors in...

Added ActiveRecord::Base.colorize_logging to control whether to use colors in logs or not (on by default)

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@860 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

activerecord/CHANGELOG

 *SVN*
 
+* Added ActiveRecord::Base.colorize_logging to control whether to use colors in logs or not (on by default)
+
 * Added support for timestamp with time zone in PostgreSQL #560 [Scott Barron]
 
 * Added MultiparameterAssignmentErrors and AttributeAssignmentError exceptions #777 [demetrius]. Documentation:

activerecord/lib/active_record/base.rb

@@ -264,6 +264,12 @@ def self.inherited(child) #:nodoc:
     cattr_accessor :pluralize_table_names
     @@pluralize_table_names = true
 
+    # Determines whether or not to use ANSI codes to colorize the logging statements committed by the connection adapter. These colors
+    # makes it much easier to overview things during debugging (when used through a reader like +tail+ and on a black background), but
+    # may complicate matters if you use software like syslog. This is true, by default.
+    cattr_accessor :colorize_logging
+    @@colorize_logging = true
+
     # Determines whether to use Time.local (using :local) or Time.utc (using :utc) when pulling dates and times from the database.
     # This is set to :local by default.
     cattr_accessor :default_timezone

activerecord/lib/active_record/connection_adapters/abstract_adapter.rb

@@ -422,6 +422,7 @@ def log_info(sql, name, runtime)
         end
 
         def format_log_entry(message, dump = nil)
+          if ActiveRecord::Base.colorize_logging
             if @@row_even then
               @@row_even = false; caller_color = "1;32"; message_color = "4;33"; dump_color = "1;37"
             else
@@ -432,6 +433,9 @@ def format_log_entry(message, dump = nil)
             log_entry << "   \e[#{dump_color}m%s\e[m" % dump if dump.kind_of?(String) && !dump.nil?
             log_entry << "   \e[#{dump_color}m%p\e[m" % dump if !dump.kind_of?(String) && !dump.nil?
             log_entry
+          else
+            "%s  %s" % [message, dump]
+          end
         end
     end