X-Git-Url: http://git.linex4red.de/pub/Android/ownCloud.git/blobdiff_plain/5e0f46cc2e9c2a7202430a4aef8ed43e3851802e..refs/heads/sdcard-save:/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java

diff --git a/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java b/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java
index d7beea0a..b19d6c2f 100644
--- a/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java
+++ b/src/com/owncloud/android/operations/CreateShareWithShareeOperation.java
@@ -29,7 +29,6 @@ package com.owncloud.android.operations;
 import com.owncloud.android.datamodel.FileDataStorageManager;
 import com.owncloud.android.datamodel.OCFile;
 import com.owncloud.android.lib.common.OwnCloudClient;
-import com.owncloud.android.lib.common.operations.RemoteOperation;
 import com.owncloud.android.lib.common.operations.RemoteOperationResult;
 import com.owncloud.android.lib.resources.files.FileUtils;
 import com.owncloud.android.lib.resources.shares.CreateRemoteShareOperation;
@@ -39,27 +38,27 @@ import com.owncloud.android.operations.common.SyncOperation;
 
 public class CreateShareWithShareeOperation extends SyncOperation {
 
-    private static final int READ_ONLY = 1;
-
     protected FileDataStorageManager mStorageManager;
 
     private String mPath;
-    private String mTargetName;
-    private boolean mWithGroup;
+    private String mShareeName;
+    private ShareType mShareType;
 
     /**
-     * Constructor
-     * @param path          Full path of the file/folder being shared. Mandatory argument
+     * Constructor.
+     *
+     * @param path          Full path of the file/folder being shared.
+     * @param shareeName    User or group name of the target sharee.
+     * @param shareType     Type of share determines type of sharee; {@link ShareType#USER} and {@link ShareType#GROUP}
+     *                      are the only valid values for the moment.
      */
-    public CreateShareWithShareeOperation(
-            String path,
-            String targetName,
-            boolean withGroup
-    ) {
-
+    public CreateShareWithShareeOperation(String path, String shareeName, ShareType shareType) {
+        if (!ShareType.USER.equals(shareType) && !ShareType.GROUP.equals(shareType)) {
+            throw new IllegalArgumentException("Illegal share type " + shareType);
+        }
         mPath = path;
-        mTargetName = targetName;
-        mWithGroup = withGroup;
+        mShareeName = shareeName;
+        mShareType = shareType;
     }
 
     @Override
@@ -72,14 +71,15 @@ public class CreateShareWithShareeOperation extends SyncOperation {
         if (!result.isSuccess() || result.getData().size() <= 0) {
         */
 
-        RemoteOperation operation = new CreateRemoteShareOperation(
+        CreateRemoteShareOperation operation = new CreateRemoteShareOperation(
                 mPath,
-                (mWithGroup ? ShareType.GROUP : ShareType.USER),
-                mTargetName,
+                mShareType,
+                mShareeName,
                 false,
                 "",
-                READ_ONLY
+                OCShare.DEFAULT_PERMISSION
         );
+        operation.setGetShareDetails(true);
         RemoteOperationResult result = operation.execute(client);
 
         
@@ -100,13 +100,8 @@ public class CreateShareWithShareeOperation extends SyncOperation {
     private void updateData(OCShare share) {
         // Update DB with the response
         share.setPath(mPath);
-        if (mPath.endsWith(FileUtils.PATH_SEPARATOR)) {
-            share.setIsFolder(true);
-        } else {
-            share.setIsFolder(false);
-        }
-        share.setPermissions(READ_ONLY);
-        
+        share.setIsFolder(mPath.endsWith(FileUtils.PATH_SEPARATOR));
+
         getStorageManager().saveShare(share);
         
         // Update OCFile with data from share: ShareByLink  and publicLink