json-schema 2.0.4 → 2.0.5

data/README.textile

@@ -22,7 +22,7 @@ From the git repo:
 
 <pre>
 $ gem build json-schema.gemspec
-$ gem install json-schema-2.0.4.gem
+$ gem install json-schema-2.0.5.gem
 </pre>
 
 

data/lib/json-schema/attributes/additionalproperties.rb

@@ -4,12 +4,16 @@ module JSON
   class Schema
     class AdditionalPropertiesAttribute < Attribute
       def self.validate(current_schema, data, fragments, processor, validator, options = {})
-        if data.is_a?(Hash)
+        if data.is_a?(Hash) && (
+          current_schema.schema['type'].nil? || (
+            current_schema.schema['type'].is_a?(String) &&
+            current_schema.schema['type'].downcase == 'object'
+          )
+        )
           extra_properties = data.keys
-
           extra_properties = remove_valid_properties(extra_properties, current_schema, validator)
 
-          addprop= current_schema.schema['additionalProperties']
+          addprop = current_schema.schema['additionalProperties']
           if addprop.is_a?(Hash)
             matching_properties= extra_properties # & addprop.keys
             matching_properties.each do |key|
@@ -57,6 +61,7 @@ module JSON
 
           extra_properties
       end
+
     end
   end
 end

data/test/test_jsonschema_draft3.rb

@@ -204,9 +204,27 @@ class JSONSchemaDraft3Test < Test::Unit::TestCase
 
     data["a"] = {"b" => "taco"}
     assert(!JSON::Validator.validate(schema,data))
-   end
 
+    # Test an array of unioned-type objects that prevent additionalProperties
+    schema["properties"]["a"] = {
+      'type' => 'array',
+      'items' => {
+        'type' => [
+          { 'type' => 'object', 'properties' => { "b" => { "type" => "integer" } } },
+          { 'type' => 'object', 'properties' => { "c" => { "type" => "string" } } }
+        ],
+        'additionalProperties' => false
+      }
+    }
+
+    data["a"] = [{"b" => 5}, {"c" => "foo"}]
+    errors = JSON::Validator.fully_validate(schema, data)
+    assert(errors.empty?, errors.join("\n"))
 
+    # This should actually pass, because this matches the first schema in the union
+    data["a"] << {"c" => false}
+    assert(JSON::Validator.validate(schema,data))
+  end
 
   def test_required
     # Set up the default datatype

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: json-schema
 version: !ruby/object:Gem::Version
-  version: 2.0.4
+  version: 2.0.5
   prerelease: 
 platform: ruby
 authors: