Even more fluent eloquent model via magic setters

neoascetic · neoascetic · commit 3a46721eca65 · 2013-05-08T18:28:22.000+11:00
Now it is possible to use Eloquent's magic setters in chains. For example:

    $model-&gt;set_foo('foo')-&gt;take(10)-&gt;set_bar(some_function());
    // ...even though setters 'foo' and 'bar' are not defined on the model
diff --git a/laravel/database/eloquent/model.php b/laravel/database/eloquent/model.php
@@ -770,6 +770,7 @@ public function __call($method, $parameters)
 		elseif (starts_with($method, 'set_'))
 		{
 			$this->set_attribute(substr($method, 4), $parameters[0]);
+			return $this;
 		}
 
 		// Finally we will assume that the method is actually the beginning of a
diff --git a/laravel/tests/cases/eloquent.test.php b/laravel/tests/cases/eloquent.test.php
@@ -133,6 +133,19 @@ public function testAttributeMagicSetterMethodChangesAttribute()
 		Model::$accessible = null;
 	}
 
+	/**
+	 * Test the Model::__set method allows chaining.
+	 *
+	 * @group laravel
+	 */
+	public function testAttributeMagicSetterMethodAllowsChaining()
+	{
+		$model = new Model;
+		$this->assertInstanceOf('Model', $model->set_foo('foo'));
+		$model->set_bar('bar')->set_baz('baz');
+		$this->assertEquals(array('foo' => 'foo', 'bar' => 'bar', 'baz' => 'baz'), $model->to_array());
+	}
+
 	/**
 	 * Test the Model::__get method.
 	 *
@@ -288,4 +301,4 @@ public function testConvertingToArray()
 
 	}
 
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -770,6 +770,7 @@ public function __call($method, $parameters)`
`770`	`770`	`elseif (starts_with($method, 'set_'))`
`771`	`771`	`{`
`772`	`772`	`$this->set_attribute(substr($method, 4), $parameters[0]);`
	`773`	`+ return $this;`
`773`	`774`	`}`
`774`	`775`
`775`	`776`	`// Finally we will assume that the method is actually the beginning of a`