Changeset 2451 for trunk/include/ws_functions.inc.php

Timestamp:

Jul 23, 2008, 2:56:22 AM (16 years ago)

Author:

rvelices

Message:

• normalize behaviour of query search versus std search (now both return items already sorted and permission checked); also more optimized sql queries (in some cases)

File:

• trunk/include/ws_functions.inc.php (modified) (3 diffs)

trunk/include/ws_functions.inc.php

@@ -824 +824 @@
   $order_by = ws_std_image_sql_order($params, 'i.');
 
+  $super_order_by = false;
   if ( !empty($order_by) )
   {
     global $conf;
     $conf['order_by'] = 'ORDER BY '.$order_by;
-    $page['super_order_by']=1; // quick_search_result might be faster
+    $super_order_by=true; // quick_search_result might be faster
   }
 
   $search_result = get_quick_search_results($params['query'],
-    implode(',', $where_clauses) );
-  $image_ids = $search_result['items'];
-
-  $image_ids = array_slice($image_ids,
-    $params['page']*$params['per_page'],
-    $params['per_page'] );
+      $super_order_by,
+      implode(',', $where_clauses)
+    );
+
+  $image_ids = array_slice(
+      $search_result['items'],
+      $params['page']*$params['per_page'],
+      $params['per_page']
+    );
 
   if ( count($image_ids) )
@@ -843 +847 @@
     $query = '
 SELECT * FROM '.IMAGES_TABLE.'
-  WHERE id IN ('
-        .wordwrap(implode(', ', $image_ids), 80, "\n")
-        .')';
-
+  WHERE id IN ('.implode(',', $image_ids).')';
+
+    $image_ids = array_flip($image_ids);
     $result = pwg_query($query);
     while ($row = mysql_fetch_assoc($result))
@@ -863 +866 @@
       }
       $image = array_merge( $image, ws_std_get_urls($row) );
-      array_push($images, $image);
-    }
-
-    $image_ids = array_flip($image_ids);
-    usort(
-        $images,
-        create_function('$i1,$i2', 'global $image_ids; return $image_ids[$i1["id"]]-$image_ids[$i2["id"]];')
-      );
+      $images[$image_ids[$image['id']]] = $image;
+    }
+    ksort($images, SORT_NUMERIC);
+    $images = array_values($images);
   }