Changeset 1222 for branches/branch-1_6/tools

Timestamp:

Apr 20, 2006, 11:12:45 PM (19 years ago)

Author:

plg

Message:

bug 339 fixed: crash on metadata synchronization when duplicate keywords.

improvement: tools/metadata.php is nearer of what metadata synchronization
uses (concerning keywords)

File:

• branches/branch-1_6/tools/metadata.php (modified) (2 diffs)

branches/branch-1_6/tools/metadata.php

@@ -29 +29 @@
 echo 'Informations are read from '.$filename.'<br /><br /><br />';
 
+/**
+ * return a cleaned IPTC value
+ *
+ * @param string value
+ * @return string
+ */
+function clean_iptc_value($value)
+{
+  // strip leading zeros (weird Kodak Scanner software)
+  while ( isset($value[0]) and $value[0] == chr(0))
+  {
+    $value = substr($value, 1);
+  }
+  // remove binary nulls
+  $value = str_replace(chr(0x00), ' ', $value);
+
+  return $value;
+}
+
 $iptc_result = array();
 $imginfo = array();
@@ -39 +58 @@
     foreach (array_keys($iptc) as $iptc_key)
     {
-      if (isset($iptc[$iptc_key][0]) and $value = $iptc[$iptc_key][0])
+      if (isset($iptc[$iptc_key][0]))
       {
-        // strip leading zeros (weird Kodak Scanner software)
-        while ($value[0] == chr(0))
+        if ($iptc_key == '2#025')
         {
-          $value = substr($value, 1);
+          $value = implode(
+            ',',
+            array_map(
+              'clean_iptc_value',
+              $iptc[$iptc_key]
+              )
+            );
         }
-        // remove binary nulls
-        $value = str_replace(chr(0x00), ' ', $value);
+        else
+        {
+          $value = clean_iptc_value($iptc[$iptc_key][0]);
+        }
+        
+        $iptc_result[$iptc_key] = $value;
       }
-      $iptc_result[$iptc_key] = $value;
     }
   }