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

ASerbinski · AndyScherzinger · commit 53bfa7cd95cc · 2024-02-27T14:07:01.000+01: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
@@ -1107,6 +1107,7 @@ public function getAccessList($nodes, $currentAccess): array {
 			));
 		$cursor = $qb->executeQuery();
 
+		$public = $cursor->rowCount() > 0;
 		$mail = [];
 		while ($row = $cursor->fetch()) {
 			if ($currentAccess === false) {
@@ -1120,7 +1121,7 @@ public function getAccessList($nodes, $currentAccess): array {
 		}
 		$cursor->closeCursor();
 
-		return ['public' => $mail];
+		return ['public' => $public, 'mail' => $mail];
 	}
 
 	public function getAllShares(): iterable {

Original file line number	Diff line number	Diff line change
`@@ -1107,6 +1107,7 @@ public function getAccessList($nodes, $currentAccess): array {`
`1107`	`1107`	`));`
`1108`	`1108`	`$cursor = $qb->executeQuery();`
`1109`	`1109`
	`1110`	`+ $public = $cursor->rowCount() > 0;`
`1110`	`1111`	`$mail = [];`
`1111`	`1112`	`while ($row = $cursor->fetch()) {`
`1112`	`1113`	`if ($currentAccess === false) {`
`@@ -1120,7 +1121,7 @@ public function getAccessList($nodes, $currentAccess): array {`
`1120`	`1121`	`}`
`1121`	`1122`	`$cursor->closeCursor();`
`1122`	`1123`
`1123`		`- return ['public' => $mail];`
	`1124`	`+ return ['public' => $public, 'mail' => $mail];`
`1124`	`1125`	`}`
`1125`	`1126`
`1126`	`1127`	`public function getAllShares(): iterable {`