summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Holywell <[email protected]>2013-08-28 04:16:38 -0700
committerSimon Holywell <[email protected]>2013-08-28 04:16:38 -0700
commitb1d868a6ad05c46e9444f4a06707c05b6bc2bf4d (patch)
treee50332a1042cd37517d8b72483cb44a531bfc4a4
parent2a3758e41f3a22b9f78c3397aa861b8e2e33f29f (diff)
parentd1a02c523a468a021b20d417be8bb1bd38755248 (diff)
Merge pull request #120 from tassoevan/develop
Ignore result columns when calling a aggregate function
-rw-r--r--idiorm.php3
-rw-r--r--test/QueryBuilderTest.php6
2 files changed, 9 insertions, 0 deletions
diff --git a/idiorm.php b/idiorm.php
index 09b211c..5a86df2 100644
--- a/idiorm.php
+++ b/idiorm.php
@@ -629,8 +629,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');