Changeset 2569

Timestamp:

Sep 22, 2008, 11:47:03 PM (16 years ago)

Author:

plg

Message:

improvement: WebService method pwg.images.add can set fill #images table
columns. rank is directly related to a category and several categories can
be linked at once. Basic technical metadata {filesize, width, height} are
automaticaly filled.

Location:

trunk

Files:

• include/ws_functions.inc.php (modified) (4 diffs)
• tools/piwigo_remote.pl (modified) (2 diffs)
• ws.php (modified) (1 diff)

trunk/include/ws_functions.inc.php

@@ -975 +975 @@
   // fwrite($fh_log, 'output: '.md5_file($thumbnail_path)."\n");
 
+  list($width, $height) = getimagesize($file_path);
+  
   // database registration
   $insert = array(
@@ -982 +984 @@
     'name' => $params['name'],
     'path' => $file_path,
-    );
+    'filesize' => floor(filesize($file_path)/1024),
+    'width' => $width,
+    'height' => $height,
+    );
+
+  $info_columns = array(
+    'name',
+    'author',
+    'comment',
+    'level',
+    'date_creation',
+    );
+
+  foreach ($info_columns as $key)
+  {
+    if (isset($params[$key]))
+    {
+      $insert[$key] = $params[$key];
+    }
+  }
 
   include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');
@@ -993 +1014 @@
   $image_id = mysql_insert_id();
 
-  $insert = array(
-    'category_id' => $params['category_id'],
-    'image_id' => $image_id,
-    );
-
-  if ('auto' == $params['rank'])
-  {
-    $query = '
+  // let's add links between the image and the categories
+  //
+  // $params['categories'] should look like 123,12;456,auto;789 which means:
+  //
+  // 1. associate with category 123 on rank 12
+  // 2. associate with category 456 on automatic rank
+  // 3. associate with category 789 on automatic rank
+  if (isset($params['categories']))
+  {
+    $cat_ids = array();
+    $rank_on_category = array();
+    $search_current_ranks = false;
+    
+    $tokens = explode(';', $params['categories']);
+    foreach ($tokens as $token)
+    {
+      list($cat_id, $rank) = explode(',', $token);
+
+      array_push($cat_ids, $cat_id);
+      
+      if (!isset($rank))
+      {
+        $rank = 'auto';
+      }
+      $rank_on_category[$cat_id] = $rank;
+
+      if ($rank == 'auto')
+      {
+        $search_current_ranks = true;
+      }
+    }
+
+    $cat_ids = array_unique($cat_ids);
+
+    if (count($cat_ids) > 0)
+    {
+      if ($search_current_ranks)
+      {
+        $query = '
 SELECT
+    category_id,
     MAX(rank) AS max_rank
   FROM '.IMAGE_CATEGORY_TABLE.'
   WHERE rank IS NOT NULL
-    AND category_id = '.$params['category_id'].'
+    AND category_id IN ('.implode(',', $cat_ids).')
+  GROUP BY category_id
 ;';
-    $row = mysql_fetch_assoc(pwg_query($query));
-    $insert['rank'] = isset($row['max_rank']) ? $row['max_rank']+1 : 1;
-  }
-  else if (is_numeric($params['rank']))
-  {
-    $insert['rank'] = (int)$params['rank'];
+        $current_rank_of = simple_hash_from_query(
+          $query,
+          'category_id',
+          'max_rank'
+          );
+
+        foreach ($cat_ids as $cat_id)
+        {
+          if ('auto' == $rank_on_category[$cat_id])
+          {
+          $rank_on_category[$cat_id] = $current_rank_of[$cat_id] + 1;
+          }
+        }
+      }
+
+      $inserts = array();
+
+      foreach ($cat_ids as $cat_id)
+      {
+        array_push(
+          $inserts,
+          array(
+            'image_id' => $image_id,
+            'category_id' => $cat_id,
+            'rank' => $rank_on_category[$cat_id],
+            )
+          );
+      }
+
+      mass_inserts(
+        IMAGE_CATEGORY_TABLE,
+        array_keys($inserts[0]),
+        $inserts
+        );
+    }
+  }
+
+  // and now, let's create tag associations
+  if (isset($params['tag_ids']))
+  {
+    set_tags(
+      explode(',', $params['tag_ids']),
+      $image_id
+      );
   }
   
-  mass_inserts(
-    IMAGE_CATEGORY_TABLE,
-    array_keys($insert),
-    array($insert)
-    );
-
   invalidate_user_cache();
 
@@ -1250 +1336 @@
     );
 }
-
 ?>

trunk/tools/piwigo_remote.pl

@@ -11 +11 @@
 GetOptions(
     \%opt,
-    qw/action=s file=s thumbnail=s category_id=i name=s rank=s/
+    qw/action=s file=s thumbnail=s categories=s define=s%/
 );
 
@@ -60 +60 @@
         thumbnail_sum => $thumbnail_sum,
         thumbnail_content => $thumbnail_content,
-        category_id => $opt{category_id},
-        name => $opt{name},
-        rank => defined($opt{rank}) ? $opt{rank} : 'auto',
+        categories => $opt{categories},
     };
+
+    foreach my $key (keys %{ $opt{define} }) {
+        $form->{$key} = $opt{define}{$key};
+    }
 
     my $response = $ua->post(

trunk/ws.php

@@ -177 +177 @@
     'ws_images_add',
     array(
-      'name',
-      'category_id',
-      'file_content',
-      'file_sum',
-      'thumbnail_content',
-      'thumbnail_sum',
-      'rank',
-      ),
-    'POST method only'
+      'file_content' => array(),
+      'file_sum' => array(),
+      'thumbnail_content' => array(),
+      'thumbnail_sum' => array(),
+      'name' => array('default' => null),
+      'author' => array('default' => null),
+      'date_creation' => array('default' => null),
+      'comment' => array('default' => null),
+      'categories' => array('default' => null),
+      'tag_ids' => array('default' => null),
+      'level' => array(
+        'default' => 0,
+        'maxValue' => $conf['available_permission_levels']
+        ),
+      ),
+    'POST method only.
+<br/><b>categories</b> is a string list "category_id[,rank];category_id[,rank]" The rank is optional and is equivalent to "auto" if not given.'
     );