OC-2868: Fix bug: Unshare a file that does not exist any more, no message is shown
authormasensio <masensio@solidgear.es>
Fri, 7 Feb 2014 12:25:53 +0000 (13:25 +0100)
committermasensio <masensio@solidgear.es>
Fri, 7 Feb 2014 12:25:53 +0000 (13:25 +0100)
src/com/owncloud/android/ui/activity/FileDisplayActivity.java
src/com/owncloud/android/ui/preview/PreviewImageActivity.java

index 6728531..e4c853e 100644 (file)
@@ -1350,9 +1350,12 @@ OCFileListFragment.ContainerActivity, FileDetailFragment.ContainerActivity, OnNa
 
     
     private void onUnshareLinkOperationFinish(UnshareLinkOperation operation, RemoteOperationResult result) {
-        if (result.isSuccess() || result.getCode() == ResultCode.SHARE_NOT_FOUND) {
+        if (result.isSuccess()) {
             refreshShowDetails();
             refeshListOfFilesFragment();
+        } else if (result.getCode() == ResultCode.SHARE_NOT_FOUND) {
+            cleanSecondFragment();
+            refeshListOfFilesFragment();
         }
     }
     
@@ -1364,12 +1367,12 @@ OCFileListFragment.ContainerActivity, FileDetailFragment.ContainerActivity, OnNa
                 file = getStorageManager().getFileByPath(file.getRemotePath()); {
                     if (!(details instanceof PreviewMediaFragment || details instanceof PreviewImageFragment)) {
                         showDetails(file);
-                    } else if (details instanceof PreviewMediaFragment) {                        
+                    } else if (details instanceof PreviewMediaFragment) {
                         startMediaPreview(file, 0, false);
                     } 
                 }
                 invalidateOptionsMenu();
-            }
+            } 
         }
     }
     
index 8ef5fc2..e0b1071 100644 (file)
@@ -48,7 +48,9 @@ import com.owncloud.android.files.services.FileUploader.FileUploaderBinder;
 import com.owncloud.android.lib.operations.common.OnRemoteOperationListener;
 import com.owncloud.android.lib.operations.common.RemoteOperation;
 import com.owncloud.android.lib.operations.common.RemoteOperationResult;
+import com.owncloud.android.lib.operations.common.RemoteOperationResult.ResultCode;
 import com.owncloud.android.operations.CreateShareOperation;
+import com.owncloud.android.operations.UnshareLinkOperation;
 import com.owncloud.android.ui.activity.FileActivity;
 import com.owncloud.android.ui.activity.FileDisplayActivity;
 import com.owncloud.android.ui.activity.PinCodeActivity;
@@ -158,9 +160,26 @@ public class PreviewImageActivity extends FileActivity implements FileFragment.C
         if (operation instanceof CreateShareOperation) {
             onCreateShareOperationFinish((CreateShareOperation) operation, result);
             
+        } else if (operation instanceof UnshareLinkOperation) {
+            onUnshareLinkOperationFinish((UnshareLinkOperation) operation, result);
+            
         }
     }
     
+    
+    private void onUnshareLinkOperationFinish(UnshareLinkOperation operation, RemoteOperationResult result) {
+        if (result.isSuccess()) {
+            OCFile file = getStorageManager().getFileByPath(getFile().getRemotePath());
+            if (file != null) {
+                setFile(file);
+                invalidateOptionsMenu();
+            }
+        } else if  (result.getCode() == ResultCode.SHARE_NOT_FOUND) {
+            backToDisplayActivity();
+        }
+            
+    }
+    
     private void onCreateShareOperationFinish(CreateShareOperation operation, RemoteOperationResult result) {
         if (result.isSuccess()) {
             OCFile file = getStorageManager().getFileByPath(getFile().getRemotePath());