From a692264fa416fec44d774bd955a06a65c7c0d158 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Schie=C3=9Fle?= <schiessle@owncloud.com>
Date: Tue, 12 Feb 2013 17:00:33 +0100
Subject: [PATCH] add option to keep duplicates in the list of users with
 access to a file, e.g. for the unshare operation we need to know if access
 was granted more than once, for example as group share and as individual
 share

---
 apps/files_encryption/hooks/hooks.php |  6 +++---
 apps/files_encryption/lib/proxy.php   | 11 ++---------
 lib/public/share.php                  | 19 ++++++++++++-------
 3 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/apps/files_encryption/hooks/hooks.php b/apps/files_encryption/hooks/hooks.php
index ffd3e4544f..5e06948aa5 100644
--- a/apps/files_encryption/hooks/hooks.php
+++ b/apps/files_encryption/hooks/hooks.php
@@ -182,7 +182,7 @@ class Hooks {
 		
 		$path = Util::getFilePath($params['itemSource']);
 
-		$shares = \OCP\Share::getUsersSharingFile( $path, 1 );
+		$shares = \OCP\Share::getUsersSharingFile( $path, true );
 		
 		return Crypt::encKeyfileToMultipleUsers($shares, $path);
 		
@@ -194,11 +194,11 @@ class Hooks {
 	public static function preUnshare( $params ) {
 
 		$path = Util::getFilePath($params['itemSource']);
-		$shares = \OCP\Share::getUsersSharingFile( $path, 1 );
+		$shares = \OCP\Share::getUsersSharingFile( $path, true, false );
 		// remove the user from the list from which the file will be unshared
 		unset($shares[$params['shareWith']]);
 		
-		return Crypt::encKeyfileToMultipleUsers($shares, $path );
+		return Crypt::encKeyfileToMultipleUsers(array_unique($shares), $path );
 	}
 	
 	/**
diff --git a/apps/files_encryption/lib/proxy.php b/apps/files_encryption/lib/proxy.php
index 40ac411539..3e4178e8a8 100644
--- a/apps/files_encryption/lib/proxy.php
+++ b/apps/files_encryption/lib/proxy.php
@@ -118,15 +118,8 @@ class Proxy extends \OC_FileProxy {
 // 					$fileOwner = \OC\Files\Filesystem::getOwner( $path );
 					
 					// List everyone sharing the file
-					$shares = \OCP\Share::getUsersSharingFile( $filePath, 1 );
-					
-					$userIds = array();
-					
-					foreach ( $shares as $share ) {
-					
-						$userIds[] = $share['userId'];
-					
-					}
+					//TODO check, is this path always the path to the source file?
+					$userIds = \OCP\Share::getUsersSharingFile( $filePath, true );
 					
 					$publicKeys = Keymanager::getPublicKeys( $rootView, $userIds );
 					
diff --git a/lib/public/share.php b/lib/public/share.php
index 55ff4d4738..68f5e93baa 100644
--- a/lib/public/share.php
+++ b/lib/public/share.php
@@ -145,11 +145,14 @@ class Share {
 	
 	/**
 	* @brief Find which users can access a shared item
-	* @return bool / array
+	* @param $path to the file
+	* @param include owner to the list of users with access to the file
+	* @param remove duplicates in the result
+	* @return array
 	* @note $path needs to be relative to user data dir, e.g. 'file.txt' 
 	*       not '/admin/data/file.txt'
 	*/
-	public static function getUsersSharingFile( $path, $includeOwner = 0 ) {
+	public static function getUsersSharingFile( $path, $includeOwner = false, $removeDuplicates = true ) {
 
 		$user = \OCP\User::getUser();
 		$path_parts = explode(DIRECTORY_SEPARATOR, trim($path, DIRECTORY_SEPARATOR));
@@ -204,14 +207,16 @@ class Share {
 
 		if ( ! empty( $shares ) ) {
 			// Include owner in list of users, if requested
-			if ( $includeOwner == 1 ) {
+			if ( $includeOwner ) {
 				$shares[] = $user;
 			}
-			return array_unique($shares);
-		} else {
-			return false;
 		}
-
+		
+		if ( $removeDuplicates )
+			return array_unique($shares);
+		else {
+			return $shares;
+		}
 	}
 
 	/**