Fixed a bug with group by and order by in elgg_get_entities. #ElggCampBA

git-svn-id: http://code.elgg.org/elgg/trunk@3694 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/engine/lib/entities.php b/engine/lib/entities.php
index 0fa08de587abb5f374a8c82979c9b48c5bbe1c04..bcf0457608244f9e9fdb6bb05cb38d7fb4c7cb98 100644 (file)
--- a/engine/lib/entities.php
+++ b/engine/lib/entities.php
@@ -1756,12 +1756,12 @@ function elgg_get_entities(array $options = array()) {
        // Add access controls
        $query .= get_access_sql_suffix('e');
        if (!$options['count']) {
-               if ($group_by = sanitise_string($options['group_by'])) {
-                       $query .= " GROUP BY $group_by";
+               if ($options['group_by'] = sanitise_string($options['group_by'])) {
+                       $query .= " GROUP BY {$options['group_by']}";
                }
 
-               if ($order_by = sanitise_string($options['order_by'])) {
-                       $query .= " ORDER BY $order_by";
+               if ($options['order_by'] = sanitise_string($options['order_by'])) {
+                       $query .= " ORDER BY {$options['order_by']}";
                }
 
                if ($options['limit']) {