Changeset 4953

Timestamp:

Feb 23, 2010, 3:38:38 PM (14 years ago)

Author:

plg

Message:

feature 1453 added: ability to check for uniqueness on filename. No change on
the default behavior: uniqueness is set on md5sum.

Location:

branches/2.0

Files:

• include/config_default.inc.php (modified) (1 diff)
• include/ws_functions.inc.php (modified) (4 diffs)
• ws.php (modified) (1 diff)

branches/2.0/include/config_default.inc.php

@@ -322 +322 @@
 // Define using double password type in admin's users management panel
 $conf['double_password_type_in_admin'] = false;
+
+// how should we check for unicity when adding a photo. Can be 'md5sum' or
+// 'filename'
+$conf['uniqueness_mode'] = 'md5sum';
 
 // +-----------------------------------------------------------------------+

branches/2.0/include/ws_functions.inc.php

@@ -1205 +1205 @@
 
   // does the image already exists ?
+  if ('md5sum' == $conf['uniqueness_mode'])
+  {
+    $where_clause = "md5sum = '".$params['original_sum']."'";
+  }
+  if ('filename' == $conf['uniqueness_mode'])
+  {
+    $where_clause = "file = '".$params['original_filename']."'";
+  }
+  
   $query = '
 SELECT
     COUNT(*) AS counter
   FROM '.IMAGES_TABLE.'
-  WHERE md5sum = \''.$params['original_sum'].'\'
+  WHERE '.$where_clause.'
 ;';
   list($counter) = mysql_fetch_row(pwg_query($query));
@@ -1598 +1607 @@
 function ws_images_exist($params, &$service)
 {
+  global $conf;
+  
   if (!is_admin() or is_adviser())
   {
@@ -1603 +1614 @@
   }
 
-  // search among photos the list of photos already added, based on md5sum
-  // list
-  $md5sums = preg_split(
-    '/[\s,;\|]/',
-    $params['md5sum_list'],
-    -1,
-    PREG_SPLIT_NO_EMPTY
-    );
-
-  $query = '
+  $split_pattern = '/[\s,;\|]/';
+
+  if ('md5sum' == $conf['uniqueness_mode'])
+  {
+    // search among photos the list of photos already added, based on md5sum
+    // list
+    $md5sums = preg_split(
+      $split_pattern,
+      $params['md5sum_list'],
+      -1,
+      PREG_SPLIT_NO_EMPTY
+    );
+
+    $query = '
 SELECT
     id,
@@ -1619 +1634 @@
   WHERE md5sum IN (\''.implode("','", $md5sums).'\')
 ;';
-  $id_of_md5 = simple_hash_from_query($query, 'md5sum', 'id');
-
-  $result = array();
-
-  foreach ($md5sums as $md5sum)
-  {
-    $result[$md5sum] = null;
-    if (isset($id_of_md5[$md5sum]))
-    {
-      $result[$md5sum] = $id_of_md5[$md5sum];
+    $id_of_md5 = simple_hash_from_query($query, 'md5sum', 'id');
+
+    $result = array();
+
+    foreach ($md5sums as $md5sum)
+    {
+      $result[$md5sum] = null;
+      if (isset($id_of_md5[$md5sum]))
+      {
+        $result[$md5sum] = $id_of_md5[$md5sum];
+      }
+    }
+  }
+  
+  if ('filename' == $conf['uniqueness_mode'])
+  {
+    // search among photos the list of photos already added, based on
+    // filename list
+    $filenames = preg_split(
+      $split_pattern,
+      $params['filename_list'],
+      -1,
+      PREG_SPLIT_NO_EMPTY
+    );
+
+    $query = '
+SELECT
+    id,
+    file
+  FROM '.IMAGES_TABLE.'
+  WHERE file IN (\''.implode("','", $filenames).'\')
+;';
+    $id_of_filename = simple_hash_from_query($query, 'file', 'id');
+
+    $result = array();
+
+    foreach ($filenames as $filename)
+    {
+      $result[$filename] = null;
+      if (isset($id_of_filename[$filename]))
+      {
+        $result[$filename] = $id_of_filename[$filename];
+      }
     }
   }

branches/2.0/ws.php

@@ -258 +258 @@
     'ws_images_exist',
     array(
-      'md5sum_list'=> array(),
+      'md5sum_list'=> array('default' => null),
+      'filename_list' => array('default' => null),
       ),
     'check existence of a photo list'