required properties ignored

[justinrainbow]

required properties ignored

[coreation]

Noticed this as well, imho the required statement is elementary to perform some schema validation, and currently this is bugged :).

[pietercolpaert]

Is this bug in every version? Or only in the dev-master?

The code seems to check this:

https://github.com/justinrainbow/json-schema/blob/master/src/JsonSchema/Constraints/Object.php#L70

[mchiocca]

The aforementioned code in Constraints/Object.php is checking for the "requires" attribute, not the "required" attribute, which is different.

The value of "required" must be an array containing property names that are required. For example,

{
    "properties": {
        "foo": {"type": "integer"}
    },
    "required": ["foo"]
}

However, the current implementation seems to support "required" only as an attribute on individual properties. For example,

{
    "properties": {
        "foo": {"type": "integer", "required": true}
    }
}

The "requires" attribute is from as early as Draft v0. I'm not that sure that it's supported in Draft v3+. The use of "required" can be done using either the former or the latter example.

[mchiocca]

This issue has been fixed by merging #41 into master. Both the draft-03 and 04 specifications for required are now supported.