Changeset 377

Show
Ignore:
Timestamp:
07/01/09 16:44:51 (4 months ago)
Author:
bettse
Message:

correction to order by clause in relationships.php

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • elgg/branches/elgg1.5/engine/lib/relationships.php

    r363 r377  
    662662                if (!$count) { 
    663663                        $query .= " group by e.guid,e.type,e.subtype,e.owner_guid,e.site_guid,e.container_guid,e.access_id,e.time_created,e.time_updated,e.enabled ";//postgres requires all fields being selected to be listed in the group by clause.   
    664             $query .= " order by total desc limit {$limit} offset {$offset}"; // Add order and limit 
     664            $query .= " order by count(distinct e.guid) desc limit {$limit} offset {$offset}"; // Add order and limit 
    665665                        return get_data($query, "entity_row_to_elggstar"); 
    666666                } else { 

  • elgg/branches/elgg1.5/engine/lib/tags.php

    r268 r377  
    137137                $query .= ' and ' . get_access_sql_suffix("e"); // Add access controls 
    138138                 
    139                 $query .= " group by msvalue.string having total > {$threshold} order by total desc limit {$limit} "; 
     139                $query .= " group by msvalue.string having count(msvalue.id) > {$threshold} order by count(msvalue.id) desc limit {$limit} "; 
    140140 
    141141                return get_data($query);