Commit: f9c1de630f35366f489cd10711971f8d22fcd0db

Author: Joël Perras | Date: 2010-08-18 10:40:19 -0400
Fixing bug with `Model::find('list')`. Fixes #121. Added missing test for `find('list') call.
diff --git a/libraries/lithium/data/Model.php b/libraries/lithium/data/Model.php index 3ce3bee..ce39393 100644 --- a/libraries/lithium/data/Model.php +++ b/libraries/lithium/data/Model.php @@ -962,9 +962,9 @@ class Model extends \lithium\core\StaticObject { array_map( function($entity) use (&$result, $meta) { - $result[$entity->{$meta['key']}] = $entity->{$meta['title']}; + $result[$entity[$meta['key']]] = $entity[$meta['title']]; }, - $chain->next($self, $params, $chain) + $chain->next($self, $params, $chain)->data() ); return $result; }, diff --git a/libraries/lithium/tests/cases/data/ModelTest.php b/libraries/lithium/tests/cases/data/ModelTest.php index 34bd001..60551ea 100644 --- a/libraries/lithium/tests/cases/data/ModelTest.php +++ b/libraries/lithium/tests/cases/data/ModelTest.php @@ -229,6 +229,12 @@ class ModelTest extends \lithium\test\Unit { $this->assertEqual($expected, $result->text); } + public function testSimpleFindList() { + $result = MockComment::find('list'); + $this->assertTrue(!empty($result)); + $this->assertTrue(is_array($result)); + } + public function testFilteredFind() { MockComment::applyFilter('find', function($self, $params, $chain) { $result = $chain->next($self, $params, $chain);