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

ASerbinski · ASerbinski · commit 01e32340af1d · 2024-03-21T18:25:31.000-04: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
@@ -1108,6 +1108,7 @@ public function getAccessList($nodes, $currentAccess): array {
 			));
 		$cursor = $qb->executeQuery();
 
+		$public = $cursor->rowCount() > 0;
 		$mail = [];
 		while ($row = $cursor->fetch()) {
 			if ($currentAccess === false) {
@@ -1121,7 +1122,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
`@@ -1108,6 +1108,7 @@ public function getAccessList($nodes, $currentAccess): array {`
`1108`	`1108`	`));`
`1109`	`1109`	`$cursor = $qb->executeQuery();`
`1110`	`1110`
	`1111`	`+ $public = $cursor->rowCount() > 0;`
`1111`	`1112`	`$mail = [];`
`1112`	`1113`	`while ($row = $cursor->fetch()) {`
`1113`	`1114`	`if ($currentAccess === false) {`
`@@ -1121,7 +1122,7 @@ public function getAccessList($nodes, $currentAccess): array {`
`1121`	`1122`	`}`
`1122`	`1123`	`$cursor->closeCursor();`
`1123`	`1124`
`1124`		`- return ['public' => $mail];`
	`1125`	`+ return ['public' => $public, 'mail' => $mail];`
`1125`	`1126`	`}`
`1126`	`1127`
`1127`	`1128`	`public function getAllShares(): iterable {`