[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [akonadi-google/wip-v0.3] libkgoogle/services: Fix the generator of move-task URL
From:       Dan_Vrátil <dan () progdan ! cz>
Date:       2012-04-01 16:08:36
Message-ID: 20120401160836.86DC3A60A9 () git ! kde ! org
[Download RAW message or body]

Git commit 6c0b10f01e23396e264c872abd21a521dd8022a4 by Dan Vrátil.
Committed on 01/04/2012 at 18:09.
Pushed by dvratil into branch 'wip-v0.3'.

Fix the generator of move-task URL

The bug was causing subtasks to be moved to the top level when modified.

M  +1    -1    libkgoogle/services/tasks.cpp

http://commits.kde.org/akonadi-google/6c0b10f01e23396e264c872abd21a521dd8022a4

diff --git a/libkgoogle/services/tasks.cpp b/libkgoogle/services/tasks.cpp
index 6ec2b90..cf136af 100644
--- a/libkgoogle/services/tasks.cpp
+++ b/libkgoogle/services/tasks.cpp
@@ -183,7 +183,7 @@ QUrl Tasks::removeTaskUrl(const QString& tasklistID, const \
QString& taskID)  
 QUrl Tasks::moveTaskUrl(const QString& tasklistID, const QString& taskID, const \
QString& newParent)  {
-    QString parent = (newParent.isEmpty()) ? "?parent=" + newParent : "";
+    QString parent = (!newParent.isEmpty()) ? "?parent=" + newParent : "";
     return "https://www.googleapis.com/tasks/v1/lists/" + tasklistID + "/tasks/" + \
taskID + "/move" + parent;  }
 


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic