Doctrine

Timestamp:

02/10/08 14:49:22 (17 months ago)

Author:

pookey

Message:

fixes #732 - yes, it's a nasty hack, but we have a new parser on the way

Files:

• branches/0.10/lib/Doctrine/Query/Having.php (modified) (1 diff)
• branches/0.9/lib/Doctrine/Query/Having.php (modified) (1 diff)
• trunk/lib/Doctrine/Query/Having.php (modified) (1 diff)

branches/0.10/lib/Doctrine/Query/Having.php

@@ -94 +94 @@
         $value     = implode(' ', $tokens);
         $part .= ' ' . $operator . ' ' . $value;
+        // check the RHS for aggregate functions
+        if (strpos($value, '(') !== false) {
+          $value = $this->parseAggregateFunction($value);
+        }
         return $part;
     }

branches/0.9/lib/Doctrine/Query/Having.php

@@ -94 +94 @@
         $value     = implode(' ', $tokens);
         $part .= ' ' . $operator . ' ' . $value;
+        // check the RHS for aggregate functions
+        if (strpos($value, '(') !== false) {
+          $value = $this->parseAggregateFunction($value);
+        }
         return $part;
     }

trunk/lib/Doctrine/Query/Having.php

@@ -94 +94 @@
         $value     = implode(' ', $tokens);
         $part .= ' ' . $operator . ' ' . $value;
+        // check the RHS for aggregate functions
+        if (strpos($value, '(') !== false) {
+          $value = $this->parseAggregateFunction($value);
+        }
         return $part;
     }