#782: Load dependencies from legacy validators

Author: nezhar

jsonschema/_legacy_validators.py

@@ -27,6 +27,30 @@ def dependencies_draft3(validator, dependencies, instance, schema):
                     yield ValidationError(message % (each, property))
 
 
+def dependencies_draft4_draft6_draft7(validator, dependencies, instance, schema):
+    """
+    The dependencies keyword has been deprecated since draft 2019-09 and has been split into dependentRequired
+    and dependentSchemas.
+    """
+    if not validator.is_type(instance, "object"):
+        return
+
+    for property, dependency in dependencies.items():
+        if property not in instance:
+            continue
+
+        if validator.is_type(dependency, "array"):
+            for each in dependency:
+                if each not in instance:
+                    message = "%r is a dependency of %r"
+                    yield ValidationError(message % (each, property))
+        else:
+            for error in validator.descend(
+                    instance, dependency, schema_path=property,
+            ):
+                yield error
+
+
 def disallow_draft3(validator, disallow, instance, schema):
     for disallowed in _utils.ensure_list(disallow):
         if validator.is_valid(instance, {"type": [disallowed]}):

jsonschema/_validators.py

@@ -301,30 +301,6 @@ def maxLength(validator, mL, instance, schema):
         yield ValidationError("%r is too long" % (instance,))
 
 
-def dependencies(validator, dependencies, instance, schema):
-    """
-    The dependencies keyword has been deprecated since draft 2019-09 and has been split into dependentRequired
-    and dependentSchemas.
-    """
-    if not validator.is_type(instance, "object"):
-        return
-
-    for property, dependency in dependencies.items():
-        if property not in instance:
-            continue
-
-        if validator.is_type(dependency, "array"):
-            for each in dependency:
-                if each not in instance:
-                    message = "%r is a dependency of %r"
-                    yield ValidationError(message % (each, property))
-        else:
-            for error in validator.descend(
-                instance, dependency, schema_path=property,
-            ):
-                yield error
-
-
 def dependentRequired(validator, dependentRequired, instance, schema):
     """
     Split from dependencies

jsonschema/validators.py

@@ -336,7 +336,7 @@ def extend(validator, validators=(), version=None, type_checker=None):
         u"additionalProperties": _validators.additionalProperties,
         u"allOf": _validators.allOf,
         u"anyOf": _validators.anyOf,
-        u"dependencies": _validators.dependencies,
+        u"dependencies": _legacy_validators.dependencies_draft4_draft6_draft7,
         u"enum": _validators.enum,
         u"format": _validators.format,
         u"items": _legacy_validators.items_draft3_draft4,
@@ -373,7 +373,7 @@ def extend(validator, validators=(), version=None, type_checker=None):
         u"anyOf": _validators.anyOf,
         u"const": _validators.const,
         u"contains": _legacy_validators.contains_draft6_draft7,
-        u"dependencies": _validators.dependencies,
+        u"dependencies": _legacy_validators.dependencies_draft4_draft6_draft7,
         u"enum": _validators.enum,
         u"exclusiveMaximum": _validators.exclusiveMaximum,
         u"exclusiveMinimum": _validators.exclusiveMinimum,
@@ -412,7 +412,7 @@ def extend(validator, validators=(), version=None, type_checker=None):
         u"anyOf": _validators.anyOf,
         u"const": _validators.const,
         u"contains": _legacy_validators.contains_draft6_draft7,
-        u"dependencies": _validators.dependencies,
+        u"dependencies": _legacy_validators.dependencies_draft4_draft6_draft7,
         u"enum": _validators.enum,
         u"exclusiveMaximum": _validators.exclusiveMaximum,
         u"exclusiveMinimum": _validators.exclusiveMinimum,