improved error handling

check if table was updated successfully and only then send a notification
mail and return "true".

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

1 files changed, 9 insertions, 3 deletions

diff --git a/apps/files_sharing/lib/Controller/RemoteController.php b/apps/files_sharing/lib/Controller/RemoteController.php
index 7c7a608ff2d..299b6fe4df7 100644
--- a/apps/files_sharing/lib/Controller/RemoteController.php
+++ b/apps/files_sharing/lib/Controller/RemoteController.php
@@ -29,6 +29,7 @@ use OCP\AppFramework\Http\DataResponse;
 use OCP\AppFramework\OCS\OCSForbiddenException;
 use OCP\AppFramework\OCS\OCSNotFoundException;
 use OCP\AppFramework\OCSController;
+use OCP\ILogger;
 use OCP\IRequest;
 
 class RemoteController extends OCSController {
@@ -36,6 +37,9 @@ class RemoteController extends OCSController {
 	/** @var Manager */
 	private $externalManager;
 
+	/** @var ILogger */
+	private $logger;
+
 	/**
 	 * @NoAdminRequired
 	 *
@@ -47,10 +51,12 @@ class RemoteController extends OCSController {
 	 */
 	public function __construct($appName,
 								IRequest $request,
-								Manager $externalManager) {
+								Manager $externalManager,
+								ILogger $logger) {
 		parent::__construct($appName, $request);
 
 		$this->externalManager = $externalManager;
+		$this->logger = $logger;
 	}
 
 	/**
@@ -78,8 +84,8 @@ class RemoteController extends OCSController {
 			return new DataResponse();
 		}
 
-		// Make sure the user has no notification for something that does not exist anymore.
-		$this->externalManager->processNotification($id);
+		$this->logger->error('Could not accept federated share with id: ' . $id,
+			['app' => 'files_sharing']);
 
 		throw new OCSNotFoundException('wrong share ID, share doesn\'t exist.');
 	}