Merge branch 'master' of https://github.com/biometrics/openbr

Scott Klum
2 parents 0645316a c6cf6ad8
Showing 2 changed files with 11 additions and 12 deletions
openbr/core/bee.cpp
openbr/openbr_plugin.cpp
@@ -99,6 +99,7 @@ void BEE::writeSigset(const QString &amp;sigset, const br::FileList &amp;files, bool ign
         QStringList metadata;
         if (!ignoreMetadata)
             foreach (const QString &key, file.localKeys()) {
+                if ((key == "Index") || (key == "Label")) continue;
                 metadata.append(key+"=\""+QtUtils::toString(file.value(key))+"\"");
             }
         lines.append("\t<biometric-signature name=\"" + file.subject() +"\">");
@@ -132,25 +132,23 @@ QVariant File::parse(const QString &amp;value)
     if (ok) return point;
     const QRectF rect = QtUtils::toRect(value, &ok);
     if (ok) return rect;
-    const float f = value.toFloat(&ok);
-    if (ok) return f;
+
+    /* We assume that if the value starts with '0'
+       then it was probably intended to be a string UID
+       and that it's numerical value is not relevant. */
+    if (!value.startsWith('0') || (value == "0")) {
+        const float f = value.toFloat(&ok);
+        if (ok) return f;
+    }
+
     return value;
 }
 void File::set(const QString &key, const QVariant &value)
 {
     if (key == "Label") {
-        bool ok = false;
         const QString valueString = value.toString();
-
-        /* We assume that if the value starts with '0'
-           then it was probably intended to be a string UID
-           and that it's numerical value is not relevant. */
-        if (value.canConvert(QVariant::Double) &&
-                (!valueString.startsWith('0') || (valueString == "0")))
-            value.toFloat(&ok);
-
-        if (!ok && !Globals->classes.contains(valueString))
+        if (!Globals->classes.contains(valueString))
             Globals->classes.insert(valueString, Globals->classes.size());
     }