From 278a40ba38762f674e915b611dd600cd527205a9 Mon Sep 17 00:00:00 2001 From: Mark Story Date: Sat, 20 Dec 2014 12:20:31 -0500 Subject: [PATCH] Remove remaining required references from Form\Schema. The requiredness of a field comes from the field having a validation rule. --- src/Form/Schema.php | 1 - tests/TestCase/Form/SchemaTest.php | 6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/Form/Schema.php b/src/Form/Schema.php index 0db1f276843..651a3efabc2 100644 --- a/src/Form/Schema.php +++ b/src/Form/Schema.php @@ -35,7 +35,6 @@ class Schema { 'type' => null, 'length' => null, 'precision' => null, - 'required' => false, ]; /** diff --git a/tests/TestCase/Form/SchemaTest.php b/tests/TestCase/Form/SchemaTest.php index e460a0a0dac..b20f94a712b 100644 --- a/tests/TestCase/Form/SchemaTest.php +++ b/tests/TestCase/Form/SchemaTest.php @@ -51,7 +51,7 @@ public function testAddingFields() { $this->assertEquals(['name'], $schema->fields()); $res = $schema->field('name'); - $expected = ['type' => 'string', 'length' => null, 'required' => false]; + $expected = ['type' => 'string', 'length' => null, 'precision' => null]; $this->assertEquals($expected, $res); $res = $schema->addField('email', 'string'); @@ -59,7 +59,7 @@ public function testAddingFields() { $this->assertEquals(['name', 'email'], $schema->fields()); $res = $schema->field('email'); - $expected = ['type' => 'string', 'length' => null, 'required' => false]; + $expected = ['type' => 'string', 'length' => null, 'precision' => null]; $this->assertEquals($expected, $res); } @@ -72,7 +72,7 @@ public function testAddingFieldsWhitelist() { $schema = new Schema(); $schema->addField('name', ['derp' => 'derp', 'type' => 'string']); - $expected = ['type' => 'string', 'length' => null, 'required' => false]; + $expected = ['type' => 'string', 'length' => null, 'precision' => null]; $this->assertEquals($expected, $schema->field('name')); }