style(PHP): adjust to updated cs-fixer

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Author: blizzz

lib/AppInfo/Application.php

@@ -148,9 +148,9 @@ public function boot(IBootContext $context): void {
 
 				// All requests that are not authenticated and match against the "/login" route are
 				// redirected to the SAML login endpoint
-				if (!$isCLI &&
-					!$userSession->isLoggedIn() &&
-					($request->getPathInfo() === '/login')) {
+				if (!$isCLI
+					&& !$userSession->isLoggedIn()
+					&& ($request->getPathInfo() === '/login')) {
 					try {
 						$params = $request->getParams();
 					} catch (\LogicException) {

lib/Controller/SAMLController.php

@@ -431,8 +431,8 @@ public function assertionConsumerService(): Http\RedirectResponse {
 	 * @throws Error
 	 */
 	public function singleLogoutService(): Http\RedirectResponse {
-		$isFromGS = ($this->config->getSystemValueBool('gs.enabled', false) &&
-					 $this->config->getSystemValueString('gss.mode', '') === 'master');
+		$isFromGS = ($this->config->getSystemValueBool('gs.enabled', false)
+					 && $this->config->getSystemValueString('gss.mode', '') === 'master');
 
 		// Some IDPs send the SLO request via POST, but OneLogin php-saml only handles GET.
 		// To hack around this issue we copy the request from _POST to _GET.

lib/DavPlugin.php

@@ -35,8 +35,8 @@ public function initialize(Server $server) {
 
 	public function beforeMethod(RequestInterface $request, ResponseInterface $response) {
 		if (
-			$this->config->getAppValue('user_saml', 'type') === 'environment-variable' &&
-			!$this->session->exists('user_saml.samlUserData')
+			$this->config->getAppValue('user_saml', 'type') === 'environment-variable'
+			&& !$this->session->exists('user_saml.samlUserData')
 		) {
 			$uidMapping = $this->samlSettings->get(1)['general-uid_mapping'];
 			if (isset($this->auth[$uidMapping])) {

lib/GroupManager.php

@@ -273,8 +273,8 @@ protected function hasGroupForeignMembers(IGroup $group): bool {
 	 * allowed only for groups owned by the SAML backend.
 	 */
 	protected function mayModifyGroup(?IGroup $group): bool {
-		$isInTransition =
-			$group !== null
+		$isInTransition
+			= $group !== null
 			&& $group->getGID() !== 'admin'
 			&& in_array('Database', $group->getBackendNames())
 			&& $this->isGroupInTransitionList($group->getGID());

tests/unit/UserBackendTest.php

@@ -106,9 +106,9 @@ public function testUpdateAttributesWithoutAttributes() {
 		$user = $this->createMock(IUser::class);
 
 		$this->config->method('getAppValue')
-			->willReturnCallback(fn (string $appId, string $key, string $default) =>
+			->willReturnCallback(fn (string $appId, string $key, string $default)
 				// Unused parameters are intentionally kept for clarity
-				$default);
+				=> $default);
 
 		$this->userManager
 			->expects($this->once())
@@ -138,9 +138,9 @@ public function testUpdateAttributesWithoutValidUser() {
 		$this->getMockedBuilder();
 
 		$this->config->method('getAppValue')
-			->willReturnCallback(fn (string $appId, string $key, string $default) =>
+			->willReturnCallback(fn (string $appId, string $key, string $default)
 				// Unused parameters are intentionally kept for clarity
-				$default);
+				=> $default);
 
 		$this->userManager
 			->expects($this->once())
@@ -227,9 +227,9 @@ public function testUpdateAttributesQuotaDefaultFallback() {
 		$attributes = ['email' => 'new@example.com', 'displayname' => 'New Displayname', 'quota' => ''];
 
 		$this->config->method('getAppValue')
-			->willReturnCallback(fn (string $appId, string $key, string $default) =>
+			->willReturnCallback(fn (string $appId, string $key, string $default)
 				// Unused $appId parameter is intentionally kept for clarity
-				match ($key) {
+				=> match ($key) {
 					'saml-attribute-mapping-email_mapping' => 'email',
 					'saml-attribute-mapping-displayName_mapping' => 'displayname',
 					'saml-attribute-mapping-quota_mapping' => 'quota',