Fix logic of the new string parsing

There was an off-by-one on every look-up, making it always find the first ; and returning empty strings all over the place.
PackageKit · Jan 10, 2018 · be3472b · be3472b
1 parent e6e4cf4
commit be3472b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/daemon.cpp b/src/daemon.cpp
@@ -234,7 +234,7 @@ QString Daemon::packageName(const QString &packageID)
 QString Daemon::packageVersion(const QString &packageID)
 {
     QString ret;
-    int start = packageID.indexOf(QLatin1Char(';'));
+    int start = packageID.indexOf(QLatin1Char(';')) + 1;
     if (start == -1) {
         return ret;
     }
@@ -249,11 +249,11 @@ QString Daemon::packageVersion(const QString &packageID)
 QString Daemon::packageArch(const QString &packageID)
 {
     QString ret;
-    int start = packageID.indexOf(QLatin1Char(';'));
+    int start = packageID.indexOf(QLatin1Char(';')) + 1;
     if (start == -1) {
         return ret;
     }
-    start = packageID.indexOf(QLatin1Char(';'), start);
+    start = packageID.indexOf(QLatin1Char(';'), start) + 1;
     if (start == -1) {
         return ret;
     }
@@ -268,15 +268,15 @@ QString Daemon::packageArch(const QString &packageID)
 QString Daemon::packageData(const QString &packageID)
 {
     QString ret;
-    int start = packageID.indexOf(QLatin1Char(';'));
+    int start = packageID.indexOf(QLatin1Char(';')) + 1;
     if (start == -1) {
         return ret;
     }
-    start = packageID.indexOf(QLatin1Char(';'), start);
+    start = packageID.indexOf(QLatin1Char(';'), start) + 1;
     if (start == -1) {
         return ret;
     }
-    start = packageID.indexOf(QLatin1Char(';'), start);
+    start = packageID.indexOf(QLatin1Char(';'), start) + 1;
     if (start == -1) {
         return ret;
     }