nextcloud / server

apps/systemtags/lib/AppInfo/Application.php

Indentation +44 added lines, -44 removed lines

@@ -35,56 +35,56 @@
 use Symfony\Component\EventDispatcher\EventDispatcher;
 
 class Application extends App implements IBootstrap {
-	public const APP_ID = 'systemtags';
+    public const APP_ID = 'systemtags';
 
-	public function __construct() {
-		parent::__construct(self::APP_ID);
-	}
+    public function __construct() {
+        parent::__construct(self::APP_ID);
+    }
 
-	public function register(IRegistrationContext $context): void {
-	}
+    public function register(IRegistrationContext $context): void {
+    }
 
-	public function boot(IBootContext $context): void {
-		$context->injectFn(function (EventDispatcher $dispatcher) use ($context) {
-			/*
+    public function boot(IBootContext $context): void {
+        $context->injectFn(function (EventDispatcher $dispatcher) use ($context) {
+            /*
 			 * @todo move the OCP events and then move the registration to `register`
 			 */
-			$dispatcher->addListener(
-				'OCA\Files::loadAdditionalScripts',
-				function () {
-					// FIXME: no public API for these ?
-					\OCP\Util::addScript('dist/systemtags');
-					\OCP\Util::addScript(self::APP_ID, 'systemtags');
-				}
-			);
+            $dispatcher->addListener(
+                'OCA\Files::loadAdditionalScripts',
+                function () {
+                    // FIXME: no public API for these ?
+                    \OCP\Util::addScript('dist/systemtags');
+                    \OCP\Util::addScript(self::APP_ID, 'systemtags');
+                }
+            );
 
-			$managerListener = function (ManagerEvent $event) use ($context) {
-				/** @var \OCA\SystemTags\Activity\Listener $listener */
-				$listener = $context->getServerContainer()->query(Listener::class);
-				$listener->event($event);
-			};
-			$dispatcher->addListener(ManagerEvent::EVENT_CREATE, $managerListener);
-			$dispatcher->addListener(ManagerEvent::EVENT_DELETE, $managerListener);
-			$dispatcher->addListener(ManagerEvent::EVENT_UPDATE, $managerListener);
+            $managerListener = function (ManagerEvent $event) use ($context) {
+                /** @var \OCA\SystemTags\Activity\Listener $listener */
+                $listener = $context->getServerContainer()->query(Listener::class);
+                $listener->event($event);
+            };
+            $dispatcher->addListener(ManagerEvent::EVENT_CREATE, $managerListener);
+            $dispatcher->addListener(ManagerEvent::EVENT_DELETE, $managerListener);
+            $dispatcher->addListener(ManagerEvent::EVENT_UPDATE, $managerListener);
 
-			$mapperListener = function (MapperEvent $event) use ($context) {
-				/** @var \OCA\SystemTags\Activity\Listener $listener */
-				$listener = $context->getServerContainer()->query(Listener::class);
-				$listener->mapperEvent($event);
-			};
-			$dispatcher->addListener(MapperEvent::EVENT_ASSIGN, $mapperListener);
-			$dispatcher->addListener(MapperEvent::EVENT_UNASSIGN, $mapperListener);
-		});
+            $mapperListener = function (MapperEvent $event) use ($context) {
+                /** @var \OCA\SystemTags\Activity\Listener $listener */
+                $listener = $context->getServerContainer()->query(Listener::class);
+                $listener->mapperEvent($event);
+            };
+            $dispatcher->addListener(MapperEvent::EVENT_ASSIGN, $mapperListener);
+            $dispatcher->addListener(MapperEvent::EVENT_UNASSIGN, $mapperListener);
+        });
 
-		\OCA\Files\App::getNavigationManager()->add(function () {
-			$l = \OC::$server->getL10N(self::APP_ID);
-			return [
-				'id' => 'systemtagsfilter',
-				'appname' => self::APP_ID,
-				'script' => 'list.php',
-				'order' => 25,
-				'name' => $l->t('Tags'),
-			];
-		});
-	}
+        \OCA\Files\App::getNavigationManager()->add(function () {
+            $l = \OC::$server->getL10N(self::APP_ID);
+            return [
+                'id' => 'systemtagsfilter',
+                'appname' => self::APP_ID,
+                'script' => 'list.php',
+                'order' => 25,
+                'name' => $l->t('Tags'),
+            ];
+        });
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -45,20 +45,20 @@ 
 	}
 
 	public function boot(IBootContext $context): void {
-		$context->injectFn(function (EventDispatcher $dispatcher) use ($context) {
+		$context->injectFn(function(EventDispatcher $dispatcher) use ($context) {
 			/*
 			 * @todo move the OCP events and then move the registration to `register`
 			 */
 			$dispatcher->addListener(
 				'OCA\Files::loadAdditionalScripts',
-				function () {
+				function() {
 					// FIXME: no public API for these ?
 					\OCP\Util::addScript('dist/systemtags');
 					\OCP\Util::addScript(self::APP_ID, 'systemtags');
 				}
 			);
 
-			$managerListener = function (ManagerEvent $event) use ($context) {
+			$managerListener = function(ManagerEvent $event) use ($context) {
 				/** @var \OCA\SystemTags\Activity\Listener $listener */
 				$listener = $context->getServerContainer()->query(Listener::class);
 				$listener->event($event);
@@ -67,7 +67,7 @@ 
 			$dispatcher->addListener(ManagerEvent::EVENT_DELETE, $managerListener);
 			$dispatcher->addListener(ManagerEvent::EVENT_UPDATE, $managerListener);
 
-			$mapperListener = function (MapperEvent $event) use ($context) {
+			$mapperListener = function(MapperEvent $event) use ($context) {
 				/** @var \OCA\SystemTags\Activity\Listener $listener */
 				$listener = $context->getServerContainer()->query(Listener::class);
 				$listener->mapperEvent($event);
@@ -76,7 +76,7 @@ 
 			$dispatcher->addListener(MapperEvent::EVENT_UNASSIGN, $mapperListener);
 		});
 
-		\OCA\Files\App::getNavigationManager()->add(function () {
+		\OCA\Files\App::getNavigationManager()->add(function() {
 			$l = \OC::$server->getL10N(self::APP_ID);
 			return [
 				'id' => 'systemtagsfilter',

apps/user_ldap/lib/AppInfo/Application.php

Indentation +82 added lines, -82 removed lines

@@ -52,86 +52,86 @@
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 
 class Application extends App implements IBootstrap {
-	public function __construct() {
-		parent::__construct('user_ldap');
-		$container = $this->getContainer();
-
-		/**
-		 * Controller
-		 */
-		$container->registerService('RenewPasswordController', function (IAppContainer $appContainer) {
-			/** @var IServerContainer $server */
-			$server = $appContainer->get(IServerContainer::class);
-
-			return new RenewPasswordController(
-				$appContainer->get('AppName'),
-				$server->getRequest(),
-				$appContainer->get('UserManager'),
-				$server->getConfig(),
-				$appContainer->get(IL10N::class),
-				$appContainer->get('Session'),
-				$server->getURLGenerator()
-			);
-		});
-
-		$container->registerService(ILDAPWrapper::class, function () {
-			return new LDAP();
-		});
-	}
-
-	public function register(IRegistrationContext $context): void {
-	}
-
-	public function boot(IBootContext $context): void {
-		$context->injectFn(function (IConfig $config,
-									 INotificationManager $notificationManager,
-									 IUserSession $userSession,
-									 IAppContainer $appContainer,
-									 EventDispatcherInterface $dispatcher,
-									 IGroupManager $groupManager) {
-			$helper = new Helper($config);
-			$configPrefixes = $helper->getServerConfigurationPrefixes(true);
-			if (count($configPrefixes) > 0) {
-				$ldapWrapper = new LDAP();
-
-				$notificationManager->registerNotifierService(Notifier::class);
-
-				$userPluginManager = $appContainer->get(UserPluginManager::class);
-				$groupPluginManager = $appContainer->get(GroupPluginManager::class);
-
-				$userBackend = new User_Proxy(
-					$configPrefixes, $ldapWrapper, $config, $notificationManager, $userSession, $userPluginManager
-				);
-				$groupBackend = new Group_Proxy($configPrefixes, $ldapWrapper, $groupPluginManager);
-				// register user backend
-				\OC_User::useBackend($userBackend);
-
-				// Hook to allow plugins to work on registered backends
-				$dispatcher->dispatch('OCA\\User_LDAP\\User\\User::postLDAPBackendAdded');
-
-				$groupManager->addBackend($groupBackend);
-			}
-		});
-
-		$context->injectFn(Closure::fromCallable([$this, 'registerBackendDependents']));
-
-		\OCP\Util::connectHook(
-			'\OCA\Files_Sharing\API\Server2Server',
-			'preLoginNameUsedAsUserName',
-			'\OCA\User_LDAP\Helper',
-			'loginName2UserName'
-		);
-	}
-
-	private function registerBackendDependents(IAppContainer $appContainer, EventDispatcherInterface $dispatcher) {
-		$dispatcher->addListener(
-			'OCA\\Files_External::loadAdditionalBackends',
-			function () use ($appContainer) {
-				$storagesBackendService = $appContainer->get(BackendService::class);
-				$storagesBackendService->registerConfigHandler('home', function () use ($appContainer) {
-					return $appContainer->get(ExtStorageConfigHandler::class);
-				});
-			}
-		);
-	}
+    public function __construct() {
+        parent::__construct('user_ldap');
+        $container = $this->getContainer();
+
+        /**
+         * Controller
+         */
+        $container->registerService('RenewPasswordController', function (IAppContainer $appContainer) {
+            /** @var IServerContainer $server */
+            $server = $appContainer->get(IServerContainer::class);
+
+            return new RenewPasswordController(
+                $appContainer->get('AppName'),
+                $server->getRequest(),
+                $appContainer->get('UserManager'),
+                $server->getConfig(),
+                $appContainer->get(IL10N::class),
+                $appContainer->get('Session'),
+                $server->getURLGenerator()
+            );
+        });
+
+        $container->registerService(ILDAPWrapper::class, function () {
+            return new LDAP();
+        });
+    }
+
+    public function register(IRegistrationContext $context): void {
+    }
+
+    public function boot(IBootContext $context): void {
+        $context->injectFn(function (IConfig $config,
+                                        INotificationManager $notificationManager,
+                                        IUserSession $userSession,
+                                        IAppContainer $appContainer,
+                                        EventDispatcherInterface $dispatcher,
+                                        IGroupManager $groupManager) {
+            $helper = new Helper($config);
+            $configPrefixes = $helper->getServerConfigurationPrefixes(true);
+            if (count($configPrefixes) > 0) {
+                $ldapWrapper = new LDAP();
+
+                $notificationManager->registerNotifierService(Notifier::class);
+
+                $userPluginManager = $appContainer->get(UserPluginManager::class);
+                $groupPluginManager = $appContainer->get(GroupPluginManager::class);
+
+                $userBackend = new User_Proxy(
+                    $configPrefixes, $ldapWrapper, $config, $notificationManager, $userSession, $userPluginManager
+                );
+                $groupBackend = new Group_Proxy($configPrefixes, $ldapWrapper, $groupPluginManager);
+                // register user backend
+                \OC_User::useBackend($userBackend);
+
+                // Hook to allow plugins to work on registered backends
+                $dispatcher->dispatch('OCA\\User_LDAP\\User\\User::postLDAPBackendAdded');
+
+                $groupManager->addBackend($groupBackend);
+            }
+        });
+
+        $context->injectFn(Closure::fromCallable([$this, 'registerBackendDependents']));
+
+        \OCP\Util::connectHook(
+            '\OCA\Files_Sharing\API\Server2Server',
+            'preLoginNameUsedAsUserName',
+            '\OCA\User_LDAP\Helper',
+            'loginName2UserName'
+        );
+    }
+
+    private function registerBackendDependents(IAppContainer $appContainer, EventDispatcherInterface $dispatcher) {
+        $dispatcher->addListener(
+            'OCA\\Files_External::loadAdditionalBackends',
+            function () use ($appContainer) {
+                $storagesBackendService = $appContainer->get(BackendService::class);
+                $storagesBackendService->registerConfigHandler('home', function () use ($appContainer) {
+                    return $appContainer->get(ExtStorageConfigHandler::class);
+                });
+            }
+        );
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -59,7 +59,7 @@ 
 		/**
 		 * Controller
 		 */
-		$container->registerService('RenewPasswordController', function (IAppContainer $appContainer) {
+		$container->registerService('RenewPasswordController', function(IAppContainer $appContainer) {
 			/** @var IServerContainer $server */
 			$server = $appContainer->get(IServerContainer::class);
 
@@ -74,7 +74,7 @@ 
 			);
 		});
 
-		$container->registerService(ILDAPWrapper::class, function () {
+		$container->registerService(ILDAPWrapper::class, function() {
 			return new LDAP();
 		});
 	}
@@ -83,7 +83,7 @@ 
 	}
 
 	public function boot(IBootContext $context): void {
-		$context->injectFn(function (IConfig $config,
+		$context->injectFn(function(IConfig $config,
 									 INotificationManager $notificationManager,
 									 IUserSession $userSession,
 									 IAppContainer $appContainer,
@@ -126,9 +126,9 @@ 
 	private function registerBackendDependents(IAppContainer $appContainer, EventDispatcherInterface $dispatcher) {
 		$dispatcher->addListener(
 			'OCA\\Files_External::loadAdditionalBackends',
-			function () use ($appContainer) {
+			function() use ($appContainer) {
 				$storagesBackendService = $appContainer->get(BackendService::class);
-				$storagesBackendService->registerConfigHandler('home', function () use ($appContainer) {
+				$storagesBackendService->registerConfigHandler('home', function() use ($appContainer) {
 					return $appContainer->get(ExtStorageConfigHandler::class);
 				});
 			}

apps/lookup_server_connector/lib/AppInfo/Application.php

Indentation +22 added lines, -22 removed lines

@@ -40,31 +40,31 @@
 use Symfony\Component\EventDispatcher\GenericEvent;
 
 class Application extends App implements IBootstrap {
-	public const APP_ID = 'lookup_server_connector';
+    public const APP_ID = 'lookup_server_connector';
 
-	public function __construct() {
-		parent::__construct(self::APP_ID);
-	}
+    public function __construct() {
+        parent::__construct(self::APP_ID);
+    }
 
-	public function register(IRegistrationContext $context): void {
-	}
+    public function register(IRegistrationContext $context): void {
+    }
 
-	public function boot(IBootContext $context): void {
-		$context->injectFn(Closure::fromCallable([$this, 'registerEventListeners']));
-	}
+    public function boot(IBootContext $context): void {
+        $context->injectFn(Closure::fromCallable([$this, 'registerEventListeners']));
+    }
 
-	/**
-	 * @todo move the OCP events and then move the registration to `register`
-	 */
-	private function registerEventListeners(IEventDispatcher $dispatcher,
-											IAppContainer $appContainer): void {
-		$dispatcher->addListener('OC\AccountManager::userUpdated', function (GenericEvent $event) use ($appContainer) {
-			/** @var IUser $user */
-			$user = $event->getSubject();
+    /**
+     * @todo move the OCP events and then move the registration to `register`
+     */
+    private function registerEventListeners(IEventDispatcher $dispatcher,
+                                            IAppContainer $appContainer): void {
+        $dispatcher->addListener('OC\AccountManager::userUpdated', function (GenericEvent $event) use ($appContainer) {
+            /** @var IUser $user */
+            $user = $event->getSubject();
 
-			/** @var UpdateLookupServer $updateLookupServer */
-			$updateLookupServer = $appContainer->get(UpdateLookupServer::class);
-			$updateLookupServer->userUpdated($user);
-		});
-	}
+            /** @var UpdateLookupServer $updateLookupServer */
+            $updateLookupServer = $appContainer->get(UpdateLookupServer::class);
+            $updateLookupServer->userUpdated($user);
+        });
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 	 */
 	private function registerEventListeners(IEventDispatcher $dispatcher,
 											IAppContainer $appContainer): void {
-		$dispatcher->addListener('OC\AccountManager::userUpdated', function (GenericEvent $event) use ($appContainer) {
+		$dispatcher->addListener('OC\AccountManager::userUpdated', function(GenericEvent $event) use ($appContainer) {
 			/** @var IUser $user */
 			$user = $event->getSubject();
 

apps/admin_audit/lib/AppInfo/Application.php

Indentation +204 added lines, -204 removed lines

@@ -68,215 +68,215 @@
 
 class Application extends App implements IBootstrap {
 
-	/** @var ILogger */
-	protected $logger;
+    /** @var ILogger */
+    protected $logger;
 
-	public function __construct() {
-		parent::__construct('admin_audit');
-	}
+    public function __construct() {
+        parent::__construct('admin_audit');
+    }
 
-	public function register(IRegistrationContext $context): void {
-	}
+    public function register(IRegistrationContext $context): void {
+    }
 
-	public function boot(IBootContext $context): void {
-		/** @var ILogger $logger */
-		$logger = $context->injectFn(
-			Closure::fromCallable([$this, 'getLogger'])
-		);
+    public function boot(IBootContext $context): void {
+        /** @var ILogger $logger */
+        $logger = $context->injectFn(
+            Closure::fromCallable([$this, 'getLogger'])
+        );
 
-		/*
+        /*
 		 * TODO: once the hooks are migrated to lazy events, this should be done
 		 *       in \OCA\AdminAudit\AppInfo\Application::register
 		 */
-		$this->registerHooks($logger, $context->getServerContainer());
-	}
-
-	private function getLogger(IConfig $config,
-							   ILogger $logger,
-							   ILogFactory $logFactory): ILogger {
-		$default = $config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/audit.log';
-		$logFile = $config->getAppValue('admin_audit', 'logfile', $default);
-
-		if ($logFile === null) {
-			return $logger;
-		}
-		return $logFactory->getCustomLogger($logFile);
-	}
-
-	/**
-	 * Register hooks in order to log them
-	 */
-	private function registerHooks(ILogger $logger,
-									 IServerContainer $serverContainer) {
-		$this->userManagementHooks($logger, $serverContainer->get(IUserSession::class));
-		$this->groupHooks($logger, $serverContainer->get(IGroupManager::class));
-		$this->authHooks($logger);
-
-		/** @var EventDispatcherInterface $eventDispatcher */
-		$eventDispatcher = $serverContainer->get(EventDispatcherInterface::class);
-		$this->consoleHooks($logger, $eventDispatcher);
-		$this->appHooks($logger, $eventDispatcher);
-
-		$this->sharingHooks($logger);
-
-		$this->fileHooks($logger, $eventDispatcher);
-		$this->trashbinHooks($logger);
-		$this->versionsHooks($logger);
-
-		$this->securityHooks($logger, $eventDispatcher);
-	}
-
-	private function userManagementHooks(ILogger $logger,
-										 IUserSession $userSession) {
-		$userActions = new UserManagement($logger);
-
-		Util::connectHook('OC_User', 'post_createUser', $userActions, 'create');
-		Util::connectHook('OC_User', 'post_deleteUser', $userActions, 'delete');
-		Util::connectHook('OC_User', 'changeUser', $userActions, 'change');
-
-		$userSession->listen('\OC\User', 'postSetPassword', [$userActions, 'setPassword']);
-		$userSession->listen('\OC\User', 'assignedUserId', [$userActions, 'assign']);
-		$userSession->listen('\OC\User', 'postUnassignedUserId', [$userActions, 'unassign']);
-	}
-
-	private function groupHooks(ILogger $logger,
-								IGroupManager $groupManager) {
-		$groupActions = new GroupManagement($logger);
-
-		/** @var IGroupManager|Manager $groupManager */
-		$groupManager->listen('\OC\Group', 'postRemoveUser', [$groupActions, 'removeUser']);
-		$groupManager->listen('\OC\Group', 'postAddUser', [$groupActions, 'addUser']);
-		$groupManager->listen('\OC\Group', 'postDelete', [$groupActions, 'deleteGroup']);
-		$groupManager->listen('\OC\Group', 'postCreate', [$groupActions, 'createGroup']);
-	}
-
-	private function sharingHooks(ILogger $logger) {
-		$shareActions = new Sharing($logger);
-
-		Util::connectHook(Share::class, 'post_shared', $shareActions, 'shared');
-		Util::connectHook(Share::class, 'post_unshare', $shareActions, 'unshare');
-		Util::connectHook(Share::class, 'post_unshareFromSelf', $shareActions, 'unshare');
-		Util::connectHook(Share::class, 'post_update_permissions', $shareActions, 'updatePermissions');
-		Util::connectHook(Share::class, 'post_update_password', $shareActions, 'updatePassword');
-		Util::connectHook(Share::class, 'post_set_expiration_date', $shareActions, 'updateExpirationDate');
-		Util::connectHook(Share::class, 'share_link_access', $shareActions, 'shareAccessed');
-	}
-
-	private function authHooks(ILogger $logger) {
-		$authActions = new Auth($logger);
-
-		Util::connectHook('OC_User', 'pre_login', $authActions, 'loginAttempt');
-		Util::connectHook('OC_User', 'post_login', $authActions, 'loginSuccessful');
-		Util::connectHook('OC_User', 'logout', $authActions, 'logout');
-	}
-
-	private function appHooks(ILogger $logger,
-							  EventDispatcherInterface $eventDispatcher) {
-		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE, function (ManagerEvent $event) use ($logger) {
-			$appActions = new AppManagement($logger);
-			$appActions->enableApp($event->getAppID());
-		});
-		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE_FOR_GROUPS, function (ManagerEvent $event) use ($logger) {
-			$appActions = new AppManagement($logger);
-			$appActions->enableAppForGroups($event->getAppID(), $event->getGroups());
-		});
-		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_DISABLE, function (ManagerEvent $event) use ($logger) {
-			$appActions = new AppManagement($logger);
-			$appActions->disableApp($event->getAppID());
-		});
-	}
-
-	private function consoleHooks(ILogger $logger,
-								  EventDispatcherInterface $eventDispatcher) {
-		$eventDispatcher->addListener(ConsoleEvent::EVENT_RUN, function (ConsoleEvent $event) use ($logger) {
-			$appActions = new Console($logger);
-			$appActions->runCommand($event->getArguments());
-		});
-	}
-
-	private function fileHooks(ILogger $logger,
-							   EventDispatcherInterface $eventDispatcher) {
-		$fileActions = new Files($logger);
-		$eventDispatcher->addListener(
-			IPreview::EVENT,
-			function (GenericEvent $event) use ($fileActions) {
-				/** @var File $file */
-				$file = $event->getSubject();
-				$fileActions->preview([
-					'path' => mb_substr($file->getInternalPath(), 5),
-					'width' => $event->getArguments()['width'],
-					'height' => $event->getArguments()['height'],
-					'crop' => $event->getArguments()['crop'],
-					'mode' => $event->getArguments()['mode']
-				]);
-			}
-		);
-
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_post_rename,
-			$fileActions,
-			'rename'
-		);
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_post_create,
-			$fileActions,
-			'create'
-		);
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_post_copy,
-			$fileActions,
-			'copy'
-		);
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_post_write,
-			$fileActions,
-			'write'
-		);
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_post_update,
-			$fileActions,
-			'update'
-		);
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_read,
-			$fileActions,
-			'read'
-		);
-		Util::connectHook(
-			Filesystem::CLASSNAME,
-			Filesystem::signal_delete,
-			$fileActions,
-			'delete'
-		);
-	}
-
-	private function versionsHooks(ILogger $logger) {
-		$versionsActions = new Versions($logger);
-		Util::connectHook('\OCP\Versions', 'rollback', $versionsActions, 'rollback');
-		Util::connectHook('\OCP\Versions', 'delete', $versionsActions, 'delete');
-	}
-
-	private function trashbinHooks(ILogger $logger) {
-		$trashActions = new Trashbin($logger);
-		Util::connectHook('\OCP\Trashbin', 'preDelete', $trashActions, 'delete');
-		Util::connectHook('\OCA\Files_Trashbin\Trashbin', 'post_restore', $trashActions, 'restore');
-	}
-
-	private function securityHooks(ILogger $logger,
-								   EventDispatcherInterface $eventDispatcher) {
-		$eventDispatcher->addListener(IProvider::EVENT_SUCCESS, function (GenericEvent $event) use ($logger) {
-			$security = new Security($logger);
-			$security->twofactorSuccess($event->getSubject(), $event->getArguments());
-		});
-		$eventDispatcher->addListener(IProvider::EVENT_FAILED, function (GenericEvent $event) use ($logger) {
-			$security = new Security($logger);
-			$security->twofactorFailed($event->getSubject(), $event->getArguments());
-		});
-	}
+        $this->registerHooks($logger, $context->getServerContainer());
+    }
+
+    private function getLogger(IConfig $config,
+                                ILogger $logger,
+                                ILogFactory $logFactory): ILogger {
+        $default = $config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/audit.log';
+        $logFile = $config->getAppValue('admin_audit', 'logfile', $default);
+
+        if ($logFile === null) {
+            return $logger;
+        }
+        return $logFactory->getCustomLogger($logFile);
+    }
+
+    /**
+     * Register hooks in order to log them
+     */
+    private function registerHooks(ILogger $logger,
+                                        IServerContainer $serverContainer) {
+        $this->userManagementHooks($logger, $serverContainer->get(IUserSession::class));
+        $this->groupHooks($logger, $serverContainer->get(IGroupManager::class));
+        $this->authHooks($logger);
+
+        /** @var EventDispatcherInterface $eventDispatcher */
+        $eventDispatcher = $serverContainer->get(EventDispatcherInterface::class);
+        $this->consoleHooks($logger, $eventDispatcher);
+        $this->appHooks($logger, $eventDispatcher);
+
+        $this->sharingHooks($logger);
+
+        $this->fileHooks($logger, $eventDispatcher);
+        $this->trashbinHooks($logger);
+        $this->versionsHooks($logger);
+
+        $this->securityHooks($logger, $eventDispatcher);
+    }
+
+    private function userManagementHooks(ILogger $logger,
+                                            IUserSession $userSession) {
+        $userActions = new UserManagement($logger);
+
+        Util::connectHook('OC_User', 'post_createUser', $userActions, 'create');
+        Util::connectHook('OC_User', 'post_deleteUser', $userActions, 'delete');
+        Util::connectHook('OC_User', 'changeUser', $userActions, 'change');
+
+        $userSession->listen('\OC\User', 'postSetPassword', [$userActions, 'setPassword']);
+        $userSession->listen('\OC\User', 'assignedUserId', [$userActions, 'assign']);
+        $userSession->listen('\OC\User', 'postUnassignedUserId', [$userActions, 'unassign']);
+    }
+
+    private function groupHooks(ILogger $logger,
+                                IGroupManager $groupManager) {
+        $groupActions = new GroupManagement($logger);
+
+        /** @var IGroupManager|Manager $groupManager */
+        $groupManager->listen('\OC\Group', 'postRemoveUser', [$groupActions, 'removeUser']);
+        $groupManager->listen('\OC\Group', 'postAddUser', [$groupActions, 'addUser']);
+        $groupManager->listen('\OC\Group', 'postDelete', [$groupActions, 'deleteGroup']);
+        $groupManager->listen('\OC\Group', 'postCreate', [$groupActions, 'createGroup']);
+    }
+
+    private function sharingHooks(ILogger $logger) {
+        $shareActions = new Sharing($logger);
+
+        Util::connectHook(Share::class, 'post_shared', $shareActions, 'shared');
+        Util::connectHook(Share::class, 'post_unshare', $shareActions, 'unshare');
+        Util::connectHook(Share::class, 'post_unshareFromSelf', $shareActions, 'unshare');
+        Util::connectHook(Share::class, 'post_update_permissions', $shareActions, 'updatePermissions');
+        Util::connectHook(Share::class, 'post_update_password', $shareActions, 'updatePassword');
+        Util::connectHook(Share::class, 'post_set_expiration_date', $shareActions, 'updateExpirationDate');
+        Util::connectHook(Share::class, 'share_link_access', $shareActions, 'shareAccessed');
+    }
+
+    private function authHooks(ILogger $logger) {
+        $authActions = new Auth($logger);
+
+        Util::connectHook('OC_User', 'pre_login', $authActions, 'loginAttempt');
+        Util::connectHook('OC_User', 'post_login', $authActions, 'loginSuccessful');
+        Util::connectHook('OC_User', 'logout', $authActions, 'logout');
+    }
+
+    private function appHooks(ILogger $logger,
+                                EventDispatcherInterface $eventDispatcher) {
+        $eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE, function (ManagerEvent $event) use ($logger) {
+            $appActions = new AppManagement($logger);
+            $appActions->enableApp($event->getAppID());
+        });
+        $eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE_FOR_GROUPS, function (ManagerEvent $event) use ($logger) {
+            $appActions = new AppManagement($logger);
+            $appActions->enableAppForGroups($event->getAppID(), $event->getGroups());
+        });
+        $eventDispatcher->addListener(ManagerEvent::EVENT_APP_DISABLE, function (ManagerEvent $event) use ($logger) {
+            $appActions = new AppManagement($logger);
+            $appActions->disableApp($event->getAppID());
+        });
+    }
+
+    private function consoleHooks(ILogger $logger,
+                                    EventDispatcherInterface $eventDispatcher) {
+        $eventDispatcher->addListener(ConsoleEvent::EVENT_RUN, function (ConsoleEvent $event) use ($logger) {
+            $appActions = new Console($logger);
+            $appActions->runCommand($event->getArguments());
+        });
+    }
+
+    private function fileHooks(ILogger $logger,
+                                EventDispatcherInterface $eventDispatcher) {
+        $fileActions = new Files($logger);
+        $eventDispatcher->addListener(
+            IPreview::EVENT,
+            function (GenericEvent $event) use ($fileActions) {
+                /** @var File $file */
+                $file = $event->getSubject();
+                $fileActions->preview([
+                    'path' => mb_substr($file->getInternalPath(), 5),
+                    'width' => $event->getArguments()['width'],
+                    'height' => $event->getArguments()['height'],
+                    'crop' => $event->getArguments()['crop'],
+                    'mode' => $event->getArguments()['mode']
+                ]);
+            }
+        );
+
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_post_rename,
+            $fileActions,
+            'rename'
+        );
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_post_create,
+            $fileActions,
+            'create'
+        );
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_post_copy,
+            $fileActions,
+            'copy'
+        );
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_post_write,
+            $fileActions,
+            'write'
+        );
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_post_update,
+            $fileActions,
+            'update'
+        );
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_read,
+            $fileActions,
+            'read'
+        );
+        Util::connectHook(
+            Filesystem::CLASSNAME,
+            Filesystem::signal_delete,
+            $fileActions,
+            'delete'
+        );
+    }
+
+    private function versionsHooks(ILogger $logger) {
+        $versionsActions = new Versions($logger);
+        Util::connectHook('\OCP\Versions', 'rollback', $versionsActions, 'rollback');
+        Util::connectHook('\OCP\Versions', 'delete', $versionsActions, 'delete');
+    }
+
+    private function trashbinHooks(ILogger $logger) {
+        $trashActions = new Trashbin($logger);
+        Util::connectHook('\OCP\Trashbin', 'preDelete', $trashActions, 'delete');
+        Util::connectHook('\OCA\Files_Trashbin\Trashbin', 'post_restore', $trashActions, 'restore');
+    }
+
+    private function securityHooks(ILogger $logger,
+                                    EventDispatcherInterface $eventDispatcher) {
+        $eventDispatcher->addListener(IProvider::EVENT_SUCCESS, function (GenericEvent $event) use ($logger) {
+            $security = new Security($logger);
+            $security->twofactorSuccess($event->getSubject(), $event->getArguments());
+        });
+        $eventDispatcher->addListener(IProvider::EVENT_FAILED, function (GenericEvent $event) use ($logger) {
+            $security = new Security($logger);
+            $security->twofactorFailed($event->getSubject(), $event->getArguments());
+        });
+    }
 }

Spacing +8 added lines, -8 removed lines

@@ -94,7 +94,7 @@ 
 	private function getLogger(IConfig $config,
 							   ILogger $logger,
 							   ILogFactory $logFactory): ILogger {
-		$default = $config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/audit.log';
+		$default = $config->getSystemValue('datadirectory', \OC::$SERVERROOT.'/data').'/audit.log';
 		$logFile = $config->getAppValue('admin_audit', 'logfile', $default);
 
 		if ($logFile === null) {
@@ -172,15 +172,15 @@ 
 
 	private function appHooks(ILogger $logger,
 							  EventDispatcherInterface $eventDispatcher) {
-		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE, function (ManagerEvent $event) use ($logger) {
+		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE, function(ManagerEvent $event) use ($logger) {
 			$appActions = new AppManagement($logger);
 			$appActions->enableApp($event->getAppID());
 		});
-		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE_FOR_GROUPS, function (ManagerEvent $event) use ($logger) {
+		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_ENABLE_FOR_GROUPS, function(ManagerEvent $event) use ($logger) {
 			$appActions = new AppManagement($logger);
 			$appActions->enableAppForGroups($event->getAppID(), $event->getGroups());
 		});
-		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_DISABLE, function (ManagerEvent $event) use ($logger) {
+		$eventDispatcher->addListener(ManagerEvent::EVENT_APP_DISABLE, function(ManagerEvent $event) use ($logger) {
 			$appActions = new AppManagement($logger);
 			$appActions->disableApp($event->getAppID());
 		});
@@ -188,7 +188,7 @@ 
 
 	private function consoleHooks(ILogger $logger,
 								  EventDispatcherInterface $eventDispatcher) {
-		$eventDispatcher->addListener(ConsoleEvent::EVENT_RUN, function (ConsoleEvent $event) use ($logger) {
+		$eventDispatcher->addListener(ConsoleEvent::EVENT_RUN, function(ConsoleEvent $event) use ($logger) {
 			$appActions = new Console($logger);
 			$appActions->runCommand($event->getArguments());
 		});
@@ -199,7 +199,7 @@ 
 		$fileActions = new Files($logger);
 		$eventDispatcher->addListener(
 			IPreview::EVENT,
-			function (GenericEvent $event) use ($fileActions) {
+			function(GenericEvent $event) use ($fileActions) {
 				/** @var File $file */
 				$file = $event->getSubject();
 				$fileActions->preview([
@@ -270,11 +270,11 @@ 
 
 	private function securityHooks(ILogger $logger,
 								   EventDispatcherInterface $eventDispatcher) {
-		$eventDispatcher->addListener(IProvider::EVENT_SUCCESS, function (GenericEvent $event) use ($logger) {
+		$eventDispatcher->addListener(IProvider::EVENT_SUCCESS, function(GenericEvent $event) use ($logger) {
 			$security = new Security($logger);
 			$security->twofactorSuccess($event->getSubject(), $event->getArguments());
 		});
-		$eventDispatcher->addListener(IProvider::EVENT_FAILED, function (GenericEvent $event) use ($logger) {
+		$eventDispatcher->addListener(IProvider::EVENT_FAILED, function(GenericEvent $event) use ($logger) {
 			$security = new Security($logger);
 			$security->twofactorFailed($event->getSubject(), $event->getArguments());
 		});

apps/twofactor_backupcodes/lib/AppInfo/Application.php

Indentation +29 added lines, -29 removed lines

@@ -46,40 +46,40 @@
 use OCP\Util;
 
 class Application extends App implements IBootstrap {
-	public const APP_ID = 'twofactor_backupcodes';
+    public const APP_ID = 'twofactor_backupcodes';
 
-	public function __construct() {
-		parent::__construct(self::APP_ID);
-	}
+    public function __construct() {
+        parent::__construct(self::APP_ID);
+    }
 
-	public function register(IRegistrationContext $context): void {
-		$this->registerHooksAndEvents($context);
-	}
+    public function register(IRegistrationContext $context): void {
+        $this->registerHooksAndEvents($context);
+    }
 
-	public function boot(IBootContext $context): void {
-		Util::connectHook('OC_User', 'post_deleteUser', $this, 'deleteUser');
+    public function boot(IBootContext $context): void {
+        Util::connectHook('OC_User', 'post_deleteUser', $this, 'deleteUser');
 
-		$context->injectFn(Closure::fromCallable([$this, 'registerNotification']));
-	}
+        $context->injectFn(Closure::fromCallable([$this, 'registerNotification']));
+    }
 
-	/**
-	 * Register the hooks and events
-	 */
-	public function registerHooksAndEvents(IRegistrationContext $context) {
-		$context->registerEventListener(CodesGenerated::class, ActivityPublisher::class);
-		$context->registerEventListener(CodesGenerated::class, RegistryUpdater::class);
-		$context->registerEventListener(CodesGenerated::class, ClearNotifications::class);
-		$context->registerEventListener(IRegistry::EVENT_PROVIDER_ENABLED, ProviderEnabled::class);
-		$context->registerEventListener(IRegistry::EVENT_PROVIDER_DISABLED, ProviderDisabled::class);
-	}
+    /**
+     * Register the hooks and events
+     */
+    public function registerHooksAndEvents(IRegistrationContext $context) {
+        $context->registerEventListener(CodesGenerated::class, ActivityPublisher::class);
+        $context->registerEventListener(CodesGenerated::class, RegistryUpdater::class);
+        $context->registerEventListener(CodesGenerated::class, ClearNotifications::class);
+        $context->registerEventListener(IRegistry::EVENT_PROVIDER_ENABLED, ProviderEnabled::class);
+        $context->registerEventListener(IRegistry::EVENT_PROVIDER_DISABLED, ProviderDisabled::class);
+    }
 
-	private function registerNotification(IManager $manager) {
-		$manager->registerNotifierService(Notifier::class);
-	}
+    private function registerNotification(IManager $manager) {
+        $manager->registerNotifierService(Notifier::class);
+    }
 
-	public function deleteUser($params) {
-		/** @var BackupCodeMapper $mapper */
-		$mapper = $this->getContainer()->query(BackupCodeMapper::class);
-		$mapper->deleteCodesByUserId($params['uid']);
-	}
+    public function deleteUser($params) {
+        /** @var BackupCodeMapper $mapper */
+        $mapper = $this->getContainer()->query(BackupCodeMapper::class);
+        $mapper->deleteCodesByUserId($params['uid']);
+    }
 }

apps/files/lib/AppInfo/Application.php

Indentation +114 added lines, -114 removed lines

@@ -60,120 +60,120 @@
 use OCP\Util;
 
 class Application extends App implements IBootstrap {
-	public const APP_ID = 'files';
-
-	public function __construct(array $urlParams=[]) {
-		parent::__construct(self::APP_ID, $urlParams);
-	}
-
-	public function register(IRegistrationContext $context): void {
-		/**
-		 * Controllers
-		 */
-		$context->registerService('APIController', function (IContainer $c) {
-			/** @var IServerContainer $server */
-			$server = $c->query(IServerContainer::class);
-
-			return new ApiController(
-				$c->query('AppName'),
-				$c->query('Request'),
-				$server->getUserSession(),
-				$c->query('TagService'),
-				$server->getPreviewManager(),
-				$server->getShareManager(),
-				$server->getConfig(),
-				$server->getUserFolder()
-			);
-		});
-
-		/**
-		 * Services
-		 */
-		$context->registerService('TagService', function (IContainer $c) {
-			/** @var IServerContainer $server */
-			$server = $c->query(IServerContainer::class);
-
-			return new TagService(
-				$server->getUserSession(),
-				$server->getActivityManager(),
-				$server->getTagManager()->load(self::APP_ID),
-				$server->getUserFolder(),
-				$server->getEventDispatcher()
-			);
-		});
-
-		/*
+    public const APP_ID = 'files';
+
+    public function __construct(array $urlParams=[]) {
+        parent::__construct(self::APP_ID, $urlParams);
+    }
+
+    public function register(IRegistrationContext $context): void {
+        /**
+         * Controllers
+         */
+        $context->registerService('APIController', function (IContainer $c) {
+            /** @var IServerContainer $server */
+            $server = $c->query(IServerContainer::class);
+
+            return new ApiController(
+                $c->query('AppName'),
+                $c->query('Request'),
+                $server->getUserSession(),
+                $c->query('TagService'),
+                $server->getPreviewManager(),
+                $server->getShareManager(),
+                $server->getConfig(),
+                $server->getUserFolder()
+            );
+        });
+
+        /**
+         * Services
+         */
+        $context->registerService('TagService', function (IContainer $c) {
+            /** @var IServerContainer $server */
+            $server = $c->query(IServerContainer::class);
+
+            return new TagService(
+                $server->getUserSession(),
+                $server->getActivityManager(),
+                $server->getTagManager()->load(self::APP_ID),
+                $server->getUserFolder(),
+                $server->getEventDispatcher()
+            );
+        });
+
+        /*
 		 * Register capabilities
 		 */
-		$context->registerCapability(Capabilities::class);
-
-		$context->registerEventListener(LoadAdditionalScriptsEvent::class, LegacyLoadAdditionalScriptsAdapter::class);
-		$context->registerEventListener(LoadSidebar::class, LoadSidebarListener::class);
-
-		$context->registerSearchProvider(FilesSearchProvider::class);
-	}
-
-	public function boot(IBootContext $context): void {
-		$context->injectFn(Closure::fromCallable([$this, 'registerCollaboration']));
-		$context->injectFn([Listener::class, 'register']);
-		$context->injectFn(Closure::fromCallable([$this, 'registerNotification']));
-		$context->injectFn(Closure::fromCallable([$this, 'registerSearchProvider']));
-		$this->registerTemplates();
-		$context->injectFn(Closure::fromCallable([$this, 'registerNavigation']));
-		$this->registerHooks();
-	}
-
-	private function registerCollaboration(IProviderManager $providerManager): void {
-		$providerManager->registerResourceProvider(ResourceProvider::class);
-	}
-
-	private function registerNotification(IManager $notifications): void {
-		$notifications->registerNotifierService(Notifier::class);
-	}
-
-	private function registerSearchProvider(ISearch $search): void {
-		$search->registerProvider(File::class, ['apps' => ['files']]);
-	}
-
-	private function registerTemplates(): void {
-		$templateManager = \OC_Helper::getFileTemplateManager();
-		$templateManager->registerTemplate('application/vnd.oasis.opendocument.presentation', 'core/templates/filetemplates/template.odp');
-		$templateManager->registerTemplate('application/vnd.oasis.opendocument.text', 'core/templates/filetemplates/template.odt');
-		$templateManager->registerTemplate('application/vnd.oasis.opendocument.spreadsheet', 'core/templates/filetemplates/template.ods');
-	}
-
-	private function registerNavigation(IL10N $l10n): void {
-		\OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
-			return [
-				'id' => 'files',
-				'appname' => 'files',
-				'script' => 'list.php',
-				'order' => 0,
-				'name' => $l10n->t('All files')
-			];
-		});
-		\OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
-			return [
-				'id' => 'recent',
-				'appname' => 'files',
-				'script' => 'recentlist.php',
-				'order' => 2,
-				'name' => $l10n->t('Recent')
-			];
-		});
-		\OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
-			return [
-				'id' => 'favorites',
-				'appname' => 'files',
-				'script' => 'simplelist.php',
-				'order' => 5,
-				'name' => $l10n->t('Favorites'),
-				'expandedState' => 'show_Quick_Access'
-			];
-		});
-	}
-
-	private function registerHooks(): void {
-		Util::connectHook('\OCP\Config', 'js', '\OCA\Files\App', 'extendJsConfig');
-	}
+        $context->registerCapability(Capabilities::class);
+
+        $context->registerEventListener(LoadAdditionalScriptsEvent::class, LegacyLoadAdditionalScriptsAdapter::class);
+        $context->registerEventListener(LoadSidebar::class, LoadSidebarListener::class);
+
+        $context->registerSearchProvider(FilesSearchProvider::class);
+    }
+
+    public function boot(IBootContext $context): void {
+        $context->injectFn(Closure::fromCallable([$this, 'registerCollaboration']));
+        $context->injectFn([Listener::class, 'register']);
+        $context->injectFn(Closure::fromCallable([$this, 'registerNotification']));
+        $context->injectFn(Closure::fromCallable([$this, 'registerSearchProvider']));
+        $this->registerTemplates();
+        $context->injectFn(Closure::fromCallable([$this, 'registerNavigation']));
+        $this->registerHooks();
+    }
+
+    private function registerCollaboration(IProviderManager $providerManager): void {
+        $providerManager->registerResourceProvider(ResourceProvider::class);
+    }
+
+    private function registerNotification(IManager $notifications): void {
+        $notifications->registerNotifierService(Notifier::class);
+    }
+
+    private function registerSearchProvider(ISearch $search): void {
+        $search->registerProvider(File::class, ['apps' => ['files']]);
+    }
+
+    private function registerTemplates(): void {
+        $templateManager = \OC_Helper::getFileTemplateManager();
+        $templateManager->registerTemplate('application/vnd.oasis.opendocument.presentation', 'core/templates/filetemplates/template.odp');
+        $templateManager->registerTemplate('application/vnd.oasis.opendocument.text', 'core/templates/filetemplates/template.odt');
+        $templateManager->registerTemplate('application/vnd.oasis.opendocument.spreadsheet', 'core/templates/filetemplates/template.ods');
+    }
+
+    private function registerNavigation(IL10N $l10n): void {
+        \OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
+            return [
+                'id' => 'files',
+                'appname' => 'files',
+                'script' => 'list.php',
+                'order' => 0,
+                'name' => $l10n->t('All files')
+            ];
+        });
+        \OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
+            return [
+                'id' => 'recent',
+                'appname' => 'files',
+                'script' => 'recentlist.php',
+                'order' => 2,
+                'name' => $l10n->t('Recent')
+            ];
+        });
+        \OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
+            return [
+                'id' => 'favorites',
+                'appname' => 'files',
+                'script' => 'simplelist.php',
+                'order' => 5,
+                'name' => $l10n->t('Favorites'),
+                'expandedState' => 'show_Quick_Access'
+            ];
+        });
+    }
+
+    private function registerHooks(): void {
+        Util::connectHook('\OCP\Config', 'js', '\OCA\Files\App', 'extendJsConfig');
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -62,7 +62,7 @@ 
 class Application extends App implements IBootstrap {
 	public const APP_ID = 'files';
 
-	public function __construct(array $urlParams=[]) {
+	public function __construct(array $urlParams = []) {
 		parent::__construct(self::APP_ID, $urlParams);
 	}
 
@@ -70,7 +70,7 @@ 
 		/**
 		 * Controllers
 		 */
-		$context->registerService('APIController', function (IContainer $c) {
+		$context->registerService('APIController', function(IContainer $c) {
 			/** @var IServerContainer $server */
 			$server = $c->query(IServerContainer::class);
 
@@ -89,7 +89,7 @@ 
 		/**
 		 * Services
 		 */
-		$context->registerService('TagService', function (IContainer $c) {
+		$context->registerService('TagService', function(IContainer $c) {
 			/** @var IServerContainer $server */
 			$server = $c->query(IServerContainer::class);
 
@@ -143,7 +143,7 @@ 
 	}
 
 	private function registerNavigation(IL10N $l10n): void {
-		\OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
+		\OCA\Files\App::getNavigationManager()->add(function() use ($l10n) {
 			return [
 				'id' => 'files',
 				'appname' => 'files',
@@ -152,7 +152,7 @@ 
 				'name' => $l10n->t('All files')
 			];
 		});
-		\OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
+		\OCA\Files\App::getNavigationManager()->add(function() use ($l10n) {
 			return [
 				'id' => 'recent',
 				'appname' => 'files',
@@ -161,7 +161,7 @@ 
 				'name' => $l10n->t('Recent')
 			];
 		});
-		\OCA\Files\App::getNavigationManager()->add(function () use ($l10n) {
+		\OCA\Files\App::getNavigationManager()->add(function() use ($l10n) {
 			return [
 				'id' => 'favorites',
 				'appname' => 'files',

apps/updatenotification/lib/AppInfo/Application.php

Indentation +39 added lines, -39 removed lines

@@ -46,49 +46,49 @@
 use OCP\Util;
 
 class Application extends App implements IBootstrap {
-	public function __construct() {
-		parent::__construct('updatenotification', []);
-	}
+    public function __construct() {
+        parent::__construct('updatenotification', []);
+    }
 
-	public function register(IRegistrationContext $context): void {
-	}
+    public function register(IRegistrationContext $context): void {
+    }
 
-	public function boot(IBootContext $context): void {
-		$context->injectFn(function (IConfig $config,
-									 INotificationManager $notificationsManager,
-									 IUserSession $userSession,
-									 IAppManager $appManager,
-									 IGroupManager $groupManager,
-									 IAppContainer $appContainer,
-									 ILogger $logger) {
-			if ($config->getSystemValue('updatechecker', true) !== true) {
-				// Updater check is disabled
-				return;
-			}
+    public function boot(IBootContext $context): void {
+        $context->injectFn(function (IConfig $config,
+                                        INotificationManager $notificationsManager,
+                                        IUserSession $userSession,
+                                        IAppManager $appManager,
+                                        IGroupManager $groupManager,
+                                        IAppContainer $appContainer,
+                                        ILogger $logger) {
+            if ($config->getSystemValue('updatechecker', true) !== true) {
+                // Updater check is disabled
+                return;
+            }
 
-			// Always register the notifier, so background jobs (without a user) can send push notifications
-			$notificationsManager->registerNotifierService(Notifier::class);
+            // Always register the notifier, so background jobs (without a user) can send push notifications
+            $notificationsManager->registerNotifierService(Notifier::class);
 
-			$user = $userSession->getUser();
-			if (!$user instanceof IUser) {
-				// Nothing to do for guests
-				return;
-			}
+            $user = $userSession->getUser();
+            if (!$user instanceof IUser) {
+                // Nothing to do for guests
+                return;
+            }
 
-			if (!$appManager->isEnabledForUser('notifications') &&
-				$groupManager->isAdmin($user->getUID())) {
-				try {
-					$updateChecker = $appContainer->get(UpdateChecker::class);
-				} catch (QueryException $e) {
-					$logger->logException($e);
-					return;
-				}
+            if (!$appManager->isEnabledForUser('notifications') &&
+                $groupManager->isAdmin($user->getUID())) {
+                try {
+                    $updateChecker = $appContainer->get(UpdateChecker::class);
+                } catch (QueryException $e) {
+                    $logger->logException($e);
+                    return;
+                }
 
-				if ($updateChecker->getUpdateState() !== []) {
-					Util::addScript('updatenotification', 'legacy-notification');
-					\OC_Hook::connect('\OCP\Config', 'js', $updateChecker, 'populateJavaScriptVariables');
-				}
-			}
-		});
-	}
+                if ($updateChecker->getUpdateState() !== []) {
+                    Util::addScript('updatenotification', 'legacy-notification');
+                    \OC_Hook::connect('\OCP\Config', 'js', $updateChecker, 'populateJavaScriptVariables');
+                }
+            }
+        });
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 	}
 
 	public function boot(IBootContext $context): void {
-		$context->injectFn(function (IConfig $config,
+		$context->injectFn(function(IConfig $config,
 									 INotificationManager $notificationsManager,
 									 IUserSession $userSession,
 									 IAppManager $appManager,

apps/comments/lib/JSSettingsHelper.php

Indentation +11 added lines, -11 removed lines

@@ -26,19 +26,19 @@
 use OCP\IConfig;
 
 class JSSettingsHelper {
-	/** @var IConfig */
-	private $c;
+    /** @var IConfig */
+    private $c;
 
-	public function __construct(IConfig $c) {
-		$this->c = $c;
-	}
+    public function __construct(IConfig $c) {
+        $this->c = $c;
+    }
 
-	public function extend(array $settings) {
-		$appConfig = json_decode($settings['array']['oc_appconfig'], true);
+    public function extend(array $settings) {
+        $appConfig = json_decode($settings['array']['oc_appconfig'], true);
 
-		$value = (int)$this->c->getAppValue('comments', 'maxAutoCompleteResults', 10);
-		$appConfig['comments']['maxAutoCompleteResults'] = $value;
+        $value = (int)$this->c->getAppValue('comments', 'maxAutoCompleteResults', 10);
+        $appConfig['comments']['maxAutoCompleteResults'] = $value;
 
-		$settings['array']['oc_appconfig'] = json_encode($appConfig);
-	}
+        $settings['array']['oc_appconfig'] = json_encode($appConfig);
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 	public function extend(array $settings) {
 		$appConfig = json_decode($settings['array']['oc_appconfig'], true);
 
-		$value = (int)$this->c->getAppValue('comments', 'maxAutoCompleteResults', 10);
+		$value = (int) $this->c->getAppValue('comments', 'maxAutoCompleteResults', 10);
 		$appConfig['comments']['maxAutoCompleteResults'] = $value;
 
 		$settings['array']['oc_appconfig'] = json_encode($appConfig);

apps/comments/lib/AppInfo/Application.php

Indentation +36 added lines, -36 removed lines

@@ -51,49 +51,49 @@
 use OCP\Util;
 
 class Application extends App implements IBootstrap {
-	public const APP_ID = 'comments';
+    public const APP_ID = 'comments';
 
-	public function __construct(array $urlParams = []) {
-		parent::__construct(self::APP_ID, $urlParams);
-	}
+    public function __construct(array $urlParams = []) {
+        parent::__construct(self::APP_ID, $urlParams);
+    }
 
-	public function register(IRegistrationContext $context): void {
-		$context->registerCapability(Capabilities::class);
+    public function register(IRegistrationContext $context): void {
+        $context->registerCapability(Capabilities::class);
 
-		$context->registerServiceAlias('NotificationsController', Notifications::class);
+        $context->registerServiceAlias('NotificationsController', Notifications::class);
 
-		$context->registerEventListener(
-			LoadAdditionalScriptsEvent::class,
-			LoadAdditionalScripts::class
-		);
-		$context->registerEventListener(
-			LoadSidebar::class,
-			LoadSidebarScripts::class
-		);
-		$context->registerEventListener(
-			CommentsEntityEvent::EVENT_ENTITY,
-			CommentsEntityEventListener::class
-		);
-		$context->registerSearchProvider(Provider::class);
-	}
+        $context->registerEventListener(
+            LoadAdditionalScriptsEvent::class,
+            LoadAdditionalScripts::class
+        );
+        $context->registerEventListener(
+            LoadSidebar::class,
+            LoadSidebarScripts::class
+        );
+        $context->registerEventListener(
+            CommentsEntityEvent::EVENT_ENTITY,
+            CommentsEntityEventListener::class
+        );
+        $context->registerSearchProvider(Provider::class);
+    }
 
-	public function boot(IBootContext $context): void {
-		$context->injectFn(Closure::fromCallable([$this, 'registerNotifier']));
-		$context->injectFn(Closure::fromCallable([$this, 'registerCommentsEventHandler']));
+    public function boot(IBootContext $context): void {
+        $context->injectFn(Closure::fromCallable([$this, 'registerNotifier']));
+        $context->injectFn(Closure::fromCallable([$this, 'registerCommentsEventHandler']));
 
-		$jsSettingsHelper = new JSSettingsHelper($context->getAppContainer()->get(IConfig::class));
-		Util::connectHook('\OCP\Config', 'js', $jsSettingsHelper, 'extend');
+        $jsSettingsHelper = new JSSettingsHelper($context->getAppContainer()->get(IConfig::class));
+        Util::connectHook('\OCP\Config', 'js', $jsSettingsHelper, 'extend');
 
-		$context->getServerContainer()->get(ISearch::class)->registerProvider(LegacyProvider::class, ['apps' => ['files']]);
-	}
+        $context->getServerContainer()->get(ISearch::class)->registerProvider(LegacyProvider::class, ['apps' => ['files']]);
+    }
 
-	protected function registerNotifier(IServerContainer $container) {
-		$container->getNotificationManager()->registerNotifierService(Notifier::class);
-	}
+    protected function registerNotifier(IServerContainer $container) {
+        $container->getNotificationManager()->registerNotifierService(Notifier::class);
+    }
 
-	protected function registerCommentsEventHandler(IServerContainer $container) {
-		$container->getCommentsManager()->registerEventHandler(function () {
-			return $this->getContainer()->query(EventHandler::class);
-		});
-	}
+    protected function registerCommentsEventHandler(IServerContainer $container) {
+        $container->getCommentsManager()->registerEventHandler(function () {
+            return $this->getContainer()->query(EventHandler::class);
+        });
+    }
 }