improved error messages

Author: Björn Schießle

apps/federatedfilesharing/lib/FederatedShareProvider.php

@@ -171,43 +171,40 @@ public function create(IShare $share) {
 			try {
 				$uidOwner = $remoteShare['owner'] . '@' . $remoteShare['remote'];
 				$shareId = $this->addShareToDB($itemSource, $itemType, $shareWith, $sharedBy, $uidOwner, $permissions, 'tmp_token_' . time());
+				$share->setId($shareId);
 				list($token, $remoteId) = $this->askOwnerToReShare($shareWith, $share, $shareId);
 				// remote share was create successfully if we get a valid token as return
 				$send = is_string($token) && $token !== '';
-				if ($send) {
-					$this->updateSuccessfulReshare($shareId, $token);
-					$this->storeRemoteId($shareId, $remoteId);
-				}
 			} catch (\Exception $e) {
 				// fall back to old re-share behavior if the remote server
 				// doesn't support flat re-shares (was introduced with ownCloud 9.1)
-				$data = $this->getRawShare($shareId);
-				$brokenShare = $this->createShareObject($data);
-				$this->removeShareFromTable($brokenShare);
-				list($shareId, $send) = $this->createFederatedShare($share);
+				$this->removeShareFromTable($share);
+				$this->createFederatedShare($share);
 			}
+			if ($send) {
+				$this->updateSuccessfulReshare($shareId, $token);
+				$this->storeRemoteId($shareId, $remoteId);
+			} else {
+				$this->removeShareFromTable($share);
+				$message_t = $this->l->t('File is already shared with %s', [$shareWith]);
+				throw new \Exception($message_t);
+			}
+
 		} else {
-			list($shareId, $send) = $this->createFederatedShare($share);
+			$this->createFederatedShare($share);
 		}
 
 		$data = $this->getRawShare($shareId);
-		$share = $this->createShareObject($data);
-
-		if ($send === false) {
-			$this->removeShareFromTable($share);
-			$message_t = $this->l->t('Sharing %s failed, could not find %s, maybe the server is currently unreachable.',
-				[$share->getNode()->getName(), $shareWith]);
-			throw new \Exception($message_t);
-		}
-
-		return $share;
+		return $this->createShareObject($data);
 	}
 
 	/**
 	 * create federated share and inform the recipient
 	 *
 	 * @param IShare $share
 	 * @return array
+	 * @throws ShareNotFound
+	 * @throws \Exception
 	 */
 	protected function createFederatedShare(IShare $share) {
 		$token = $this->tokenHandler->generateToken();
@@ -235,7 +232,15 @@ protected function createFederatedShare(IShare $share) {
 			$sharedByFederatedId
 		);
 
-		return [$shareId, $send];
+		if ($send === false) {
+			$data = $this->getRawShare($shareId);
+			$share = $this->createShareObject($data);
+			$this->removeShareFromTable($share);
+			$message_t = $this->l->t('Sharing %s failed, could not find %s, maybe the server is currently unreachable.',
+				[$share->getNode()->getName(), $share->getSharedWith()]);
+			throw new \Exception($message_t);
+		}
+
 	}
 
 	/**

apps/federatedfilesharing/lib/Notifications.php

@@ -272,6 +272,7 @@ protected function getTimestamp() {
 	 * @param string $urlSuffix
 	 * @param array $fields post parameters
 	 * @return array
+	 * @throws \Exception
 	 */
 	protected function tryHttpPostToShareEndpoint($remoteDomain, $urlSuffix, array $fields) {
 		$client = $this->httpClientService->newClient();

apps/federatedfilesharing/lib/RequestHandler.php

@@ -243,7 +243,7 @@ public function reShare($params) {
 					$this->federatedShareProvider->storeRemoteId((int)$result->getId(), $remoteId);
 					return new \OC_OCS_Result(['token' => $result->getToken(), 'remoteId' => $result->getId()]);
 				} catch (\Exception $e) {
-					return new \OC_OCS_Result(null, Http::STATUS_INTERNAL_SERVER_ERROR);
+					return new \OC_OCS_Result(null, Http::STATUS_BAD_REQUEST);
 				}
 			} else {
 				return new \OC_OCS_Result(null, Http::STATUS_FORBIDDEN);