From: David A. Velasco <dvelasco@solidgear.es>
Date: Wed, 21 Nov 2012 15:28:20 +0000 (+0100)
Subject: Fixed bug: renaming a folder could miss-update the storage path of children not-down... 
X-Git-Tag: oc-android-1.4.3~88^2~6
X-Git-Url: http://git.linex4red.de/pub/Android/ownCloud.git/commitdiff_plain/ccb796173b51981988f4cf6f20d2429df2faa2b0?ds=sidebyside;hp=--cc

Fixed bug: renaming a folder could miss-update the storage path of children not-down files if a download file is in the folder
---

ccb796173b51981988f4cf6f20d2429df2faa2b0
diff --git a/src/com/owncloud/android/datamodel/FileDataStorageManager.java b/src/com/owncloud/android/datamodel/FileDataStorageManager.java
index be016555..4ed425e1 100644
--- a/src/com/owncloud/android/datamodel/FileDataStorageManager.java
+++ b/src/com/owncloud/android/datamodel/FileDataStorageManager.java
@@ -533,12 +533,12 @@ public class FileDataStorageManager implements DataStorageManager {
 
             /// 2. prepare a batch of update operations to change all the descendants
             ArrayList<ContentProviderOperation> operations = new ArrayList<ContentProviderOperation>(c.getCount());
-            ContentValues cv = new ContentValues();
             int lengthOfOldPath = dir.getRemotePath().length();
             String defaultSavePath = FileDownloader.getSavePath(mAccount.name);
             int lengthOfOldStoragePath = defaultSavePath.length() + lengthOfOldPath;
             if (c.moveToFirst()) {
                 do {
+                    ContentValues cv = new ContentValues(); // don't take the constructor out of the loop and clear the object
                     OCFile child = createFileInstance(c);
                     cv.put(ProviderTableMeta.FILE_PATH, newPath + child.getRemotePath().substring(lengthOfOldPath));
                     if (child.getStoragePath() != null && child.getStoragePath().startsWith(defaultSavePath)) {