tagged_logger 0.3.2 → 0.3.3

data/README.markdown

@@ -9,7 +9,8 @@ Every time you want to log something, simply write:
 and do not worry about what kind of logger you use and how your code accesses it.
 You may configure these things later, one day writing to STDOUT works for you, another
 day you'll need something more sophisticated, like several log files each serving 
-different components and for different audience. 
+different components and for different audience. [Here](http://afurmanov.com/2009/10/19/tagged-logger-introduction)
+I have described in more details why I created it.
 
 ## Installation
     $ gem install tagged_logger

data/lib/tagged_logger.rb

@@ -1,5 +1,5 @@
 require 'delegate'
-require 'facets/dictionary'
+require 'hashery/dictionary'
 
 class TaggedLogger
   @rename_rules = Dictionary.new
@@ -13,8 +13,13 @@ class TaggedLogger
       init
     end
     
-    def rules(&block)
-      add_logger_generator_in_Object
+    def rules(options = {}, &block)
+      klasses = []
+#      klasses << AbstractController::Base if Object.const_defined? :AbstractController
+      klasses << Object
+      override = options.delete :override
+      klasses = klasses.select{ |klass| !klass.respond_to?(:logger, true)}  if !override
+      klasses.each{ |klass| inject_logger_method_in_call_chain(klass)}
       instance_eval(&block)
     end
     
@@ -119,9 +124,8 @@ class TaggedLogger
       end
     end
     
-    def add_logger_generator_in_Object
-      return if Object.respond_to?(:logger, true) #no harm
-      Object.class_eval do
+    def inject_logger_method_in_call_chain(definee_klass)
+      definee_klass.class_eval do
         def logger
           klass = self.class == Class ? self : self.class
           result = klass.class_eval do

data/tagged_logger.rb

@@ -0,0 +1 @@
+require File.dirname(__FILE__) + '/lib/tagged_logger.rb'

data/test/test.rb

@@ -35,6 +35,34 @@ class TaggedLoggerTest < Test::Unit::TestCase
       assert_nothing_raised { Class.new.logger }
     end
     
+    should "not override exsiting #logger method" do
+      GlobalObject1 = Object
+      GlobalObject1.instance_eval { define_method(:logger){@@stub_out}}
+      class GlobalObject1
+        def foo; logger.write("debug"); end
+      end
+      TaggedLogger.rules do
+        reset
+        debug(/.*/) { |level, tag, msg| debugger; @@stub_out.write("hmmm, something new") }
+      end
+      mock(@@stub_out).write("debug")
+      Object.new.foo
+    end
+
+    should "override exsiting #logger method when intialized with :override" do
+      GlobalObject2 = Object
+      class GlobalObject2
+        def logger; @logger ||= Logger.new('/dev/null');  end
+        def foo; logger.debug("debug"); end
+      end
+      TaggedLogger.rules(:override => true) do
+        reset
+        debug(/.*/) { |level, tag, msg| @@stub_out.write(msg) }
+      end
+      mock(@@stub_out).write("debug")
+      Object.new.foo
+    end
+    
     context "everything gets logged to @@stub_out;" do
       setup do 
         TaggedLogger.rules do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: tagged_logger
 version: !ruby/object:Gem::Version 
-  hash: 23
+  hash: 21
   prerelease: false
   segments: 
   - 0
   - 3
-  - 2
-  version: 0.3.2
+  - 3
+  version: 0.3.3
 platform: ruby
 authors: 
 - Aleksandr Furmanov
@@ -15,11 +15,11 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-09-28 00:00:00 -05:00
+date: 2010-10-17 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: facets
+  name: hashery
   prerelease: false
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
@@ -74,7 +74,7 @@ files:
 - MIT-LICENSE
 - Rakefile
 - README.markdown
-- VERSION
+- tagged_logger.rb
 - lib/tagged_logger.rb
 - test/expected_examples_output.txt
 - test/test.rb

data/VERSION

@@ -1 +0,0 @@
-0.3.1