@@ -97,9 +97,7 @@ module DataMapper def validation_property_value(name) - return self.instance_variable_get("@#{name}") if self.instance_variables.include?(name) - return self.send(name) if self.respond_to?(name) - nil + self.respond_to?(name, true) ? self.send(name) : nil end # Get the corresponding Resource property, if it exists. dm-validations/lib/dm-validations.rb @@ -336,4 +336,17 @@ describe DataMapper::Validate do invoice.all_valid?.should == true end + + it "should retrieve private instance variables for validation" do + class Raft + include DataMapper::Resource + property :length, Integer, :accessor => :private + + def initialize(length) + @length = length + end + end + + Raft.new(10).validation_property_value("length").should == 10 + end end dm-validations/spec/integration/validation_spec.rb f6a5af1e6e9f23bb5b7a2afe00d39d8cbdcf2f7f Bernerd Schaefer bj.schaefer@gmail.com http://github.com/sam/dm-more/commit/ae861dd819363f8c22a3f657d3fc8377209fa945 ae861dd819363f8c22a3f657d3fc8377209fa945 2008-07-10T07:35:00-07:00 2008-07-10T07:35:00-07:00 Applied David Yeu's fix for private properties on dm-validations. Resolves #373. 695763afb20d886637e3be34730ad1de3a23935a Bernerd Schaefer bj.schaefer@gmail.com