nextcloud / server

core/Command/TwoFactorAuth/Cleanup.php

Indentation +17 added lines, -17 removed lines

@@ -33,29 +33,29 @@
 
 class Cleanup extends Base {
 
-	/** @var IRegistry */
-	private $registry;
+    /** @var IRegistry */
+    private $registry;
 
-	public function __construct(IRegistry $registry) {
-		parent::__construct();
+    public function __construct(IRegistry $registry) {
+        parent::__construct();
 
-		$this->registry = $registry;
-	}
+        $this->registry = $registry;
+    }
 
-	protected function configure() {
-		parent::configure();
+    protected function configure() {
+        parent::configure();
 
-		$this->setName('twofactorauth:cleanup');
-		$this->setDescription('Clean up the two-factor user-provider association of an uninstalled/removed provider');
-		$this->addArgument('provider-id', InputArgument::REQUIRED);
-	}
+        $this->setName('twofactorauth:cleanup');
+        $this->setDescription('Clean up the two-factor user-provider association of an uninstalled/removed provider');
+        $this->addArgument('provider-id', InputArgument::REQUIRED);
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output) {
-		$providerId = $input->getArgument('provider-id');
+    protected function execute(InputInterface $input, OutputInterface $output) {
+        $providerId = $input->getArgument('provider-id');
 
-		$this->registry->cleanUp($providerId);
+        $this->registry->cleanUp($providerId);
 
-		$output->writeln("<info>All user-provider associations for provider <options=bold>$providerId</> have been removed.</info>");
-	}
+        $output->writeln("<info>All user-provider associations for provider <options=bold>$providerId</> have been removed.</info>");
+    }
 
 }

core/Command/TwoFactorAuth/Disable.php

Indentation +32 added lines, -32 removed lines

@@ -32,42 +32,42 @@
 
 class Disable extends Base {
 
-	/** @var ProviderManager */
-	private $manager;
+    /** @var ProviderManager */
+    private $manager;
 
-	/** @var IUserManager */
-	protected $userManager;
+    /** @var IUserManager */
+    protected $userManager;
 
-	public function __construct(ProviderManager $manager, IUserManager $userManager) {
-		parent::__construct('twofactorauth:disable');
-		$this->manager = $manager;
-		$this->userManager = $userManager;
-	}
+    public function __construct(ProviderManager $manager, IUserManager $userManager) {
+        parent::__construct('twofactorauth:disable');
+        $this->manager = $manager;
+        $this->userManager = $userManager;
+    }
 
-	protected function configure() {
-		parent::configure();
+    protected function configure() {
+        parent::configure();
 
-		$this->setName('twofactorauth:disable');
-		$this->setDescription('Disable two-factor authentication for a user');
-		$this->addArgument('uid', InputArgument::REQUIRED);
-		$this->addArgument('provider_id', InputArgument::REQUIRED);
-	}
+        $this->setName('twofactorauth:disable');
+        $this->setDescription('Disable two-factor authentication for a user');
+        $this->addArgument('uid', InputArgument::REQUIRED);
+        $this->addArgument('provider_id', InputArgument::REQUIRED);
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output) {
-		$uid = $input->getArgument('uid');
-		$providerId = $input->getArgument('provider_id');
-		$user = $this->userManager->get($uid);
-		if (is_null($user)) {
-			$output->writeln("<error>Invalid UID</error>");
-			return 1;
-		}
-		if ($this->manager->tryDisableProviderFor($providerId, $user)) {
-			$output->writeln("Two-factor provider <options=bold>$providerId</> disabled for user <options=bold>$uid</>.");
-			return 0;
-		} else {
-			$output->writeln("<error>The provider does not support this operation.</error>");
-			return 2;
-		}
-	}
+    protected function execute(InputInterface $input, OutputInterface $output) {
+        $uid = $input->getArgument('uid');
+        $providerId = $input->getArgument('provider_id');
+        $user = $this->userManager->get($uid);
+        if (is_null($user)) {
+            $output->writeln("<error>Invalid UID</error>");
+            return 1;
+        }
+        if ($this->manager->tryDisableProviderFor($providerId, $user)) {
+            $output->writeln("Two-factor provider <options=bold>$providerId</> disabled for user <options=bold>$uid</>.");
+            return 0;
+        } else {
+            $output->writeln("<error>The provider does not support this operation.</error>");
+            return 2;
+        }
+    }
 
 }

