From 88272fee3629a7da9b9bf234e09659204aedf40a Mon Sep 17 00:00:00 2001 From: Joseph Schilz Date: Tue, 4 Oct 2016 08:09:04 -0700 Subject: [PATCH] Fluent interface for fillFromFields. --- src/orm-wrapper/PropelObjectWrapper.php | 4 +++- test/PropelObjectWrapperTest.php | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/orm-wrapper/PropelObjectWrapper.php b/src/orm-wrapper/PropelObjectWrapper.php index 1cf66ad..0485a28 100644 --- a/src/orm-wrapper/PropelObjectWrapper.php +++ b/src/orm-wrapper/PropelObjectWrapper.php @@ -171,7 +171,7 @@ protected function addBehaviorConstraintsToFields(array $fields) * Expects that $fields contains a set of $fieldName => $field pairs. * * @param FieldInterface[] $fields - * @return void + * @return ObjectWrapperInterface */ public function fillFromFields(array $fields) { @@ -199,6 +199,8 @@ public function fillFromFields(array $fields) } } } + + return $this; } /** diff --git a/test/PropelObjectWrapperTest.php b/test/PropelObjectWrapperTest.php index ebbc9b6..99d8781 100644 --- a/test/PropelObjectWrapperTest.php +++ b/test/PropelObjectWrapperTest.php @@ -2,6 +2,7 @@ namespace Athens\Core\Test; +use Athens\Core\ORMWrapper\ObjectWrapperInterface; use PHPUnit_Framework_TestCase; use Athens\Propel\ORMWrapper\PropelObjectWrapper; @@ -391,9 +392,10 @@ public function testFillFromFields() $field->setValidatedData($values[$fieldName]); } - $wrappedTestClass->fillFromFields($fields); + $result = $wrappedTestClass->fillFromFields($fields); $this->assertEquals($values, $wrappedTestClass->getValues()); + $this->assertInstanceOf(ObjectWrapperInterface::class, $result); } public function testSave()