Remove Validator::coerce()

Author: erayd

src/JsonSchema/Validator.php

@@ -26,7 +26,9 @@ class Validator extends BaseConstraint
     /**
      * Validates the given data against the schema and returns an object containing the results
      * Both the php object and the schema are supposed to be a result of a json_decode call.
-     * The validation works as defined by the schema proposal in http://json-schema.org
+     * The validation works as defined by the schema proposal in http://json-schema.org.
+     *
+     * Note that the first argument is passwd by reference, so you must pass in a variable.
      *
      * {@inheritDoc}
      */
@@ -42,16 +44,4 @@ public function check(&$value, $schema = null, array $config = array())
 
         $this->addErrors(array_unique($validator->getErrors(), SORT_REGULAR));
     }
-
-    /**
-     * Does everything that check does, but will also coerce string values in the input to their expected
-     * types defined in the schema whenever possible. Note that the first argument is passed by reference,
-     * so you must pass in a variable.
-     *
-     * {@inheritDoc}
-     */
-    public function coerce(&$value, $schema = null)
-    {
-        return $this->check($value, $schema, array('coerce-types' => true));
-    }
 }

tests/Constraints/CoerciveTest.php

@@ -31,7 +31,7 @@ public function testValidCoerceCasesUsingAssoc($input, $schema)
 
         $value = json_decode($input, true);
 
-        $validator->coerce($value, $schema);
+        $validator->check($value, $schema, array('coerce-types' => true));
         $this->assertTrue($validator->isValid(), print_r($validator->getErrors(), true));
     }
 
@@ -52,7 +52,7 @@ public function testValidCoerceCases($input, $schema, $errors = array())
         $this->assertTrue(gettype($value->integer) == "string");
         $this->assertTrue(gettype($value->boolean) == "string");
 
-        $validator->coerce($value, $schema);
+        $validator->check($value, $schema, array('coerce-types' => true));
 
         $this->assertTrue(gettype($value->number) == "double");
         $this->assertTrue(gettype($value->integer) == "integer");
@@ -88,7 +88,7 @@ public function testInvalidCoerceCases($input, $schema, $errors = array())
 
         $validator = new Validator(new Factory($schemaStorage, null, array('checkMode' => $checkMode)));
         $value = json_decode($input);
-        $validator->coerce($value, $schema);
+        $validator->check($value, $schema, array('coerce-types' => true));
 
         if (array() !== $errors) {
             $this->assertEquals($errors, $validator->getErrors(), print_r($validator->getErrors(), true));
@@ -108,7 +108,7 @@ public function testInvalidCoerceCasesUsingAssoc($input, $schema, $errors = arra
 
         $validator = new Validator(new Factory($schemaStorage, null, array('checkMode' => $checkMode)));
         $value = json_decode($input, true);
-        $validator->coerce($value, $schema);
+        $validator->check($value, $schema, array('coerce-types' => true));
 
         if (array() !== $errors) {
             $this->assertEquals($errors, $validator->getErrors(), print_r($validator->getErrors(), true));