core/Command/TwoFactorAuth/Enable.php

Indentation +32 added lines, -32 removed lines

@@ -31,42 +31,42 @@
 
 class Enable extends Base {
 
-	/** @var ProviderManager */
-	private $manager;
+    /** @var ProviderManager */
+    private $manager;
 
-	/** @var IUserManager */
-	protected $userManager;
+    /** @var IUserManager */
+    protected $userManager;
 
-	public function __construct(ProviderManager $manager, IUserManager $userManager) {
-		parent::__construct('twofactorauth:enable');
-		$this->manager = $manager;
-		$this->userManager = $userManager;
-	}
+    public function __construct(ProviderManager $manager, IUserManager $userManager) {
+        parent::__construct('twofactorauth:enable');
+        $this->manager = $manager;
+        $this->userManager = $userManager;
+    }
 
-	protected function configure() {
-		parent::configure();
+    protected function configure() {
+        parent::configure();
 
-		$this->setName('twofactorauth:enable');
-		$this->setDescription('Enable two-factor authentication for a user');
-		$this->addArgument('uid', InputArgument::REQUIRED);
-		$this->addArgument('provider_id', InputArgument::REQUIRED);
-	}
+        $this->setName('twofactorauth:enable');
+        $this->setDescription('Enable two-factor authentication for a user');
+        $this->addArgument('uid', InputArgument::REQUIRED);
+        $this->addArgument('provider_id', InputArgument::REQUIRED);
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output) {
-		$uid = $input->getArgument('uid');
-		$providerId = $input->getArgument('provider_id');
-		$user = $this->userManager->get($uid);
-		if (is_null($user)) {
-			$output->writeln("<error>Invalid UID</error>");
-			return 1;
-		}
-		if ($this->manager->tryEnableProviderFor($providerId, $user)) {
-			$output->writeln("Two-factor provider <options=bold>$providerId</> enabled for user <options=bold>$uid</>.");
-			return 0;
-		} else {
-			$output->writeln("<error>The provider does not support this operation.</error>");
-			return 2;
-		}
-	}
+    protected function execute(InputInterface $input, OutputInterface $output) {
+        $uid = $input->getArgument('uid');
+        $providerId = $input->getArgument('provider_id');
+        $user = $this->userManager->get($uid);
+        if (is_null($user)) {
+            $output->writeln("<error>Invalid UID</error>");
+            return 1;
+        }
+        if ($this->manager->tryEnableProviderFor($providerId, $user)) {
+            $output->writeln("Two-factor provider <options=bold>$providerId</> enabled for user <options=bold>$uid</>.");
+            return 0;
+        } else {
+            $output->writeln("<error>The provider does not support this operation.</error>");
+            return 2;
+        }
+    }
 
 }

lib/public/Authentication/TwoFactorAuth/IProvidesIcons.php

Indentation +16 added lines, -16 removed lines

@@ -34,22 +34,22 @@
  */
 interface IProvidesIcons extends IProvider {
 
-	/**
-	 * Get the path to the light (white) icon of this provider
-	 *
-	 * @return String
-	 *
-	 * @since 15.0.0
-	 */
-	public function getLightIcon(): String;
+    /**
+     * Get the path to the light (white) icon of this provider
+     *
+     * @return String
+     *
+     * @since 15.0.0
+     */
+    public function getLightIcon(): String;
 
-	/**
-	 * Get the path to the dark (black) icon of this provider
-	 *
-	 * @return String
-	 *
-	 * @since 15.0.0
-	 */
-	public function getDarkIcon(): String;
+    /**
+     * Get the path to the dark (black) icon of this provider
+     *
+     * @return String
+     *
+     * @since 15.0.0
+     */
+    public function getDarkIcon(): String;
 
 }

