fix: fix UserHomeSetupListener disabling with nested events

Signed-off-by: Robin Appelman <robin@icewind.nl>

Author: icewind1991

apps/files_sharing/lib/Listener/SharesUpdatedListener.php

@@ -61,7 +61,7 @@ public function __construct(
 
 	public function handle(Event $event): void {
 		// don't trigger the on-setup checks if this handler triggers an fs setup
-		$this->homeSetupListener->setDisabled(true);
+		$oldState = $this->homeSetupListener->setDisabled(true);
 
 		if ($event instanceof UserShareAccessUpdatedEvent) {
 			foreach ($event->getUsers() as $user) {
@@ -111,7 +111,7 @@ public function handle(Event $event): void {
 			}
 		}
 
-		$this->homeSetupListener->setDisabled(false);
+		$this->homeSetupListener->setDisabled($oldState);
 	}
 
 	private function markOrRun(IUser $user, callable $callback): void {

apps/files_sharing/lib/Listener/UserHomeSetupListener.php

@@ -30,8 +30,10 @@ public function __construct(
 	) {
 	}
 
-	public function setDisabled(bool $disabled): void {
+	public function setDisabled(bool $disabled): bool {
+		$previous = $this->disabled;
 		$this->disabled = $disabled;
+		return $previous;
 	}
 	public function handle(Event $event): void {
 		if (!$event instanceof UserHomeSetupEvent) {