Merge pull request #1991 from nextcloud/backport/1989/stable31

[stable31] update child share permissions

Author: ArtificialOwl

lib/Db/ShareWrapperRequest.php

@@ -93,6 +93,22 @@ public function update(ShareWrapper $shareWrapper): void {
 		$qb->execute();
 	}
 
+	/**
+	 * update permissions and attributes from child
+	 */
+	public function updateChildPermissions(ShareWrapper $shareWrapper): void {
+		$qb = $this->getShareUpdateSql();
+		$shareAttributes = $this->formatShareAttributes($shareWrapper->getAttributes());
+
+		$qb->set('permissions', $qb->createNamedParameter($shareWrapper->getPermissions()))
+			->set('attributes', $qb->createNamedParameter($shareAttributes));
+
+		$qb->limitToShareParent((int)$shareWrapper->getId());
+		$qb->gt('permissions', 0);
+
+		$qb->execute();
+	}
+
 
 	/**
 	 * @param Membership $membership

lib/Service/ShareWrapperService.php

@@ -91,6 +91,10 @@ public function update(ShareWrapper $shareWrapper): void {
 		$this->shareWrapperRequest->update($shareWrapper);
 	}
 
+	public function updateChildPermissions(ShareWrapper $shareWrapper): void {
+		$this->cache->clear('');
+		$this->shareWrapperRequest->updateChildPermissions($shareWrapper);
+	}
 
 	/**
 	 * @param ShareWrapper $shareWrapper

lib/ShareByCircleProvider.php

@@ -212,6 +212,7 @@ public function update(IShare $share): IShare {
 			->setShareNote($share->getNote());
 
 		$this->shareWrapperService->update($wrappedShare);
+		$this->shareWrapperService->updateChildPermissions($wrappedShare);
 
 		return $wrappedShare->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
 	}