Merge pull request #4270 from owncloud/fix_4253

cancel sharing if some users doesn't have a working encryption set-up.

2 files changed, 14 insertions, 11 deletions

diff --git a/apps/files_encryption/hooks/hooks.php b/apps/files_encryption/hooks/hooks.php
index b2a17f6bca5..741df166b70 100644
--- a/apps/files_encryption/hooks/hooks.php
+++ b/apps/files_encryption/hooks/hooks.php
@@ -238,6 +238,7 @@ class Hooks {
 	 */

 	public static function preShared($params) {

 

+		$l = new \OC_L10N('files_encryption');

 		$users = array();

 		$view = new \OC\Files\View('/public-keys/');

 

@@ -250,21 +251,18 @@ class Hooks {
 				break;

 		}

 

-		$error = false;

+		$notConfigured = array();

 		foreach ($users as $user) {

 			if (!$view->file_exists($user . '.public.key')) {

-				$error = true;

-				break;

+				$notConfigured[] = $user;

 			}

 		}

 

-		if ($error) // Set flag var 'run' to notify emitting

-			// script that hook execution failed

-		{

-			$params['run']->run = false;

+		if (count($notConfigured) > 0) {

+			$params['run'] = false;

+			$params['error'] = $l->t('Following users are not set up for encryption:') . ' ' . join(', ' , $notConfigured);

 		}

-		// TODO: Make sure files_sharing provides user

-		// feedback on failed share

+		

 	}

 

 	/**

diff --git a/apps/files_encryption/tests/share.php b/apps/files_encryption/tests/share.php
index ebf678da78e..5f3d5005090 100755
--- a/apps/files_encryption/tests/share.php
+++ b/apps/files_encryption/tests/share.php
@@ -881,8 +881,13 @@ class Test_Encryption_Share extends \PHPUnit_Framework_TestCase {
 		\OC_FileProxy::$enabled = $proxyStatus;
 
 		// share the file
-		\OCP\Share::shareItem('file', $fileInfo['fileid'], \OCP\Share::SHARE_TYPE_GROUP, \Test_Encryption_Share::TEST_ENCRYPTION_SHARE_GROUP1, OCP\PERMISSION_ALL);
-
+		try {
+			\OCP\Share::shareItem('file', $fileInfo['fileid'], \OCP\Share::SHARE_TYPE_GROUP, \Test_Encryption_Share::TEST_ENCRYPTION_SHARE_GROUP1, OCP\PERMISSION_ALL);
+		} catch (Exception $e) {
+			$this->assertEquals(0, strpos($e->getMessage(), "Following users are not set up for encryption"));
+		}
+		
+		
 		// login as admin
 		\Test_Encryption_Util::loginHelper(\Test_Encryption_Share::TEST_ENCRYPTION_SHARE_USER1);