ensure a team stays in sharelist when made owner,

just like with users when they transfer ownership.
This means that when removing ownership, the team is
still in the list of sharees.

Signed-off-by: Jos Poortvliet <jospoortvliet@gmail.com>

Author: jospoortvliet

lib/Service/BoardService.php

@@ -646,6 +646,19 @@ public function transferBoardOwnership(int $boardId, string $newOwner, bool $cha
 
 			$this->boardMapper->transferOwnership($previousOwner, $newOwner, $boardId, $newOwnerType);
 
+			// When transferring ownership to a circle, re-add the circle as an explicit ACL
+			// sharee with full rights so it remains visible in the sharing sidebar.
+			// (It was removed above to prevent a duplicate DB entry.)
+			if ($newOwnerType === Acl::PERMISSION_TYPE_CIRCLE) {
+				try {
+					$this->addAcl($boardId, Acl::PERMISSION_TYPE_CIRCLE, $newOwner, true, true, true);
+				} catch (DbException $e) {
+					if ($e->getReason() !== DbException::REASON_UNIQUE_CONSTRAINT_VIOLATION) {
+						throw $e;
+					}
+				}
+			}
+
 			// Card-content remap is only meaningful when transferring to a user, not a circle
 			if ($changeContent && $newOwnerType === Acl::PERMISSION_TYPE_USER) {
 				$this->assignedUsersMapper->remapAssignedUser($boardId, $previousOwner, $newOwner);