nextcloud / server

lib/private/AppFramework/Bootstrap/ARegistration.php

Indentation +11 added lines, -11 removed lines

@@ -29,17 +29,17 @@
  * @psalm-immutable
  */
 abstract class ARegistration {
-	/** @var string */
-	private $appId;
+    /** @var string */
+    private $appId;
 
-	public function __construct(string $appId) {
-		$this->appId = $appId;
-	}
+    public function __construct(string $appId) {
+        $this->appId = $appId;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function getAppId(): string {
-		return $this->appId;
-	}
+    /**
+     * @return string
+     */
+    public function getAppId(): string {
+        return $this->appId;
+    }
 }

lib/private/AppFramework/Bootstrap/FunctionInjector.php

Indentation +29 added lines, -29 removed lines

@@ -33,36 +33,36 @@
 use function array_map;
 
 class FunctionInjector {
-	/** @var ContainerInterface */
-	private $container;
+    /** @var ContainerInterface */
+    private $container;
 
-	public function __construct(ContainerInterface $container) {
-		$this->container = $container;
-	}
+    public function __construct(ContainerInterface $container) {
+        $this->container = $container;
+    }
 
-	public function injectFn(callable $fn) {
-		$reflected = new ReflectionFunction(Closure::fromCallable($fn));
-		return $fn(...array_map(function (ReflectionParameter $param) {
-			// First we try by type (more likely these days)
-			if (($type = $param->getType()) !== null) {
-				try {
-					return $this->container->get($type->getName());
-				} catch (QueryException $ex) {
-					// Ignore and try name as well
-				}
-			}
-			// Second we try by name (mostly for primitives)
-			try {
-				return $this->container->get($param->getName());
-			} catch (QueryException $ex) {
-				// As a last resort we pass `null` if allowed
-				if ($type !== null && $type->allowsNull()) {
-					return null;
-				}
+    public function injectFn(callable $fn) {
+        $reflected = new ReflectionFunction(Closure::fromCallable($fn));
+        return $fn(...array_map(function (ReflectionParameter $param) {
+            // First we try by type (more likely these days)
+            if (($type = $param->getType()) !== null) {
+                try {
+                    return $this->container->get($type->getName());
+                } catch (QueryException $ex) {
+                    // Ignore and try name as well
+                }
+            }
+            // Second we try by name (mostly for primitives)
+            try {
+                return $this->container->get($param->getName());
+            } catch (QueryException $ex) {
+                // As a last resort we pass `null` if allowed
+                if ($type !== null && $type->allowsNull()) {
+                    return null;
+                }
 
-				// Nothing worked, time to bail out
-				throw $ex;
-			}
-		}, $reflected->getParameters()));
-	}
+                // Nothing worked, time to bail out
+                throw $ex;
+            }
+        }, $reflected->getParameters()));
+    }
 }

lib/private/Share20/ShareHelper.php

Indentation +183 added lines, -183 removed lines

@@ -31,187 +31,187 @@
 use OCP\Share\IShareHelper;
 
 class ShareHelper implements IShareHelper {
-	/** @var IManager */
-	private $shareManager;
-
-	public function __construct(IManager $shareManager) {
-		$this->shareManager = $shareManager;
-	}
-
-	/**
-	 * @param Node $node
-	 * @return array [ users => [Mapping $uid => $pathForUser], remotes => [Mapping $cloudId => $pathToMountRoot]]
-	 */
-	public function getPathsForAccessList(Node $node) {
-		$result = [
-			'users' => [],
-			'remotes' => [],
-		];
-
-		$accessList = $this->shareManager->getAccessList($node, true, true);
-		if (!empty($accessList['users'])) {
-			$result['users'] = $this->getPathsForUsers($node, $accessList['users']);
-		}
-		if (!empty($accessList['remote'])) {
-			$result['remotes'] = $this->getPathsForRemotes($node, $accessList['remote']);
-		}
-
-		return $result;
-	}
-
-	/**
-	 * Sample:
-	 * $users = [
-	 *   'test1' => ['node_id' => 16, 'node_path' => '/foo'],
-	 *   'test2' => ['node_id' => 23, 'node_path' => '/bar'],
-	 *   'test3' => ['node_id' => 42, 'node_path' => '/cat'],
-	 *   'test4' => ['node_id' => 48, 'node_path' => '/dog'],
-	 * ];
-	 *
-	 * Node tree:
-	 * - SixTeen is the parent of TwentyThree
-	 * - TwentyThree is the parent of FortyTwo
-	 * - FortyEight does not exist
-	 *
-	 * $return = [
-	 *   'test1' => '/foo/TwentyThree/FortyTwo',
-	 *   'test2' => '/bar/FortyTwo',
-	 *   'test3' => '/cat',
-	 * ],
-	 *
-	 * @param Node $node
-	 * @param array[] $users
-	 * @return array
-	 */
-	protected function getPathsForUsers(Node $node, array $users) {
-		/** @var array[] $byId */
-		$byId = [];
-		/** @var array[] $results */
-		$results = [];
-
-		foreach ($users as $uid => $info) {
-			if (!isset($byId[$info['node_id']])) {
-				$byId[$info['node_id']] = [];
-			}
-			$byId[$info['node_id']][$uid] = $info['node_path'];
-		}
-
-		try {
-			if (isset($byId[$node->getId()])) {
-				foreach ($byId[$node->getId()] as $uid => $path) {
-					$results[$uid] = $path;
-				}
-				unset($byId[$node->getId()]);
-			}
-		} catch (NotFoundException $e) {
-			return $results;
-		} catch (InvalidPathException $e) {
-			return $results;
-		}
-
-		if (empty($byId)) {
-			return $results;
-		}
-
-		$item = $node;
-		$appendix = '/' . $node->getName();
-		while (!empty($byId)) {
-			try {
-				/** @var Node $item */
-				$item = $item->getParent();
-
-				if (!empty($byId[$item->getId()])) {
-					foreach ($byId[$item->getId()] as $uid => $path) {
-						$results[$uid] = $path . $appendix;
-					}
-					unset($byId[$item->getId()]);
-				}
-
-				$appendix = '/' . $item->getName() . $appendix;
-			} catch (NotFoundException $e) {
-				return $results;
-			} catch (InvalidPathException $e) {
-				return $results;
-			} catch (NotPermittedException $e) {
-				return $results;
-			}
-		}
-
-		return $results;
-	}
-
-	/**
-	 * Sample:
-	 * $remotes = [
-	 *   'test1' => ['node_id' => 16, 'token' => 't1'],
-	 *   'test2' => ['node_id' => 23, 'token' => 't2'],
-	 *   'test3' => ['node_id' => 42, 'token' => 't3'],
-	 *   'test4' => ['node_id' => 48, 'token' => 't4'],
-	 * ];
-	 *
-	 * Node tree:
-	 * - SixTeen is the parent of TwentyThree
-	 * - TwentyThree is the parent of FortyTwo
-	 * - FortyEight does not exist
-	 *
-	 * $return = [
-	 *   'test1' => ['token' => 't1', 'node_path' => '/SixTeen'],
-	 *   'test2' => ['token' => 't2', 'node_path' => '/SixTeen/TwentyThree'],
-	 *   'test3' => ['token' => 't3', 'node_path' => '/SixTeen/TwentyThree/FortyTwo'],
-	 * ],
-	 *
-	 * @param Node $node
-	 * @param array[] $remotes
-	 * @return array
-	 */
-	protected function getPathsForRemotes(Node $node, array $remotes) {
-		/** @var array[] $byId */
-		$byId = [];
-		/** @var array[] $results */
-		$results = [];
-
-		foreach ($remotes as $cloudId => $info) {
-			if (!isset($byId[$info['node_id']])) {
-				$byId[$info['node_id']] = [];
-			}
-			$byId[$info['node_id']][$cloudId] = $info['token'];
-		}
-
-		$item = $node;
-		while (!empty($byId)) {
-			try {
-				if (!empty($byId[$item->getId()])) {
-					$path = $this->getMountedPath($item);
-					foreach ($byId[$item->getId()] as $uid => $token) {
-						$results[$uid] = [
-							'node_path' => $path,
-							'token' => $token,
-						];
-					}
-					unset($byId[$item->getId()]);
-				}
-
-				/** @var Node $item */
-				$item = $item->getParent();
-			} catch (NotFoundException $e) {
-				return $results;
-			} catch (InvalidPathException $e) {
-				return $results;
-			} catch (NotPermittedException $e) {
-				return $results;
-			}
-		}
-
-		return $results;
-	}
-
-	/**
-	 * @param Node $node
-	 * @return string
-	 */
-	protected function getMountedPath(Node $node) {
-		$path = $node->getPath();
-		$sections = explode('/', $path, 4);
-		return '/' . $sections[3];
-	}
+    /** @var IManager */
+    private $shareManager;
+
+    public function __construct(IManager $shareManager) {
+        $this->shareManager = $shareManager;
+    }
+
+    /**
+     * @param Node $node
+     * @return array [ users => [Mapping $uid => $pathForUser], remotes => [Mapping $cloudId => $pathToMountRoot]]
+     */
+    public function getPathsForAccessList(Node $node) {
+        $result = [
+            'users' => [],
+            'remotes' => [],
+        ];
+
+        $accessList = $this->shareManager->getAccessList($node, true, true);
+        if (!empty($accessList['users'])) {
+            $result['users'] = $this->getPathsForUsers($node, $accessList['users']);
+        }
+        if (!empty($accessList['remote'])) {
+            $result['remotes'] = $this->getPathsForRemotes($node, $accessList['remote']);
+        }
+
+        return $result;
+    }
+
+    /**
+     * Sample:
+     * $users = [
+     *   'test1' => ['node_id' => 16, 'node_path' => '/foo'],
+     *   'test2' => ['node_id' => 23, 'node_path' => '/bar'],
+     *   'test3' => ['node_id' => 42, 'node_path' => '/cat'],
+     *   'test4' => ['node_id' => 48, 'node_path' => '/dog'],
+     * ];
+     *
+     * Node tree:
+     * - SixTeen is the parent of TwentyThree
+     * - TwentyThree is the parent of FortyTwo
+     * - FortyEight does not exist
+     *
+     * $return = [
+     *   'test1' => '/foo/TwentyThree/FortyTwo',
+     *   'test2' => '/bar/FortyTwo',
+     *   'test3' => '/cat',
+     * ],
+     *
+     * @param Node $node
+     * @param array[] $users
+     * @return array
+     */
+    protected function getPathsForUsers(Node $node, array $users) {
+        /** @var array[] $byId */
+        $byId = [];
+        /** @var array[] $results */
+        $results = [];
+
+        foreach ($users as $uid => $info) {
+            if (!isset($byId[$info['node_id']])) {
+                $byId[$info['node_id']] = [];
+            }
+            $byId[$info['node_id']][$uid] = $info['node_path'];
+        }
+
+        try {
+            if (isset($byId[$node->getId()])) {
+                foreach ($byId[$node->getId()] as $uid => $path) {
+                    $results[$uid] = $path;
+                }
+                unset($byId[$node->getId()]);
+            }
+        } catch (NotFoundException $e) {
+            return $results;
+        } catch (InvalidPathException $e) {
+            return $results;
+        }
+
+        if (empty($byId)) {
+            return $results;
+        }
+
+        $item = $node;
+        $appendix = '/' . $node->getName();
+        while (!empty($byId)) {
+            try {
+                /** @var Node $item */
+                $item = $item->getParent();
+
+                if (!empty($byId[$item->getId()])) {
+                    foreach ($byId[$item->getId()] as $uid => $path) {
+                        $results[$uid] = $path . $appendix;
+                    }
+                    unset($byId[$item->getId()]);
+                }
+
+                $appendix = '/' . $item->getName() . $appendix;
+            } catch (NotFoundException $e) {
+                return $results;
+            } catch (InvalidPathException $e) {
+                return $results;
+            } catch (NotPermittedException $e) {
+                return $results;
+            }
+        }
+
+        return $results;
+    }
+
+    /**
+     * Sample:
+     * $remotes = [
+     *   'test1' => ['node_id' => 16, 'token' => 't1'],
+     *   'test2' => ['node_id' => 23, 'token' => 't2'],
+     *   'test3' => ['node_id' => 42, 'token' => 't3'],
+     *   'test4' => ['node_id' => 48, 'token' => 't4'],
+     * ];
+     *
+     * Node tree:
+     * - SixTeen is the parent of TwentyThree
+     * - TwentyThree is the parent of FortyTwo
+     * - FortyEight does not exist
+     *
+     * $return = [
+     *   'test1' => ['token' => 't1', 'node_path' => '/SixTeen'],
+     *   'test2' => ['token' => 't2', 'node_path' => '/SixTeen/TwentyThree'],
+     *   'test3' => ['token' => 't3', 'node_path' => '/SixTeen/TwentyThree/FortyTwo'],
+     * ],
+     *
+     * @param Node $node
+     * @param array[] $remotes
+     * @return array
+     */
+    protected function getPathsForRemotes(Node $node, array $remotes) {
+        /** @var array[] $byId */
+        $byId = [];
+        /** @var array[] $results */
+        $results = [];
+
+        foreach ($remotes as $cloudId => $info) {
+            if (!isset($byId[$info['node_id']])) {
+                $byId[$info['node_id']] = [];
+            }
+            $byId[$info['node_id']][$cloudId] = $info['token'];
+        }
+
+        $item = $node;
+        while (!empty($byId)) {
+            try {
+                if (!empty($byId[$item->getId()])) {
+                    $path = $this->getMountedPath($item);
+                    foreach ($byId[$item->getId()] as $uid => $token) {
+                        $results[$uid] = [
+                            'node_path' => $path,
+                            'token' => $token,
+                        ];
+                    }
+                    unset($byId[$item->getId()]);
+                }
+
+                /** @var Node $item */
+                $item = $item->getParent();
+            } catch (NotFoundException $e) {
+                return $results;
+            } catch (InvalidPathException $e) {
+                return $results;
+            } catch (NotPermittedException $e) {
+                return $results;
+            }
+        }
+
+        return $results;
+    }
+
+    /**
+     * @param Node $node
+     * @return string
+     */
+    protected function getMountedPath(Node $node) {
+        $path = $node->getPath();
+        $sections = explode('/', $path, 4);
+        return '/' . $sections[3];
+    }
 }

lib/private/Share20/UserRemovedListener.php

Indentation +11 added lines, -11 removed lines

@@ -34,18 +34,18 @@
  * @template-implements IEventListener<UserRemovedEvent>
  */
 class UserRemovedListener implements IEventListener {
-	/** @var IManager */
-	protected $shareManager;
+    /** @var IManager */
+    protected $shareManager;
 
-	public function __construct(IManager $shareManager) {
-		$this->shareManager = $shareManager;
-	}
+    public function __construct(IManager $shareManager) {
+        $this->shareManager = $shareManager;
+    }
 
-	public function handle(Event $event): void {
-		if (!$event instanceof UserRemovedEvent) {
-			return;
-		}
+    public function handle(Event $event): void {
+        if (!$event instanceof UserRemovedEvent) {
+            return;
+        }
 
-		$this->shareManager->userDeletedFromGroup($event->getUser()->getUID(), $event->getGroup()->getGID());
-	}
+        $this->shareManager->userDeletedFromGroup($event->getUser()->getUID(), $event->getGroup()->getGID());
+    }
 }

lib/private/Comments/ManagerFactory.php

Indentation +23 added lines, -23 removed lines

@@ -30,29 +30,29 @@
 use OCP\IServerContainer;
 
 class ManagerFactory implements ICommentsManagerFactory {
-	/**
-	 * Server container
-	 *
-	 * @var IServerContainer
-	 */
-	private $serverContainer;
+    /**
+     * Server container
+     *
+     * @var IServerContainer
+     */
+    private $serverContainer;
 
-	/**
-	 * Constructor for the comments manager factory
-	 *
-	 * @param IServerContainer $serverContainer server container
-	 */
-	public function __construct(IServerContainer $serverContainer) {
-		$this->serverContainer = $serverContainer;
-	}
+    /**
+     * Constructor for the comments manager factory
+     *
+     * @param IServerContainer $serverContainer server container
+     */
+    public function __construct(IServerContainer $serverContainer) {
+        $this->serverContainer = $serverContainer;
+    }
 
-	/**
-	 * creates and returns an instance of the ICommentsManager
-	 *
-	 * @return ICommentsManager
-	 * @since 9.0.0
-	 */
-	public function getManager() {
-		return $this->serverContainer->get(Manager::class);
-	}
+    /**
+     * creates and returns an instance of the ICommentsManager
+     *
+     * @return ICommentsManager
+     * @since 9.0.0
+     */
+    public function getManager() {
+        return $this->serverContainer->get(Manager::class);
+    }
 }

lib/private/Contacts/ContactsMenu/ActionFactory.php

Indentation +17 added lines, -17 removed lines

@@ -27,22 +27,22 @@
 use OCP\Contacts\ContactsMenu\ILinkAction;
 
 class ActionFactory implements IActionFactory {
-	/**
-	 * {@inheritDoc}
-	 */
-	public function newLinkAction(string $icon, string $name, string $href, string $appId = ''): ILinkAction {
-		$action = new LinkAction();
-		$action->setName($name);
-		$action->setIcon($icon);
-		$action->setHref($href);
-		$action->setAppId($appId);
-		return $action;
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function newLinkAction(string $icon, string $name, string $href, string $appId = ''): ILinkAction {
+        $action = new LinkAction();
+        $action->setName($name);
+        $action->setIcon($icon);
+        $action->setHref($href);
+        $action->setAppId($appId);
+        return $action;
+    }
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function newEMailAction(string $icon, string $name, string $email, string $appId = ''): ILinkAction {
-		return $this->newLinkAction($icon, $name, 'mailto:' . $email, $appId);
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function newEMailAction(string $icon, string $name, string $email, string $appId = ''): ILinkAction {
+        return $this->newLinkAction($icon, $name, 'mailto:' . $email, $appId);
+    }
 }

lib/private/Repair/NC14/AddPreviewBackgroundCleanupJob.php

Indentation +11 added lines, -11 removed lines

@@ -31,18 +31,18 @@
 use OCP\Migration\IRepairStep;
 
 class AddPreviewBackgroundCleanupJob implements IRepairStep {
-	/** @var IJobList */
-	private $jobList;
+    /** @var IJobList */
+    private $jobList;
 
-	public function __construct(IJobList $jobList) {
-		$this->jobList = $jobList;
-	}
+    public function __construct(IJobList $jobList) {
+        $this->jobList = $jobList;
+    }
 
-	public function getName(): string {
-		return 'Add preview background cleanup job';
-	}
+    public function getName(): string {
+        return 'Add preview background cleanup job';
+    }
 
-	public function run(IOutput $output) {
-		$this->jobList->add(BackgroundCleanupJob::class);
-	}
+    public function run(IOutput $output) {
+        $this->jobList->add(BackgroundCleanupJob::class);
+    }
 }

lib/private/Repair/AddCleanupUpdaterBackupsJob.php

Indentation +11 added lines, -11 removed lines

@@ -28,18 +28,18 @@
 use OCP\Migration\IRepairStep;
 
 class AddCleanupUpdaterBackupsJob implements IRepairStep {
-	/** @var IJobList */
-	protected $jobList;
+    /** @var IJobList */
+    protected $jobList;
 
-	public function __construct(IJobList $jobList) {
-		$this->jobList = $jobList;
-	}
+    public function __construct(IJobList $jobList) {
+        $this->jobList = $jobList;
+    }
 
-	public function getName() {
-		return 'Queue a one-time job to cleanup old backups of the updater';
-	}
+    public function getName() {
+        return 'Queue a one-time job to cleanup old backups of the updater';
+    }
 
-	public function run(IOutput $output) {
-		$this->jobList->add(BackgroundCleanupUpdaterBackupsJob::class);
-	}
+    public function run(IOutput $output) {
+        $this->jobList->add(BackgroundCleanupUpdaterBackupsJob::class);
+    }
 }

lib/private/Repair/NC20/ShippedDashboardEnable.php

Indentation +15 added lines, -15 removed lines

@@ -30,22 +30,22 @@
 use OCP\Migration\IRepairStep;
 
 class ShippedDashboardEnable implements IRepairStep {
-	/** @var IConfig */
-	private $config;
+    /** @var IConfig */
+    private $config;
 
-	public function __construct(IConfig $config) {
-		$this->config = $config;
-	}
+    public function __construct(IConfig $config) {
+        $this->config = $config;
+    }
 
-	public function getName() {
-		return 'Remove old dashboard app config data';
-	}
+    public function getName() {
+        return 'Remove old dashboard app config data';
+    }
 
-	public function run(IOutput $output) {
-		$version = $this->config->getAppValue('dashboard', 'version', '7.0.0');
-		if (version_compare($version, '7.0.0', '<')) {
-			$this->config->deleteAppValues('dashboard');
-			$output->info('Removed old dashboard app config');
-		}
-	}
+    public function run(IOutput $output) {
+        $version = $this->config->getAppValue('dashboard', 'version', '7.0.0');
+        if (version_compare($version, '7.0.0', '<')) {
+            $this->config->deleteAppValues('dashboard');
+            $output->info('Removed old dashboard app config');
+        }
+    }
 }