apps/twofactor_backupcodes/lib/Migration/CheckBackupCodes.php

Indentation +11 added lines, -11 removed lines

@@ -30,19 +30,19 @@
 
 class CheckBackupCodes 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 background job to check for backup codes';
-	}
+    public function getName(): string {
+        return 'Add background job to check for backup codes';
+    }
 
-	public function run(IOutput $output) {
-		$this->jobList->add(\OCA\TwoFactorBackupCodes\BackgroundJob\CheckBackupCodes::class);
-	}
+    public function run(IOutput $output) {
+        $this->jobList->add(\OCA\TwoFactorBackupCodes\BackgroundJob\CheckBackupCodes::class);
+    }
 
 }

lib/private/Setup/Sqlite.php

Spacing +2 added lines, -2 removed lines

@@ -64,10 +64,10 @@
 	public function setupDatabase($username) {
 		$datadir = $this->config->getValue(
 			'datadirectory',
-			\OC::$SERVERROOT . '/data'
+			\OC::$SERVERROOT.'/data'
 		);
 
-		$sqliteFile = $datadir . '/' . $this->dbName . 'db';
+		$sqliteFile = $datadir.'/'.$this->dbName.'db';
 		if (file_exists($sqliteFile)) {
 			unlink($sqliteFile);
 		}

Indentation +29 added lines, -29 removed lines

@@ -26,14 +26,14 @@ 
 use OC\DB\ConnectionFactory;
 
 class Sqlite extends AbstractDatabase {
-	public $dbprettyname = 'Sqlite';
+    public $dbprettyname = 'Sqlite';
 
-	public function validate($config) {
-		return [];
-	}
+    public function validate($config) {
+        return [];
+    }
 
-	public function initialize($config) {
-		/*
+    public function initialize($config) {
+        /*
 		 * Web: When using web based installer its not possible to set dbname
 		 * or dbtableprefix. Defaults used from ConnectionFactory and dbtype = 'sqlite'
 		 * is written to config.php.
@@ -44,32 +44,32 @@ 
 		 * in connection factory configuration is obtained from config.php.
 		 */
 
-		$this->dbName = empty($config['dbname'])
-			? ConnectionFactory::DEFAULT_DBNAME
-			: $config['dbname'];
+        $this->dbName = empty($config['dbname'])
+            ? ConnectionFactory::DEFAULT_DBNAME
+            : $config['dbname'];
 
-		$this->tablePrefix = empty($config['dbtableprefix'])
-			? ConnectionFactory::DEFAULT_DBTABLEPREFIX
-			: $config['dbtableprefix'];
+        $this->tablePrefix = empty($config['dbtableprefix'])
+            ? ConnectionFactory::DEFAULT_DBTABLEPREFIX
+            : $config['dbtableprefix'];
 
-		if ($this->dbName !== ConnectionFactory::DEFAULT_DBNAME) {
-			$this->config->setValue('dbname', $this->dbName);
-		}
+        if ($this->dbName !== ConnectionFactory::DEFAULT_DBNAME) {
+            $this->config->setValue('dbname', $this->dbName);
+        }
 
-		if ($this->tablePrefix !== ConnectionFactory::DEFAULT_DBTABLEPREFIX) {
-			$this->config->setValue('dbtableprefix', $this->tablePrefix);
-		}
-	}
+        if ($this->tablePrefix !== ConnectionFactory::DEFAULT_DBTABLEPREFIX) {
+            $this->config->setValue('dbtableprefix', $this->tablePrefix);
+        }
+    }
 
-	public function setupDatabase($username) {
-		$datadir = $this->config->getValue(
-			'datadirectory',
-			\OC::$SERVERROOT . '/data'
-		);
+    public function setupDatabase($username) {
+        $datadir = $this->config->getValue(
+            'datadirectory',
+            \OC::$SERVERROOT . '/data'
+        );
 
-		$sqliteFile = $datadir . '/' . $this->dbName . 'db';
-		if (file_exists($sqliteFile)) {
-			unlink($sqliteFile);
-		}
-	}
+        $sqliteFile = $datadir . '/' . $this->dbName . 'db';
+        if (file_exists($sqliteFile)) {
+            unlink($sqliteFile);
+        }
+    }
 }

