Merge pull request #24152 from nextcloud/backport/24098/stable18

[stable18] circleId too short in some request

Author: MorrisJobke

apps/files_sharing/lib/Controller/ShareAPIController.php

@@ -1609,10 +1609,11 @@ private function shareProviderResharingRights(string $userId, IShare $share, $no
 			$hasCircleId = (substr($share->getSharedWith(), -1) === ']');
 			$shareWithStart = ($hasCircleId ? strrpos($share->getSharedWith(), '[') + 1 : 0);
 			$shareWithLength = ($hasCircleId ? -1 : strpos($share->getSharedWith(), ' '));
-			if (is_bool($shareWithLength)) {
-				$shareWithLength = -1;
+			if ($shareWithLength === false) {
+				$sharedWith = substr($share->getSharedWith(), $shareWithStart);
+			} else {
+				$sharedWith = substr($share->getSharedWith(), $shareWithStart, $shareWithLength);
 			}
-			$sharedWith = substr($share->getSharedWith(), $shareWithStart, $shareWithLength);
 			try {
 				$member = \OCA\Circles\Api\v1\Circles::getMember($sharedWith, $userId, 1);
 				if ($member->getLevel() >= 4) {