Nah, it's strlen() not length(), also added support for changed usage

in same compound, i.e. /dev/hda moved to /dev/hdb or something.

diff --git a/src/InfoCon/hwdb/update.wml b/src/InfoCon/hwdb/update.wml
index 1672743..3dd8e02 100644 (file)
--- a/src/InfoCon/hwdb/update.wml
+++ b/src/InfoCon/hwdb/update.wml
@@ -109,12 +109,16 @@
        if (pg_NumRows ($sth) > 0) {
          $row = pg_fetch_array ($sth, 0);
          if ($row['compound'] != $compound
-           || (length ($row['usage')) != 0 && $row['usage'] != $usage)) {
+           || (strlen ($row['usage']) != 0 && $row['usage'] != $usage)) {
            $query = sprintf ("UPDATE screw SET endtime = now() WHERE oid = %d", $row['oid']);
            $sth = pg_exec ($dbh, $query) or die_query("Cannot update component history!", $query);
 
            $query = sprintf ("INSERT INTO screw (component,compound,usage,starttime) "
-                            ."VALUES (%d,%d,'%s'now())", $id, $compound, addslashes ($usage));
+                            ."VALUES (%d,%d,'%s',now())", $id, $compound, addslashes ($usage));
+           $sth = pg_exec ($dbh, $query) or die_query("Cannot update component history!", $query);
+         } elseif (strlen ($row['usage']) == 0 && strlen ($usage) != 0) {
+           $query = sprintf ("UPDATE screw SET usage = '%s' WHERE oid = %d",
+                             addslashes ($usage), $row['oid']);
            $sth = pg_exec ($dbh, $query) or die_query("Cannot update component history!", $query);
          }
        } else {