From: David A. Velasco <dvelasco@solidgear.es>
Date: Thu, 29 Oct 2015 15:34:12 +0000 (+0100)
Subject: Let server decide default permissions for new share
X-Git-Tag: oc-android-1.9^2~30^2~3
X-Git-Url: http://git.linex4red.de/pub/Android/ownCloud.git/commitdiff_plain/58559b784ba4e410c5662a8015af4f847c5647c5?ds=inline;hp=--cc

Let server decide default permissions for new share
---

58559b784ba4e410c5662a8015af4f847c5647c5
diff --git a/src/com/owncloud/android/operations/CreateShareViaLinkOperation.java b/src/com/owncloud/android/operations/CreateShareViaLinkOperation.java
index 10af2722..373c57d1 100644
--- a/src/com/owncloud/android/operations/CreateShareViaLinkOperation.java
+++ b/src/com/owncloud/android/operations/CreateShareViaLinkOperation.java
@@ -44,8 +44,6 @@ import com.owncloud.android.operations.common.SyncOperation;
 
 public class CreateShareViaLinkOperation extends SyncOperation {
 
-    private static final int READ_ONLY = 1;
-
     protected FileDataStorageManager mStorageManager;
 
     private String mPath;
@@ -87,7 +85,7 @@ public class CreateShareViaLinkOperation extends SyncOperation {
                     "",
                     false,
                     mPassword,
-                    READ_ONLY
+                    OCShare.DEFAULT_PERMISSION
             );
             result = operation.execute(client);
         }
@@ -144,8 +142,7 @@ public class CreateShareViaLinkOperation extends SyncOperation {
         } else {
             share.setIsFolder(false);
         }
-        share.setPermissions(READ_ONLY);
-        
+
         getStorageManager().saveShare(share);
         
         // Update OCFile with data from share: ShareByLink  and publicLink
diff --git a/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java b/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java
index 9b71695e..b19d6c2f 100644
--- a/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java
+++ b/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java
@@ -38,9 +38,6 @@ import com.owncloud.android.operations.common.SyncOperation;
 
 public class CreateShareWithShareeOperation extends SyncOperation {
 
-    private static final int READ_ONLY = 1;
-    private static final int ALL_PRIVILEGES = 31;
-
     protected FileDataStorageManager mStorageManager;
 
     private String mPath;
@@ -80,7 +77,7 @@ public class CreateShareWithShareeOperation extends SyncOperation {
                 mShareeName,
                 false,
                 "",
-                ALL_PRIVILEGES
+                OCShare.DEFAULT_PERMISSION
         );
         operation.setGetShareDetails(true);
         RemoteOperationResult result = operation.execute(client);
@@ -104,8 +101,7 @@ public class CreateShareWithShareeOperation extends SyncOperation {
         // Update DB with the response
         share.setPath(mPath);
         share.setIsFolder(mPath.endsWith(FileUtils.PATH_SEPARATOR));
-        share.setPermissions(READ_ONLY);
-        
+
         getStorageManager().saveShare(share);
         
         // Update OCFile with data from share: ShareByLink  and publicLink