overider 0.0.1 → 0.1

data/README.mkd

@@ -13,15 +13,18 @@ but may become a problem for larger projects.
 In the [same post](http://blog.jayfields.com/2006/12/ruby-alias-method-alternative.html),
 Jay offered an interesting alternative to `alias`,
 using instead `Module#instance_method` and `Module#define_method`.
-This is much better,
-but I'm not a Ruby ninja yet,
-and I would rather not have to remember the Baroque syntax in `x = self.instance_method(:on!)`
-and `x.bind(self).call` every time I want to do this,
-let alone what they mean when I look at my code several weeks later.
+This is much better, but I'm not quite satisfied.
+While obvious enough to experienced Ruby-hands,
+people who are newer to Ruby (like me!)
+might have trouble remembering the syntax of `x = self.instance_method(:x)`
+and `x.bind(self).call` every time they need to use this.
+What I'd really like is a way to override defined methods that's a no-brainer.
 
 So I thought about how I could take Jay's example one step further 
 and after some trial and lots of error came up with this.
 
+Thanks and credit go to [Jay Fields](http://blog.jayfields.com/) for his post.
+
 ## Description
 A mix-in module that allows for super-clean method over-riding without resorting to `alias` 
 or making unbound methods visible.
@@ -48,3 +51,7 @@ or making unbound methods visible.
 ## See also
   * [Ruby: Alias method alternative, Jay Fields](http://blog.jayfields.com/2006/12/ruby-alias-method-alternative.html)
   * <https://github.com/soveran/override>
+
+## LICENSE
+* [Ruby License](http://www.ruby-lang.org/en/LICENSE.txt)
+* <a rel="license" href="http://creativecommons.org/licenses/by-sa/3.0/"><img alt="Creative Commons License" style="border-width:0" src="http://i.creativecommons.org/l/by-sa/3.0/88x31.png" /></a><br /><span xmlns:dct="http://purl.org/dc/terms/" href="http://purl.org/dc/dcmitype/Text" property="dct:title" rel="dct:type">overider.gem</span> by <a xmlns:cc="http://creativecommons.org/ns#" href="http://westside-consulting.com/" property="cc:attributionName" rel="cc:attributionURL">Lawrence Siden</a> is licensed under a <a rel="license" href="http://creativecommons.org/licenses/by-sa/3.0/">Creative Commons Attribution-ShareAlike 3.0 Unported License</a>.<br />Based on a work at <a xmlns:dct="http://purl.org/dc/terms/" href="https://github.com/lsiden" rel="dct:source">github.com</a>.

data/lib/overider.rb

@@ -5,11 +5,9 @@ module Overider
     orig_unbound_method = self.instance_method(sym)
 
     define_method(sym) do |*a, &blk|
-      orig_self = self
-      obj = Object.new
 
-      obj.define_singleton_method(:overiden) do |*a, &blk|
-        orig_bound_method = orig_unbound_method.bind(orig_self)
+      self.define_singleton_method(:overiden) do |*a, &blk|
+        orig_bound_method = orig_unbound_method.bind(self)
 
         if !!blk then
           orig_bound_method.call *a, &blk
@@ -19,9 +17,9 @@ module Overider
       end
 
       if !!blk then
-        obj.instance_exec(*a, blk, &pr)
+        self.instance_exec *a, blk, &pr
       else
-        obj.instance_exec(*a, &pr)
+        self.instance_exec *a, &pr
       end
     end
   end

data/lib/overider/version.rb

@@ -1,3 +1,3 @@
 module Overider
-  VERSION = "0.0.1"
+  VERSION = "0.1"
 end

data/spec/overider_spec.rb

@@ -153,4 +153,53 @@ describe Overider do
     result = F.new.hello ->{ "block" }
     result.should == "hello block in F"
   end
+
+  it 'can access instance variables' do
+    module HelloModule
+      def hello
+        "hello "
+      end
+    end
+
+    class G
+      include HelloModule
+      extend Overider
+
+      def initialize
+        @one = 'one'
+        @two = 'two'
+      end
+
+      # In the overide block, we have to declare blk as a normal block arg, not &blk.
+      # It must be a Proc or lambda
+      overide :hello do 
+        overiden + "#{@one} and #{@two}"
+      end
+    end
+
+    result = G.new.hello ->{ "block" }
+    result.should == "hello one and two"
+  end
+
+  it 'can access instance variables defined in base class' do
+    module HelloModule
+      def hello
+        "hello "
+      end
+    end
+
+    class H < G
+      include HelloModule
+      extend Overider
+
+      # In the overide block, we have to declare blk as a normal block arg, not &blk.
+      # It must be a Proc or lambda
+      overide :hello do 
+        overiden + " and again: #{@one} and #{@two}"
+      end
+    end
+
+    result = H.new.hello ->{ "block" }
+    result.should == "hello one and two and again: one and two"
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: overider
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: '0.1'
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-12-23 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &76593190 !ruby/object:Gem::Requirement
+  requirement: &79785640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +21,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *76593190
+  version_requirements: *79785640
 description: ! "\n    class A\n      def hello\n        \"hello\"\n      end\n    end\n\n
   \   # Later, I want to overide class A methods\n\n    class A\n      extend Overider\n\n
   \     overide (:hello) do |*a|\n        overiden(*a) + \" overide\"\n      end\n