From: brettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Fri, 20 Nov 2009 22:24:26 +0000 (+0000)
Subject: Fixed a bug with group by and order by in elgg_get_entities. #ElggCampBA
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=3c670790fc2d2b4092daee497354ca95d58e2f4f;p=lorea%2Felgg.git

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 0fa08de58..bcf045760 100644
--- 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']) {