From 780d80d7c369c9f3068406d69387126a78936ee5 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Mon, 7 Dec 2015 16:38:49 +0100 Subject: [PATCH] The ajax code path unshares a link share when updating the password In order to not mess up existing shares if the password gets verified we should first fire this validation. --- lib/private/share/share.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lib/private/share/share.php b/lib/private/share/share.php index 70f9a6e8920..8a8da038f3d 100644 --- a/lib/private/share/share.php +++ b/lib/private/share/share.php @@ -764,6 +764,11 @@ class Share extends Constants { $updateExistingShare = false; if (\OC::$server->getAppConfig()->getValue('core', 'shareapi_allow_links', 'yes') == 'yes') { + // IF the password is changed via the old ajax endpoint verify it before deleting the old share + if ($passwordChanged === true) { + self::verifyPassword($shareWith); + } + // when updating a link share // FIXME Don't delete link if we update it if ($checkExists = self::getItems($itemType, $itemSource, self::SHARE_TYPE_LINK, null,