deac/couchrest_model
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Validation inheritance: do deeper dup so that child classes don't add to the parent's validations

Browse source
This commit is contained in:
Geoff Buesing 2009-03-25 00:18:28 -05:00 committed by Matt Aimonetti
parent e466b20df9
commit 49d19e2a29
2 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/couchrest/mixins/validation.rb
View file

@ -63,7 +63,9 @@ module CouchRest
# share the validations with subclasses # share the validations with subclasses
def self.inherited(subklass) def self.inherited(subklass)
subklass.instance_variable_set(:@validations, self.validators.dup) self.validators.contexts.each do |k, v|
subklass.validators.contexts[k] = v.dup
end
end end
EOS EOS

6
spec/couchrest/more/extended_doc_subclass_spec.rb
View file

@ -42,6 +42,12 @@ describe "Subclassing an ExtendedDocument" do
validated_fields.should include(:job_title) validated_fields.should include(:job_title)
end end
it "should not add to the parent's validations" do
validated_fields = Card.validators.contexts[:default].map{|v| v.field_name}
validated_fields.should_not include(:extension_code)
validated_fields.should_not include(:job_title)
end
it "should inherit default property values" do it "should inherit default property values" do
@card.bg_color.should == '#ccc' @card.bg_color.should == '#ccc'
end end