ShareByMailProvider: getAccessList: set both array keys 'public' and 'mail'

ASerbinski · kesselb · commit e6f987de6b7c · 2023-07-24T12:53:53.000+02:00
To preserve prior behaviour where 'public' was set 'true' if there
are any mail recipients. The 'mail' key will be an array of email
recipients.

Signed-off-by: Adam Serbinski &lt;adam@serbinski.com&gt;
diff --git a/apps/sharebymail/lib/ShareByMailProvider.php b/apps/sharebymail/lib/ShareByMailProvider.php
@@ -1223,6 +1223,7 @@ public function getAccessList($nodes, $currentAccess) {
 			));
 		$cursor = $qb->executeQuery();
 
+		$public = $cursor->rowCount() > 0;
 		$mail = [];
 		while ($row = $cursor->fetch()) {
 			if ($currentAccess === false) {
@@ -1236,7 +1237,7 @@ public function getAccessList($nodes, $currentAccess) {
 		}
 		$cursor->closeCursor();
 
-		return ['public' => $mail];
+		return ['public' => $public, 'mail' => $mail];
 	}
 
 	public function getAllShares(): iterable {

Original file line number	Diff line number	Diff line change
`@@ -1223,6 +1223,7 @@ public function getAccessList($nodes, $currentAccess) {`
`1223`	`1223`	`));`
`1224`	`1224`	`$cursor = $qb->executeQuery();`
`1225`	`1225`
	`1226`	`+ $public = $cursor->rowCount() > 0;`
`1226`	`1227`	`$mail = [];`
`1227`	`1228`	`while ($row = $cursor->fetch()) {`
`1228`	`1229`	`if ($currentAccess === false) {`
`@@ -1236,7 +1237,7 @@ public function getAccessList($nodes, $currentAccess) {`
`1236`	`1237`	`}`
`1237`	`1238`	`$cursor->closeCursor();`
`1238`	`1239`
`1239`		`- return ['public' => $mail];`
	`1240`	`+ return ['public' => $public, 'mail' => $mail];`
`1240`	`1241`	`}`
`1241`	`1242`
`1242`	`1243`	`public function getAllShares(): iterable {`