Fix for #2201 - Cannot use "having" in a CDbCriteria to count AR model values (results in SQL error) #2230

Merged
merged 3 commits into from Mar 21, 2013

Projects

None yet

3 participants

@ivokund
Contributor
ivokund commented Mar 20, 2013

Fix for #2201 - adds the missing params array from the query object.

ivokund added some commits Mar 20, 2013
@ivokund ivokund Update CActiveFinder.php
Adds the missing "params" array from the query object.
16e7bac
@ivokund ivokund Update CHANGELOG 6e69cff
@klimov-paul klimov-paul was assigned Mar 21, 2013
@klimov-paul klimov-paul commented on an outdated diff Mar 21, 2013
@@ -30,6 +30,7 @@ Version 1.1.14 work in progress
- Bug #2123: Fixed error in plural rules handling if locale has no plural rules defined (cebe, stepanselyuk)
- Bug #2146: CEmailValidator fix when fsockopen() can output uncatched error 'Connection refused (61)' (armab)
- Bug #2159: Fixed SQL syntax for delete command with join in MySQL (serebrov)
+- Bug #2201: Cannot use "having" in a CDbCriteria to count AR model values, results in SQL error (ivokund)
@klimov-paul
klimov-paul Mar 21, 2013 Member

Please change to:
"Cannot use "having" with bound params in a CActiveRecord::count()"
or something similar.

@klimov-paul
Member

Please pull the changes from "upstream master".

@samdark
Member
samdark commented Mar 21, 2013

@ivokund it would be great if you woul create a unit test for your case.

@klimov-paul klimov-paul merged commit b6f4cdb into yiisoft:master Mar 21, 2013
@klimov-paul
Member

Merged.
Unit test has been updated.

@klimov-paul
Member

@ivokund, thank you for working on this one.

@ivokund ivokund deleted the opus-online:ar-fix-2201 branch Mar 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment