maintain 0.2.5 → 0.2.6

data/VERSION

@@ -1 +1 @@
-0.2.5
+0.2.6

data/lib/maintain.rb

@@ -45,35 +45,28 @@ module Maintain
     class_eval <<-EOC, __FILE__
       def #{attribute}=(value)
         # Find the maintainer on this attribute so we can use it to set values.
-        if maintainer = self.class.maintainers[:#{attribute}]
-          changed = #{attribute} != value
-          # Run the exit hook if we're changing the value
-          maintainer.hook(:exit, #{attribute}.name, self) if changed
+        maintainer = self.class.maintainers[:#{attribute}]
+        changed = #{attribute} != value
+        # Run the exit hook if we're changing the value
+        maintainer.hook(:exit, #{attribute}.name, self) if changed
 
-          # Then set the value itself. Maintainer::State will return the value you set,
-          # so if we're setting to nil we get rid of the attribute entirely - it's not
-          # needed and we want the getter to return nil in that case.
-          #{attribute}.set_value(value)
+        # Then set the value itself. Maintainer::State will return the value you set,
+        # so if we're setting to nil we get rid of the attribute entirely - it's not
+        # needed and we want the getter to return nil in that case.
+        #{attribute}.set_value(value)
 
-          # Allow the back end to write values in an ORM-specific way
-          if maintainer.back_end
-            maintainer.back_end.write(self, :#{attribute}, #{attribute}.value)
-          end
-
-          # Last but not least, run the enter hooks for the new value - cause that's how
-          # we do.
-          maintainer.hook(:enter, #{attribute}.name, self) if changed
-        else
-          @#{attribute} = value
+        # Allow the back end to write values in an ORM-specific way
+        if maintainer.back_end
+          maintainer.back_end.write(self, :#{attribute}, #{attribute}.value)
         end
+
+        # Last but not least, run the enter hooks for the new value - cause that's how
+        # we do.
+        maintainer.hook(:enter, #{attribute}.name, self) if changed
       end
 
       def #{attribute}
-        if maintainer = self.class.maintainers[:#{attribute}]
-          @#{attribute} ||= maintainer.value(self)
-        else
-          @#{attribute}
-        end
+        @#{attribute} ||= self.class.maintainers[:#{attribute}].value(self)
       end
     EOC
 
@@ -85,7 +78,15 @@ module Maintain
   alias :maintains :maintain
 
   def maintainers #:nodoc:
-    @maintainers ||= {}
+    @maintainers ||= begin
+      maintainers = {}
+      superk = superclass
+      while superk.respond_to?(:maintainers)
+        maintainers.merge!(superk.maintainers)
+        superk = superk.superclass
+      end
+      maintainers
+    end
   end
 
   if File.file?(version_path = File.join(File.dirname(__FILE__), '..', 'VERSION'))

data/maintain.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{maintain}
-  s.version = "0.2.4"
+  s.version = "0.2.6"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Flip Sasser"]
-  s.date = %q{2011-02-23}
+  s.date = %q{2011-02-28}
   s.description = %q{
       Maintain is a simple state machine mixin for Ruby objects. It supports comparisons, bitmasks,
       and hooks that really work. It can be used for multiple attributes and will always do its best to
@@ -47,7 +47,8 @@ Gem::Specification.new do |s|
     "spec/object_spec.rb",
     "spec/proxy_spec.rb",
     "spec/setting_state_spec.rb",
-    "spec/spec.opts"
+    "spec/spec.opts",
+    "spec/subclass_spec.rb"
   ]
   s.homepage = %q{http://github.com/flipsasser/maintain}
   s.require_paths = ["lib"]
@@ -65,7 +66,8 @@ Gem::Specification.new do |s|
     "spec/maintain_spec.rb",
     "spec/object_spec.rb",
     "spec/proxy_spec.rb",
-    "spec/setting_state_spec.rb"
+    "spec/setting_state_spec.rb",
+    "spec/subclass_spec.rb"
   ]
 
   if s.respond_to? :specification_version then

data/spec/integer_spec.rb

@@ -13,7 +13,7 @@ describe Maintain do
     before :each do
       MaintainTest.maintain :kind, :integer => true do
         state :man, 1
-        state :woman, 2
+        state :woman, 2, :default => true
         state :none, 3
       end
       @maintainer = MaintainTest.new
@@ -24,6 +24,10 @@ describe Maintain do
       @maintainer.none?.should be_true
     end
 
+    it "should handle defaults just fine" do
+      @maintainer.woman?.should be_true
+    end
+
     it "should return valid names, too" do
       @maintainer.kind = :woman
       @maintainer.kind.should == 2

data/spec/subclass_spec.rb

@@ -0,0 +1,37 @@
+# Basic class method specs
+
+require 'lib/maintain'
+
+describe Maintain do
+  before :each do
+    class ::MaintainTest
+      attr_accessor :existant_attribute
+      extend Maintain
+    end
+
+    class ::MaintainTestSubclass < ::MaintainTest; end
+  end
+
+  it "should inherit maintainers from parent classes" do
+    MaintainTest.maintain :status do
+      state :new
+      state :old
+    end
+    MaintainTestSubclass.maintainers[:status].should_not be_nil
+  end
+
+  it "should not propagate maintainers up the class system" do
+    MaintainTest.maintain :status do
+      state :new
+      state :old
+    end
+    MaintainTestSubclass.maintain :foo do
+      state :bar
+      state :baz
+    end
+    MaintainTest.maintainers[:foo].should be_nil
+    MaintainTestSubclass.maintainers[:status].should_not be_nil
+  end
+
+  
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: maintain
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 27
   prerelease: false
   segments: 
   - 0
   - 2
-  - 5
-  version: 0.2.5
+  - 6
+  version: 0.2.6
 platform: ruby
 authors: 
 - Flip Sasser
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-02-23 00:00:00 -05:00
+date: 2011-02-28 00:00:00 -05:00
 default_executable: 
 dependencies: []
 
@@ -56,6 +56,7 @@ files:
 - spec/proxy_spec.rb
 - spec/setting_state_spec.rb
 - spec/spec.opts
+- spec/subclass_spec.rb
 has_rdoc: true
 homepage: http://github.com/flipsasser/maintain
 licenses: []
@@ -103,3 +104,4 @@ test_files:
 - spec/object_spec.rb
 - spec/proxy_spec.rb
 - spec/setting_state_spec.rb
+- spec/subclass_spec.rb