lib/private/DB/ConnectionFactory.php

Spacing +3 added lines, -3 removed lines

@@ -140,7 +140,7 @@ 
 				if ($host === '') {
 					$additionalConnectionParams['dbname'] = $dbName; // use dbname as easy connect name
 				} else {
-					$additionalConnectionParams['dbname'] = '//' . $host . (!empty($port) ? ":{$port}" : "") . '/' . $dbName;
+					$additionalConnectionParams['dbname'] = '//'.$host.(!empty($port) ? ":{$port}" : "").'/'.$dbName;
 				}
 				unset($additionalConnectionParams['host']);
 				break;
@@ -195,8 +195,8 @@ 
 		$name = $this->config->getValue('dbname', self::DEFAULT_DBNAME);
 
 		if ($this->normalizeType($type) === 'sqlite3') {
-			$dataDir = $this->config->getValue("datadirectory", \OC::$SERVERROOT . '/data');
-			$connectionParams['path'] = $dataDir . '/' . $name . '.db';
+			$dataDir = $this->config->getValue("datadirectory", \OC::$SERVERROOT.'/data');
+			$connectionParams['path'] = $dataDir.'/'.$name.'.db';
 		} else {
 			$host = $this->config->getValue('dbhost', '');
 			$connectionParams = array_merge($connectionParams, $this->splitHostFromPortAndSocket($host));

Indentation +194 added lines, -194 removed lines

@@ -40,219 +40,219 @@
  * Takes care of creating and configuring Doctrine connections.
  */
 class ConnectionFactory {
-	/** @var string default database name */
-	public const DEFAULT_DBNAME = 'owncloud';
+    /** @var string default database name */
+    public const DEFAULT_DBNAME = 'owncloud';
 
-	/** @var string default database table prefix */
-	public const DEFAULT_DBTABLEPREFIX = 'oc_';
+    /** @var string default database table prefix */
+    public const DEFAULT_DBTABLEPREFIX = 'oc_';
 
-	/**
-	 * @var array
-	 *
-	 * Array mapping DBMS type to default connection parameters passed to
-	 * \Doctrine\DBAL\DriverManager::getConnection().
-	 */
-	protected $defaultConnectionParams = [
-		'mysql' => [
-			'adapter' => AdapterMySQL::class,
-			'charset' => 'UTF8',
-			'driver' => 'pdo_mysql',
-			'wrapperClass' => Connection::class,
-		],
-		'oci' => [
-			'adapter' => AdapterOCI8::class,
-			'charset' => 'AL32UTF8',
-			'driver' => 'oci8',
-			'wrapperClass' => OracleConnection::class,
-		],
-		'pgsql' => [
-			'adapter' => AdapterPgSql::class,
-			'driver' => 'pdo_pgsql',
-			'wrapperClass' => Connection::class,
-		],
-		'sqlite3' => [
-			'adapter' => AdapterSqlite::class,
-			'driver' => 'pdo_sqlite',
-			'wrapperClass' => Connection::class,
-		],
-	];
+    /**
+     * @var array
+     *
+     * Array mapping DBMS type to default connection parameters passed to
+     * \Doctrine\DBAL\DriverManager::getConnection().
+     */
+    protected $defaultConnectionParams = [
+        'mysql' => [
+            'adapter' => AdapterMySQL::class,
+            'charset' => 'UTF8',
+            'driver' => 'pdo_mysql',
+            'wrapperClass' => Connection::class,
+        ],
+        'oci' => [
+            'adapter' => AdapterOCI8::class,
+            'charset' => 'AL32UTF8',
+            'driver' => 'oci8',
+            'wrapperClass' => OracleConnection::class,
+        ],
+        'pgsql' => [
+            'adapter' => AdapterPgSql::class,
+            'driver' => 'pdo_pgsql',
+            'wrapperClass' => Connection::class,
+        ],
+        'sqlite3' => [
+            'adapter' => AdapterSqlite::class,
+            'driver' => 'pdo_sqlite',
+            'wrapperClass' => Connection::class,
+        ],
+    ];
 
-	/** @var SystemConfig */
-	private $config;
+    /** @var SystemConfig */
+    private $config;
 
-	/**
-	 * ConnectionFactory constructor.
-	 *
-	 * @param SystemConfig $systemConfig
-	 */
-	public function __construct(SystemConfig $systemConfig) {
-		$this->config = $systemConfig;
-		if ($this->config->getValue('mysql.utf8mb4', false)) {
-			$this->defaultConnectionParams['mysql']['charset'] = 'utf8mb4';
-		}
-	}
+    /**
+     * ConnectionFactory constructor.
+     *
+     * @param SystemConfig $systemConfig
+     */
+    public function __construct(SystemConfig $systemConfig) {
+        $this->config = $systemConfig;
+        if ($this->config->getValue('mysql.utf8mb4', false)) {
+            $this->defaultConnectionParams['mysql']['charset'] = 'utf8mb4';
+        }
+    }
 
-	/**
-	 * @brief Get default connection parameters for a given DBMS.
-	 * @param string $type DBMS type
-	 * @throws \InvalidArgumentException If $type is invalid
-	 * @return array Default connection parameters.
-	 */
-	public function getDefaultConnectionParams($type) {
-		$normalizedType = $this->normalizeType($type);
-		if (!isset($this->defaultConnectionParams[$normalizedType])) {
-			throw new \InvalidArgumentException("Unsupported type: $type");
-		}
-		$result = $this->defaultConnectionParams[$normalizedType];
-		// \PDO::MYSQL_ATTR_FOUND_ROWS may not be defined, e.g. when the MySQL
-		// driver is missing. In this case, we won't be able to connect anyway.
-		if ($normalizedType === 'mysql' && defined('\PDO::MYSQL_ATTR_FOUND_ROWS')) {
-			$result['driverOptions'] = [
-				\PDO::MYSQL_ATTR_FOUND_ROWS => true,
-			];
-		}
-		return $result;
-	}
+    /**
+     * @brief Get default connection parameters for a given DBMS.
+     * @param string $type DBMS type
+     * @throws \InvalidArgumentException If $type is invalid
+     * @return array Default connection parameters.
+     */
+    public function getDefaultConnectionParams($type) {
+        $normalizedType = $this->normalizeType($type);
+        if (!isset($this->defaultConnectionParams[$normalizedType])) {
+            throw new \InvalidArgumentException("Unsupported type: $type");
+        }
+        $result = $this->defaultConnectionParams[$normalizedType];
+        // \PDO::MYSQL_ATTR_FOUND_ROWS may not be defined, e.g. when the MySQL
+        // driver is missing. In this case, we won't be able to connect anyway.
+        if ($normalizedType === 'mysql' && defined('\PDO::MYSQL_ATTR_FOUND_ROWS')) {
+            $result['driverOptions'] = [
+                \PDO::MYSQL_ATTR_FOUND_ROWS => true,
+            ];
+        }
+        return $result;
+    }
 
-	/**
-	 * @brief Get default connection parameters for a given DBMS.
-	 * @param string $type DBMS type
-	 * @param array $additionalConnectionParams Additional connection parameters
-	 * @return \OC\DB\Connection
-	 */
-	public function getConnection($type, $additionalConnectionParams) {
-		$normalizedType = $this->normalizeType($type);
-		$eventManager = new EventManager();
-		$eventManager->addEventSubscriber(new SetTransactionIsolationLevel());
-		switch ($normalizedType) {
-			case 'mysql':
-				$eventManager->addEventSubscriber(
-					new SQLSessionInit("SET SESSION AUTOCOMMIT=1"));
-				break;
-			case 'oci':
-				$eventManager->addEventSubscriber(new OracleSessionInit);
-				// the driverOptions are unused in dbal and need to be mapped to the parameters
-				if (isset($additionalConnectionParams['driverOptions'])) {
-					$additionalConnectionParams = array_merge($additionalConnectionParams, $additionalConnectionParams['driverOptions']);
-				}
-				$host = $additionalConnectionParams['host'];
-				$port = isset($additionalConnectionParams['port']) ? $additionalConnectionParams['port'] : null;
-				$dbName = $additionalConnectionParams['dbname'];
+    /**
+     * @brief Get default connection parameters for a given DBMS.
+     * @param string $type DBMS type
+     * @param array $additionalConnectionParams Additional connection parameters
+     * @return \OC\DB\Connection
+     */
+    public function getConnection($type, $additionalConnectionParams) {
+        $normalizedType = $this->normalizeType($type);
+        $eventManager = new EventManager();
+        $eventManager->addEventSubscriber(new SetTransactionIsolationLevel());
+        switch ($normalizedType) {
+            case 'mysql':
+                $eventManager->addEventSubscriber(
+                    new SQLSessionInit("SET SESSION AUTOCOMMIT=1"));
+                break;
+            case 'oci':
+                $eventManager->addEventSubscriber(new OracleSessionInit);
+                // the driverOptions are unused in dbal and need to be mapped to the parameters
+                if (isset($additionalConnectionParams['driverOptions'])) {
+                    $additionalConnectionParams = array_merge($additionalConnectionParams, $additionalConnectionParams['driverOptions']);
+                }
+                $host = $additionalConnectionParams['host'];
+                $port = isset($additionalConnectionParams['port']) ? $additionalConnectionParams['port'] : null;
+                $dbName = $additionalConnectionParams['dbname'];
 
-				// we set the connect string as dbname and unset the host to coerce doctrine into using it as connect string
-				if ($host === '') {
-					$additionalConnectionParams['dbname'] = $dbName; // use dbname as easy connect name
-				} else {
-					$additionalConnectionParams['dbname'] = '//' . $host . (!empty($port) ? ":{$port}" : "") . '/' . $dbName;
-				}
-				unset($additionalConnectionParams['host']);
-				break;
+                // we set the connect string as dbname and unset the host to coerce doctrine into using it as connect string
+                if ($host === '') {
+                    $additionalConnectionParams['dbname'] = $dbName; // use dbname as easy connect name
+                } else {
+                    $additionalConnectionParams['dbname'] = '//' . $host . (!empty($port) ? ":{$port}" : "") . '/' . $dbName;
+                }
+                unset($additionalConnectionParams['host']);
+                break;
 
-			case 'sqlite3':
-				$journalMode = $additionalConnectionParams['sqlite.journal_mode'];
-				$additionalConnectionParams['platform'] = new OCSqlitePlatform();
-				$eventManager->addEventSubscriber(new SQLiteSessionInit(true, $journalMode));
-				break;
-		}
-		/** @var Connection $connection */
-		$connection = DriverManager::getConnection(
-			array_merge($this->getDefaultConnectionParams($type), $additionalConnectionParams),
-			new Configuration(),
-			$eventManager
-		);
-		return $connection;
-	}
+            case 'sqlite3':
+                $journalMode = $additionalConnectionParams['sqlite.journal_mode'];
+                $additionalConnectionParams['platform'] = new OCSqlitePlatform();
+                $eventManager->addEventSubscriber(new SQLiteSessionInit(true, $journalMode));
+                break;
+        }
+        /** @var Connection $connection */
+        $connection = DriverManager::getConnection(
+            array_merge($this->getDefaultConnectionParams($type), $additionalConnectionParams),
+            new Configuration(),
+            $eventManager
+        );
+        return $connection;
+    }
 
-	/**
-	 * @brief Normalize DBMS type
-	 * @param string $type DBMS type
-	 * @return string Normalized DBMS type
-	 */
-	public function normalizeType($type) {
-		return $type === 'sqlite' ? 'sqlite3' : $type;
-	}
+    /**
+     * @brief Normalize DBMS type
+     * @param string $type DBMS type
+     * @return string Normalized DBMS type
+     */
+    public function normalizeType($type) {
+        return $type === 'sqlite' ? 'sqlite3' : $type;
+    }
 
-	/**
-	 * Checks whether the specified DBMS type is valid.
-	 *
-	 * @param string $type
-	 * @return bool
-	 */
-	public function isValidType($type) {
-		$normalizedType = $this->normalizeType($type);
-		return isset($this->defaultConnectionParams[$normalizedType]);
-	}
+    /**
+     * Checks whether the specified DBMS type is valid.
+     *
+     * @param string $type
+     * @return bool
+     */
+    public function isValidType($type) {
+        $normalizedType = $this->normalizeType($type);
+        return isset($this->defaultConnectionParams[$normalizedType]);
+    }
 
-	/**
-	 * Create the connection parameters for the config
-	 *
-	 * @return array
-	 */
-	public function createConnectionParams() {
-		$type = $this->config->getValue('dbtype', 'sqlite');
+    /**
+     * Create the connection parameters for the config
+     *
+     * @return array
+     */
+    public function createConnectionParams() {
+        $type = $this->config->getValue('dbtype', 'sqlite');
 
-		$connectionParams = [
-			'user' => $this->config->getValue('dbuser', ''),
-			'password' => $this->config->getValue('dbpassword', ''),
-		];
-		$name = $this->config->getValue('dbname', self::DEFAULT_DBNAME);
+        $connectionParams = [
+            'user' => $this->config->getValue('dbuser', ''),
+            'password' => $this->config->getValue('dbpassword', ''),
+        ];
+        $name = $this->config->getValue('dbname', self::DEFAULT_DBNAME);
 
-		if ($this->normalizeType($type) === 'sqlite3') {
-			$dataDir = $this->config->getValue("datadirectory", \OC::$SERVERROOT . '/data');
-			$connectionParams['path'] = $dataDir . '/' . $name . '.db';
-		} else {
-			$host = $this->config->getValue('dbhost', '');
-			$connectionParams = array_merge($connectionParams, $this->splitHostFromPortAndSocket($host));
-			$connectionParams['dbname'] = $name;
-		}
+        if ($this->normalizeType($type) === 'sqlite3') {
+            $dataDir = $this->config->getValue("datadirectory", \OC::$SERVERROOT . '/data');
+            $connectionParams['path'] = $dataDir . '/' . $name . '.db';
+        } else {
+            $host = $this->config->getValue('dbhost', '');
+            $connectionParams = array_merge($connectionParams, $this->splitHostFromPortAndSocket($host));
+            $connectionParams['dbname'] = $name;
+        }
 
-		$connectionParams['tablePrefix'] = $this->config->getValue('dbtableprefix', self::DEFAULT_DBTABLEPREFIX);
-		$connectionParams['sqlite.journal_mode'] = $this->config->getValue('sqlite.journal_mode', 'WAL');
+        $connectionParams['tablePrefix'] = $this->config->getValue('dbtableprefix', self::DEFAULT_DBTABLEPREFIX);
+        $connectionParams['sqlite.journal_mode'] = $this->config->getValue('sqlite.journal_mode', 'WAL');
 
-		//additional driver options, eg. for mysql ssl
-		$driverOptions = $this->config->getValue('dbdriveroptions', null);
-		if ($driverOptions) {
-			$connectionParams['driverOptions'] = $driverOptions;
-		}
+        //additional driver options, eg. for mysql ssl
+        $driverOptions = $this->config->getValue('dbdriveroptions', null);
+        if ($driverOptions) {
+            $connectionParams['driverOptions'] = $driverOptions;
+        }
 
-		// set default table creation options
-		$connectionParams['defaultTableOptions'] = [
-			'collate' => 'utf8_bin',
-			'tablePrefix' => $connectionParams['tablePrefix']
-		];
+        // set default table creation options
+        $connectionParams['defaultTableOptions'] = [
+            'collate' => 'utf8_bin',
+            'tablePrefix' => $connectionParams['tablePrefix']
+        ];
 
-		if ($this->config->getValue('mysql.utf8mb4', false)) {
-			$connectionParams['defaultTableOptions'] = [
-				'collate' => 'utf8mb4_bin',
-				'charset' => 'utf8mb4',
-				'row_format' => 'compressed',
-				'tablePrefix' => $connectionParams['tablePrefix']
-			];
-		}
+        if ($this->config->getValue('mysql.utf8mb4', false)) {
+            $connectionParams['defaultTableOptions'] = [
+                'collate' => 'utf8mb4_bin',
+                'charset' => 'utf8mb4',
+                'row_format' => 'compressed',
+                'tablePrefix' => $connectionParams['tablePrefix']
+            ];
+        }
 
-		return $connectionParams;
-	}
+        return $connectionParams;
+    }
 
-	/**
-	 * @param string $host
-	 * @return array
-	 */
-	protected function splitHostFromPortAndSocket($host): array {
-		$params = [
-			'host' => $host,
-		];
+    /**
+     * @param string $host
+     * @return array
+     */
+    protected function splitHostFromPortAndSocket($host): array {
+        $params = [
+            'host' => $host,
+        ];
 
-		$matches = [];
-		if (preg_match('/^(.*):([^\]:]+)$/', $host, $matches)) {
-			// Host variable carries a port or socket.
-			$params['host'] = $matches[1];
-			if (is_numeric($matches[2])) {
-				$params['port'] = (int) $matches[2];
-			} else {
-				$params['unix_socket'] = $matches[2];
-			}
-		}
+        $matches = [];
+        if (preg_match('/^(.*):([^\]:]+)$/', $host, $matches)) {
+            // Host variable carries a port or socket.
+            $params['host'] = $matches[1];
+            if (is_numeric($matches[2])) {
+                $params['port'] = (int) $matches[2];
+            } else {
+                $params['unix_socket'] = $matches[2];
+            }
+        }
 
-		return $params;
-	}
+        return $params;
+    }
 }

core/Migrations/Version15000Date20180926101451.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 		$schema = $schemaClosure();
 
 		$table = $schema->getTable('authtoken');
-		$table->addColumn('password_invalid','boolean', [
+		$table->addColumn('password_invalid', 'boolean', [
 			'notnull' => true,
 			'default' => false,
 		]);

Indentation +18 added lines, -18 removed lines

@@ -33,23 +33,23 @@
 
 class Version15000Date20180926101451 extends SimpleMigrationStep {
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 * @return null|ISchemaWrapper
-	 */
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
-		/** @var ISchemaWrapper $schema */
-		$schema = $schemaClosure();
-
-		$table = $schema->getTable('authtoken');
-		$table->addColumn('password_invalid','boolean', [
-			'notnull' => true,
-			'default' => false,
-		]);
-
-		return $schema;
-	}
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     * @return null|ISchemaWrapper
+     */
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
+        /** @var ISchemaWrapper $schema */
+        $schema = $schemaClosure();
+
+        $table = $schema->getTable('authtoken');
+        $table->addColumn('password_invalid','boolean', [
+            'notnull' => true,
+            'default' => false,
+        ]);
+
+        return $schema;
+    }
 
 }

lib/public/Authentication/TwoFactorAuth/IPersonalProviderSettings.php

Indentation +6 added lines, -6 removed lines

@@ -33,11 +33,11 @@
  */
 interface IPersonalProviderSettings {
 
-	/**
-	 * @return Template
-	 *
-	 * @since 15.0.0
-	 */
-	public function getBody(): Template;
+    /**
+     * @return Template
+     *
+     * @since 15.0.0
+     */
+    public function getBody(): Template;
 
 }