diff options
author | Tasso Evangelista <[email protected]> | 2013-05-08 02:13:26 -0300 |
---|---|---|
committer | Tasso Evangelista <[email protected]> | 2013-05-08 02:13:26 -0300 |
commit | d1a02c523a468a021b20d417be8bb1bd38755248 (patch) | |
tree | 6f94b403e099d12a5cb66c8e0c4647fc0e4ca0c2 | |
parent | 01112f5ac14b24aed8038cc0ae99801007ef26b8 (diff) |
Ignore result columns when calling a aggregate function
-rw-r--r-- | idiorm.php | 3 | ||||
-rw-r--r-- | test/QueryBuilderTest.php | 6 |
2 files changed, 9 insertions, 0 deletions
@@ -616,8 +616,11 @@ if('*' != $column) { $column = $this->_quote_identifier($column); } + $result_columns = $this->_result_columns; + $this->_result_columns = array(); $this->select_expr("$sql_function($column)", $alias); $result = $this->find_one(); + $this->_result_columns = $result_columns; $return_value = 0; if($result !== false && isset($result->$alias)) { diff --git a/test/QueryBuilderTest.php b/test/QueryBuilderTest.php index bd888c0..6888c85 100644 --- a/test/QueryBuilderTest.php +++ b/test/QueryBuilderTest.php @@ -477,6 +477,12 @@ class QueryBuilderTest extends PHPUnit_Framework_TestCase { $expected = "SELECT COUNT(*) AS `count` FROM `widget` LIMIT 1"; $this->assertEquals($expected, ORM::get_last_query()); } + + public function testIgnoreSelectAndCount() { + ORM::for_table('widget')->select('test')->count(); + $expected = "SELECT COUNT(*) AS `count` FROM `widget` LIMIT 1"; + $this->assertEquals($expected, ORM::get_last_query()); + } public function testMax() { ORM::for_table('person')->max('height'); |