nextcloud / server

tests/Core/Command/Config/System/CastHelperTest.php

Indentation +44 added lines, -44 removed lines

@@ -10,60 +10,60 @@
 use Test\TestCase;
 
 class CastHelperTest extends TestCase {
-	private CastHelper $castHelper;
+    private CastHelper $castHelper;
 
-	protected function setUp(): void {
-		parent::setUp();
-		$this->castHelper = new CastHelper();
-	}
+    protected function setUp(): void {
+        parent::setUp();
+        $this->castHelper = new CastHelper();
+    }
 
-	public static function castValueProvider(): array {
-		return [
-			[null, 'string', ['value' => '', 'readable-value' => 'empty string']],
+    public static function castValueProvider(): array {
+        return [
+            [null, 'string', ['value' => '', 'readable-value' => 'empty string']],
 
-			['abc', 'string', ['value' => 'abc', 'readable-value' => 'string abc']],
+            ['abc', 'string', ['value' => 'abc', 'readable-value' => 'string abc']],
 
-			['123', 'integer', ['value' => 123, 'readable-value' => 'integer 123']],
-			['456', 'int', ['value' => 456, 'readable-value' => 'integer 456']],
+            ['123', 'integer', ['value' => 123, 'readable-value' => 'integer 123']],
+            ['456', 'int', ['value' => 456, 'readable-value' => 'integer 456']],
 
-			['2.25', 'double', ['value' => 2.25, 'readable-value' => 'double 2.25']],
-			['0.5', 'float', ['value' => 0.5, 'readable-value' => 'double 0.5']],
+            ['2.25', 'double', ['value' => 2.25, 'readable-value' => 'double 2.25']],
+            ['0.5', 'float', ['value' => 0.5, 'readable-value' => 'double 0.5']],
 
-			['', 'null', ['value' => null, 'readable-value' => 'null']],
+            ['', 'null', ['value' => null, 'readable-value' => 'null']],
 
-			['true', 'boolean', ['value' => true, 'readable-value' => 'boolean true']],
-			['false', 'bool', ['value' => false, 'readable-value' => 'boolean false']],
-		];
-	}
+            ['true', 'boolean', ['value' => true, 'readable-value' => 'boolean true']],
+            ['false', 'bool', ['value' => false, 'readable-value' => 'boolean false']],
+        ];
+    }
 
-	/**
-	 * @dataProvider castValueProvider
-	 */
-	public function testCastValue($value, $type, $expectedValue): void {
-		$this->assertSame(
-			$expectedValue,
-			$this->castHelper->castValue($value, $type)
-		);
-	}
+    /**
+     * @dataProvider castValueProvider
+     */
+    public function testCastValue($value, $type, $expectedValue): void {
+        $this->assertSame(
+            $expectedValue,
+            $this->castHelper->castValue($value, $type)
+        );
+    }
 
-	public static function castValueInvalidProvider(): array {
-		return [
-			['123', 'foobar'],
+    public static function castValueInvalidProvider(): array {
+        return [
+            ['123', 'foobar'],
 
-			[null, 'integer'],
-			['abc', 'integer'],
-			['76ggg', 'double'],
-			['true', 'float'],
-			['foobar', 'boolean'],
-		];
-	}
+            [null, 'integer'],
+            ['abc', 'integer'],
+            ['76ggg', 'double'],
+            ['true', 'float'],
+            ['foobar', 'boolean'],
+        ];
+    }
 
-	/**
-	 * @dataProvider castValueInvalidProvider
-	 */
-	public function testCastValueInvalid($value, $type): void {
-		$this->expectException(\InvalidArgumentException::class);
+    /**
+     * @dataProvider castValueInvalidProvider
+     */
+    public function testCastValueInvalid($value, $type): void {
+        $this->expectException(\InvalidArgumentException::class);
 
-		$this->castHelper->castValue($value, $type);
-	}
+        $this->castHelper->castValue($value, $type);
+    }
 }

tests/Core/Command/Config/System/SetConfigTest.php

Indentation +98 added lines, -98 removed lines

@@ -15,102 +15,102 @@
 use Test\TestCase;
 
 class SetConfigTest extends TestCase {
-	/** @var \PHPUnit\Framework\MockObject\MockObject */
-	protected $systemConfig;
-
-	/** @var \PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleInput;
-	/** @var \PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleOutput;
-
-	/** @var \Symfony\Component\Console\Command\Command */
-	protected $command;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$systemConfig = $this->systemConfig = $this->getMockBuilder(SystemConfig::class)
-			->disableOriginalConstructor()
-			->getMock();
-		$this->consoleInput = $this->getMockBuilder(InputInterface::class)->getMock();
-		$this->consoleOutput = $this->getMockBuilder(OutputInterface::class)->getMock();
-
-		/** @var \OC\SystemConfig $systemConfig */
-		$this->command = new SetConfig($systemConfig, new CastHelper());
-	}
-
-
-	public static function dataTest() {
-		return [
-			[['name'], 'newvalue', null, 'newvalue'],
-			[['a', 'b', 'c'], 'foobar', null, ['b' => ['c' => 'foobar']]],
-			[['a', 'b', 'c'], 'foobar', ['b' => ['d' => 'barfoo']], ['b' => ['d' => 'barfoo', 'c' => 'foobar']]],
-		];
-	}
-
-	/**
-	 * @dataProvider dataTest
-	 *
-	 * @param array $configNames
-	 * @param string $newValue
-	 * @param mixed $existingData
-	 * @param mixed $expectedValue
-	 */
-	public function testSet($configNames, $newValue, $existingData, $expectedValue): void {
-		$this->systemConfig->expects($this->once())
-			->method('setValue')
-			->with($configNames[0], $expectedValue);
-		$this->systemConfig->method('getValue')
-			->with($configNames[0])
-			->willReturn($existingData);
-
-		$this->consoleInput->expects($this->once())
-			->method('getArgument')
-			->with('name')
-			->willReturn($configNames);
-		$this->consoleInput->method('getOption')
-			->willReturnMap([
-				['value', $newValue],
-				['type', 'string'],
-			]);
-
-		$this->invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-	}
-
-	public static function setUpdateOnlyProvider(): array {
-		return [
-			[['name'], null],
-			[['a', 'b', 'c'], null],
-			[['a', 'b', 'c'], ['b' => 'foobar']],
-			[['a', 'b', 'c'], ['b' => ['d' => 'foobar']]],
-		];
-	}
-
-	/**
-	 * @dataProvider setUpdateOnlyProvider
-	 */
-	public function testSetUpdateOnly($configNames, $existingData): void {
-		$this->expectException(\UnexpectedValueException::class);
-
-		$this->systemConfig->expects($this->never())
-			->method('setValue');
-		$this->systemConfig->method('getValue')
-			->with($configNames[0])
-			->willReturn($existingData);
-		$this->systemConfig->method('getKeys')
-			->willReturn($existingData ? $configNames[0] : []);
-
-		$this->consoleInput->expects($this->once())
-			->method('getArgument')
-			->with('name')
-			->willReturn($configNames);
-		$this->consoleInput->method('getOption')
-			->willReturnMap([
-				['value', 'foobar'],
-				['type', 'string'],
-				['update-only', true],
-			]);
-
-		$this->invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-	}
+    /** @var \PHPUnit\Framework\MockObject\MockObject */
+    protected $systemConfig;
+
+    /** @var \PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleInput;
+    /** @var \PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleOutput;
+
+    /** @var \Symfony\Component\Console\Command\Command */
+    protected $command;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $systemConfig = $this->systemConfig = $this->getMockBuilder(SystemConfig::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->consoleInput = $this->getMockBuilder(InputInterface::class)->getMock();
+        $this->consoleOutput = $this->getMockBuilder(OutputInterface::class)->getMock();
+
+        /** @var \OC\SystemConfig $systemConfig */
+        $this->command = new SetConfig($systemConfig, new CastHelper());
+    }
+
+
+    public static function dataTest() {
+        return [
+            [['name'], 'newvalue', null, 'newvalue'],
+            [['a', 'b', 'c'], 'foobar', null, ['b' => ['c' => 'foobar']]],
+            [['a', 'b', 'c'], 'foobar', ['b' => ['d' => 'barfoo']], ['b' => ['d' => 'barfoo', 'c' => 'foobar']]],
+        ];
+    }
+
+    /**
+     * @dataProvider dataTest
+     *
+     * @param array $configNames
+     * @param string $newValue
+     * @param mixed $existingData
+     * @param mixed $expectedValue
+     */
+    public function testSet($configNames, $newValue, $existingData, $expectedValue): void {
+        $this->systemConfig->expects($this->once())
+            ->method('setValue')
+            ->with($configNames[0], $expectedValue);
+        $this->systemConfig->method('getValue')
+            ->with($configNames[0])
+            ->willReturn($existingData);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getArgument')
+            ->with('name')
+            ->willReturn($configNames);
+        $this->consoleInput->method('getOption')
+            ->willReturnMap([
+                ['value', $newValue],
+                ['type', 'string'],
+            ]);
+
+        $this->invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+    }
+
+    public static function setUpdateOnlyProvider(): array {
+        return [
+            [['name'], null],
+            [['a', 'b', 'c'], null],
+            [['a', 'b', 'c'], ['b' => 'foobar']],
+            [['a', 'b', 'c'], ['b' => ['d' => 'foobar']]],
+        ];
+    }
+
+    /**
+     * @dataProvider setUpdateOnlyProvider
+     */
+    public function testSetUpdateOnly($configNames, $existingData): void {
+        $this->expectException(\UnexpectedValueException::class);
+
+        $this->systemConfig->expects($this->never())
+            ->method('setValue');
+        $this->systemConfig->method('getValue')
+            ->with($configNames[0])
+            ->willReturn($existingData);
+        $this->systemConfig->method('getKeys')
+            ->willReturn($existingData ? $configNames[0] : []);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getArgument')
+            ->with('name')
+            ->willReturn($configNames);
+        $this->consoleInput->method('getOption')
+            ->willReturnMap([
+                ['value', 'foobar'],
+                ['type', 'string'],
+                ['update-only', true],
+            ]);
+
+        $this->invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+    }
 }

core/register_command.php

Indentation +117 added lines, -117 removed lines

@@ -122,141 +122,141 @@
 $config = Server::get(IConfig::class);
 
 if ($config->getSystemValueBool('installed', false)) {
-	$application->add(Server::get(Disable::class));
-	$application->add(Server::get(Enable::class));
-	$application->add(Server::get(Install::class));
-	$application->add(Server::get(GetPath::class));
-	$application->add(Server::get(ListApps::class));
-	$application->add(Server::get(Remove::class));
-	$application->add(Server::get(Update::class));
+    $application->add(Server::get(Disable::class));
+    $application->add(Server::get(Enable::class));
+    $application->add(Server::get(Install::class));
+    $application->add(Server::get(GetPath::class));
+    $application->add(Server::get(ListApps::class));
+    $application->add(Server::get(Remove::class));
+    $application->add(Server::get(Update::class));
 
-	$application->add(Server::get(Cleanup::class));
-	$application->add(Server::get(Enforce::class));
-	$application->add(Server::get(Command\TwoFactorAuth\Enable::class));
-	$application->add(Server::get(Command\TwoFactorAuth\Disable::class));
-	$application->add(Server::get(State::class));
+    $application->add(Server::get(Cleanup::class));
+    $application->add(Server::get(Enforce::class));
+    $application->add(Server::get(Command\TwoFactorAuth\Enable::class));
+    $application->add(Server::get(Command\TwoFactorAuth\Disable::class));
+    $application->add(Server::get(State::class));
 
-	$application->add(Server::get(Mode::class));
-	$application->add(Server::get(Job::class));
-	$application->add(Server::get(ListCommand::class));
-	$application->add(Server::get(Delete::class));
-	$application->add(Server::get(JobWorker::class));
+    $application->add(Server::get(Mode::class));
+    $application->add(Server::get(Job::class));
+    $application->add(Server::get(ListCommand::class));
+    $application->add(Server::get(Delete::class));
+    $application->add(Server::get(JobWorker::class));
 
-	$application->add(Server::get(Test::class));
+    $application->add(Server::get(Test::class));
 
-	$application->add(Server::get(DeleteConfig::class));
-	$application->add(Server::get(GetConfig::class));
-	$application->add(Server::get(SetConfig::class));
-	$application->add(Server::get(Import::class));
-	$application->add(Server::get(ListConfigs::class));
-	$application->add(Server::get(Command\Config\System\DeleteConfig::class));
-	$application->add(Server::get(Command\Config\System\GetConfig::class));
-	$application->add(Server::get(Command\Config\System\SetConfig::class));
+    $application->add(Server::get(DeleteConfig::class));
+    $application->add(Server::get(GetConfig::class));
+    $application->add(Server::get(SetConfig::class));
+    $application->add(Server::get(Import::class));
+    $application->add(Server::get(ListConfigs::class));
+    $application->add(Server::get(Command\Config\System\DeleteConfig::class));
+    $application->add(Server::get(Command\Config\System\GetConfig::class));
+    $application->add(Server::get(Command\Config\System\SetConfig::class));
 
-	$application->add(Server::get(File::class));
-	$application->add(Server::get(Space::class));
-	$application->add(Server::get(Storage::class));
-	$application->add(Server::get(Storages::class));
+    $application->add(Server::get(File::class));
+    $application->add(Server::get(Space::class));
+    $application->add(Server::get(Storage::class));
+    $application->add(Server::get(Storages::class));
 
-	$application->add(Server::get(ConvertType::class));
-	$application->add(Server::get(ConvertMysqlToMB4::class));
-	$application->add(Server::get(ConvertFilecacheBigInt::class));
-	$application->add(Server::get(AddMissingColumns::class));
-	$application->add(Server::get(AddMissingIndices::class));
-	$application->add(Server::get(AddMissingPrimaryKeys::class));
-	$application->add(Server::get(ExpectedSchema::class));
-	$application->add(Server::get(ExportSchema::class));
+    $application->add(Server::get(ConvertType::class));
+    $application->add(Server::get(ConvertMysqlToMB4::class));
+    $application->add(Server::get(ConvertFilecacheBigInt::class));
+    $application->add(Server::get(AddMissingColumns::class));
+    $application->add(Server::get(AddMissingIndices::class));
+    $application->add(Server::get(AddMissingPrimaryKeys::class));
+    $application->add(Server::get(ExpectedSchema::class));
+    $application->add(Server::get(ExportSchema::class));
 
-	$application->add(Server::get(GenerateMetadataCommand::class));
-	$application->add(Server::get(PreviewCommand::class));
-	if ($config->getSystemValueBool('debug', false)) {
-		$application->add(Server::get(StatusCommand::class));
-		$application->add(Server::get(MigrateCommand::class));
-		$application->add(Server::get(GenerateCommand::class));
-		$application->add(Server::get(ExecuteCommand::class));
-	}
+    $application->add(Server::get(GenerateMetadataCommand::class));
+    $application->add(Server::get(PreviewCommand::class));
+    if ($config->getSystemValueBool('debug', false)) {
+        $application->add(Server::get(StatusCommand::class));
+        $application->add(Server::get(MigrateCommand::class));
+        $application->add(Server::get(GenerateCommand::class));
+        $application->add(Server::get(ExecuteCommand::class));
+    }
 
-	$application->add(Server::get(Command\Encryption\Disable::class));
-	$application->add(Server::get(Command\Encryption\Enable::class));
-	$application->add(Server::get(ListModules::class));
-	$application->add(Server::get(SetDefaultModule::class));
-	$application->add(Server::get(Command\Encryption\Status::class));
-	$application->add(Server::get(EncryptAll::class));
-	$application->add(Server::get(DecryptAll::class));
+    $application->add(Server::get(Command\Encryption\Disable::class));
+    $application->add(Server::get(Command\Encryption\Enable::class));
+    $application->add(Server::get(ListModules::class));
+    $application->add(Server::get(SetDefaultModule::class));
+    $application->add(Server::get(Command\Encryption\Status::class));
+    $application->add(Server::get(EncryptAll::class));
+    $application->add(Server::get(DecryptAll::class));
 
-	$application->add(Server::get(Manage::class));
-	$application->add(Server::get(Command\Log\File::class));
+    $application->add(Server::get(Manage::class));
+    $application->add(Server::get(Command\Log\File::class));
 
-	$application->add(Server::get(ChangeKeyStorageRoot::class));
-	$application->add(Server::get(ShowKeyStorageRoot::class));
-	$application->add(Server::get(MigrateKeyStorage::class));
+    $application->add(Server::get(ChangeKeyStorageRoot::class));
+    $application->add(Server::get(ShowKeyStorageRoot::class));
+    $application->add(Server::get(MigrateKeyStorage::class));
 
-	$application->add(Server::get(DataFingerprint::class));
-	$application->add(Server::get(UpdateDB::class));
-	$application->add(Server::get(UpdateJS::class));
-	$application->add(Server::get(Command\Maintenance\Mode::class));
-	$application->add(Server::get(UpdateHtaccess::class));
-	$application->add(Server::get(UpdateTheme::class));
+    $application->add(Server::get(DataFingerprint::class));
+    $application->add(Server::get(UpdateDB::class));
+    $application->add(Server::get(UpdateJS::class));
+    $application->add(Server::get(Command\Maintenance\Mode::class));
+    $application->add(Server::get(UpdateHtaccess::class));
+    $application->add(Server::get(UpdateTheme::class));
 
-	$application->add(Server::get(Upgrade::class));
-	$application->add(Server::get(Repair::class));
-	$application->add(Server::get(RepairShareOwnership::class));
+    $application->add(Server::get(Upgrade::class));
+    $application->add(Server::get(Repair::class));
+    $application->add(Server::get(RepairShareOwnership::class));
 
-	$application->add(Server::get(Command\Preview\Cleanup::class));
-	$application->add(Server::get(Generate::class));
-	$application->add(Server::get(Command\Preview\Repair::class));
-	$application->add(Server::get(ResetRenderedTexts::class));
+    $application->add(Server::get(Command\Preview\Cleanup::class));
+    $application->add(Server::get(Generate::class));
+    $application->add(Server::get(Command\Preview\Repair::class));
+    $application->add(Server::get(ResetRenderedTexts::class));
 
-	$application->add(Server::get(Add::class));
-	$application->add(Server::get(Command\User\Delete::class));
-	$application->add(Server::get(Command\User\Disable::class));
-	$application->add(Server::get(Command\User\Enable::class));
-	$application->add(Server::get(LastSeen::class));
-	$application->add(Server::get(Report::class));
-	$application->add(Server::get(ResetPassword::class));
-	$application->add(Server::get(Setting::class));
-	$application->add(Server::get(Profile::class));
-	$application->add(Server::get(Command\User\ListCommand::class));
-	$application->add(Server::get(ClearGeneratedAvatarCacheCommand::class));
-	$application->add(Server::get(Info::class));
-	$application->add(Server::get(SyncAccountDataCommand::class));
-	$application->add(Server::get(Command\User\AuthTokens\Add::class));
-	$application->add(Server::get(Command\User\AuthTokens\ListCommand::class));
-	$application->add(Server::get(Command\User\AuthTokens\Delete::class));
-	$application->add(Server::get(Verify::class));
-	$application->add(Server::get(Welcome::class));
+    $application->add(Server::get(Add::class));
+    $application->add(Server::get(Command\User\Delete::class));
+    $application->add(Server::get(Command\User\Disable::class));
+    $application->add(Server::get(Command\User\Enable::class));
+    $application->add(Server::get(LastSeen::class));
+    $application->add(Server::get(Report::class));
+    $application->add(Server::get(ResetPassword::class));
+    $application->add(Server::get(Setting::class));
+    $application->add(Server::get(Profile::class));
+    $application->add(Server::get(Command\User\ListCommand::class));
+    $application->add(Server::get(ClearGeneratedAvatarCacheCommand::class));
+    $application->add(Server::get(Info::class));
+    $application->add(Server::get(SyncAccountDataCommand::class));
+    $application->add(Server::get(Command\User\AuthTokens\Add::class));
+    $application->add(Server::get(Command\User\AuthTokens\ListCommand::class));
+    $application->add(Server::get(Command\User\AuthTokens\Delete::class));
+    $application->add(Server::get(Verify::class));
+    $application->add(Server::get(Welcome::class));
 
-	$application->add(Server::get(Command\Group\Add::class));
-	$application->add(Server::get(Command\Group\Delete::class));
-	$application->add(Server::get(Command\Group\ListCommand::class));
-	$application->add(Server::get(AddUser::class));
-	$application->add(Server::get(RemoveUser::class));
-	$application->add(Server::get(Command\Group\Info::class));
+    $application->add(Server::get(Command\Group\Add::class));
+    $application->add(Server::get(Command\Group\Delete::class));
+    $application->add(Server::get(Command\Group\ListCommand::class));
+    $application->add(Server::get(AddUser::class));
+    $application->add(Server::get(RemoveUser::class));
+    $application->add(Server::get(Command\Group\Info::class));
 
-	$application->add(Server::get(Command\SystemTag\ListCommand::class));
-	$application->add(Server::get(Command\SystemTag\Delete::class));
-	$application->add(Server::get(Command\SystemTag\Add::class));
-	$application->add(Server::get(Edit::class));
+    $application->add(Server::get(Command\SystemTag\ListCommand::class));
+    $application->add(Server::get(Command\SystemTag\Delete::class));
+    $application->add(Server::get(Command\SystemTag\Add::class));
+    $application->add(Server::get(Edit::class));
 
-	$application->add(Server::get(ListCertificates::class));
-	$application->add(Server::get(ExportCertificates::class));
-	$application->add(Server::get(ImportCertificate::class));
-	$application->add(Server::get(RemoveCertificate::class));
-	$application->add(Server::get(BruteforceAttempts::class));
-	$application->add(Server::get(BruteforceResetAttempts::class));
-	$application->add(Server::get(SetupChecks::class));
-	$application->add(Server::get(Get::class));
+    $application->add(Server::get(ListCertificates::class));
+    $application->add(Server::get(ExportCertificates::class));
+    $application->add(Server::get(ImportCertificate::class));
+    $application->add(Server::get(RemoveCertificate::class));
+    $application->add(Server::get(BruteforceAttempts::class));
+    $application->add(Server::get(BruteforceResetAttempts::class));
+    $application->add(Server::get(SetupChecks::class));
+    $application->add(Server::get(Get::class));
 
-	$application->add(Server::get(GetCommand::class));
-	$application->add(Server::get(EnabledCommand::class));
-	$application->add(Server::get(Command\TaskProcessing\ListCommand::class));
-	$application->add(Server::get(Statistics::class));
+    $application->add(Server::get(GetCommand::class));
+    $application->add(Server::get(EnabledCommand::class));
+    $application->add(Server::get(Command\TaskProcessing\ListCommand::class));
+    $application->add(Server::get(Statistics::class));
 
-	$application->add(Server::get(RedisCommand::class));
-	$application->add(Server::get(DistributedClear::class));
-	$application->add(Server::get(DistributedDelete::class));
-	$application->add(Server::get(DistributedGet::class));
-	$application->add(Server::get(DistributedSet::class));
+    $application->add(Server::get(RedisCommand::class));
+    $application->add(Server::get(DistributedClear::class));
+    $application->add(Server::get(DistributedDelete::class));
+    $application->add(Server::get(DistributedGet::class));
+    $application->add(Server::get(DistributedSet::class));
 } else {
-	$application->add(Server::get(Command\Maintenance\Install::class));
+    $application->add(Server::get(Command\Maintenance\Install::class));
 }

core/Command/Memcache/DistributedDelete.php

Indentation +23 added lines, -23 removed lines

@@ -15,29 +15,29 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class DistributedDelete extends Base {
-	public function __construct(
-		protected ICacheFactory $cacheFactory,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ICacheFactory $cacheFactory,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('memcache:distributed:delete')
-			->setDescription('Delete a value in the distributed memcache')
-			->addArgument('key', InputArgument::REQUIRED, 'The key to delete');
-		parent::configure();
-	}
+    protected function configure(): void {
+        $this
+            ->setName('memcache:distributed:delete')
+            ->setDescription('Delete a value in the distributed memcache')
+            ->addArgument('key', InputArgument::REQUIRED, 'The key to delete');
+        parent::configure();
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$cache = $this->cacheFactory->createDistributed();
-		$key = $input->getArgument('key');
-		if ($cache->remove($key)) {
-			$output->writeln('<info>Distributed cache key <info>' . $key . '</info> deleted</info>');
-			return 0;
-		} else {
-			$output->writeln('<error>Failed to delete cache key ' . $key . '</error>');
-			return 1;
-		}
-	}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $cache = $this->cacheFactory->createDistributed();
+        $key = $input->getArgument('key');
+        if ($cache->remove($key)) {
+            $output->writeln('<info>Distributed cache key <info>' . $key . '</info> deleted</info>');
+            return 0;
+        } else {
+            $output->writeln('<error>Failed to delete cache key ' . $key . '</error>');
+            return 1;
+        }
+    }
 }

core/Command/Memcache/DistributedClear.php

Indentation +27 added lines, -27 removed lines

@@ -15,33 +15,33 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class DistributedClear extends Base {
-	public function __construct(
-		protected ICacheFactory $cacheFactory,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ICacheFactory $cacheFactory,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('memcache:distributed:clear')
-			->setDescription('Clear values from the distributed memcache')
-			->addOption('prefix', null, InputOption::VALUE_REQUIRED, 'Only remove keys matching the prefix');
-		parent::configure();
-	}
+    protected function configure(): void {
+        $this
+            ->setName('memcache:distributed:clear')
+            ->setDescription('Clear values from the distributed memcache')
+            ->addOption('prefix', null, InputOption::VALUE_REQUIRED, 'Only remove keys matching the prefix');
+        parent::configure();
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$cache = $this->cacheFactory->createDistributed();
-		$prefix = $input->getOption('prefix');
-		if ($cache->clear($prefix)) {
-			if ($prefix) {
-				$output->writeln('<info>Distributed cache matching prefix ' . $prefix . ' cleared</info>');
-			} else {
-				$output->writeln('<info>Distributed cache cleared</info>');
-			}
-			return 0;
-		} else {
-			$output->writeln('<error>Failed to clear cache</error>');
-			return 1;
-		}
-	}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $cache = $this->cacheFactory->createDistributed();
+        $prefix = $input->getOption('prefix');
+        if ($cache->clear($prefix)) {
+            if ($prefix) {
+                $output->writeln('<info>Distributed cache matching prefix ' . $prefix . ' cleared</info>');
+            } else {
+                $output->writeln('<info>Distributed cache cleared</info>');
+            }
+            return 0;
+        } else {
+            $output->writeln('<error>Failed to clear cache</error>');
+            return 1;
+        }
+    }
 }

core/Command/Memcache/DistributedGet.php

Indentation +19 added lines, -19 removed lines

@@ -15,26 +15,26 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class DistributedGet extends Base {
-	public function __construct(
-		protected ICacheFactory $cacheFactory,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ICacheFactory $cacheFactory,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('memcache:distributed:get')
-			->setDescription('Get a value from the distributed memcache')
-			->addArgument('key', InputArgument::REQUIRED, 'The key to retrieve');
-		parent::configure();
-	}
+    protected function configure(): void {
+        $this
+            ->setName('memcache:distributed:get')
+            ->setDescription('Get a value from the distributed memcache')
+            ->addArgument('key', InputArgument::REQUIRED, 'The key to retrieve');
+        parent::configure();
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$cache = $this->cacheFactory->createDistributed();
-		$key = $input->getArgument('key');
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $cache = $this->cacheFactory->createDistributed();
+        $key = $input->getArgument('key');
 
-		$value = $cache->get($key);
-		$this->writeMixedInOutputFormat($input, $output, $value);
-		return 0;
-	}
+        $value = $cache->get($key);
+        $this->writeMixedInOutputFormat($input, $output, $value);
+        return 0;
+    }
 }

core/Command/Memcache/DistributedSet.php

Indentation +35 added lines, -35 removed lines

@@ -17,41 +17,41 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class DistributedSet extends Base {
-	public function __construct(
-		protected ICacheFactory $cacheFactory,
-		private CastHelper $castHelper,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected ICacheFactory $cacheFactory,
+        private CastHelper $castHelper,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('memcache:distributed:set')
-			->setDescription('Set a value in the distributed memcache')
-			->addArgument('key', InputArgument::REQUIRED, 'The key to set')
-			->addArgument('value', InputArgument::REQUIRED, 'The value to set')
-			->addOption(
-				'type',
-				null,
-				InputOption::VALUE_REQUIRED,
-				'Value type [string, integer, float, boolean, json, null]',
-				'string'
-			);
-		parent::configure();
-	}
+    protected function configure(): void {
+        $this
+            ->setName('memcache:distributed:set')
+            ->setDescription('Set a value in the distributed memcache')
+            ->addArgument('key', InputArgument::REQUIRED, 'The key to set')
+            ->addArgument('value', InputArgument::REQUIRED, 'The value to set')
+            ->addOption(
+                'type',
+                null,
+                InputOption::VALUE_REQUIRED,
+                'Value type [string, integer, float, boolean, json, null]',
+                'string'
+            );
+        parent::configure();
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$cache = $this->cacheFactory->createDistributed();
-		$key = $input->getArgument('key');
-		$value = $input->getArgument('value');
-		$type = $input->getOption('type');
-		['value' => $value, 'readable-value' => $readable] = $this->castHelper->castValue($value, $type);
-		if ($cache->set($key, $value)) {
-			$output->writeln('Distributed cache key <info>' . $key . '</info> set to <info>' . $readable . '</info>');
-			return 0;
-		} else {
-			$output->writeln('<error>Failed to set cache key ' . $key . '</error>');
-			return 1;
-		}
-	}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $cache = $this->cacheFactory->createDistributed();
+        $key = $input->getArgument('key');
+        $value = $input->getArgument('value');
+        $type = $input->getOption('type');
+        ['value' => $value, 'readable-value' => $readable] = $this->castHelper->castValue($value, $type);
+        if ($cache->set($key, $value)) {
+            $output->writeln('Distributed cache key <info>' . $key . '</info> set to <info>' . $readable . '</info>');
+            return 0;
+        } else {
+            $output->writeln('<error>Failed to set cache key ' . $key . '</error>');
+            return 1;
+        }
+    }
 }

core/Command/Config/System/SetConfig.php

Indentation +96 added lines, -96 removed lines

@@ -15,110 +15,110 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class SetConfig extends Base {
-	public function __construct(
-		SystemConfig $systemConfig,
-		private CastHelper $castHelper,
-	) {
-		parent::__construct($systemConfig);
-	}
+    public function __construct(
+        SystemConfig $systemConfig,
+        private CastHelper $castHelper,
+    ) {
+        parent::__construct($systemConfig);
+    }
 
-	protected function configure() {
-		parent::configure();
+    protected function configure() {
+        parent::configure();
 
-		$this
-			->setName('config:system:set')
-			->setDescription('Set a system config value')
-			->addArgument(
-				'name',
-				InputArgument::REQUIRED | InputArgument::IS_ARRAY,
-				'Name of the config parameter, specify multiple for array parameter'
-			)
-			->addOption(
-				'type',
-				null,
-				InputOption::VALUE_REQUIRED,
-				'Value type [string, integer, double, boolean]',
-				'string'
-			)
-			->addOption(
-				'value',
-				null,
-				InputOption::VALUE_REQUIRED,
-				'The new value of the config'
-			)
-			->addOption(
-				'update-only',
-				null,
-				InputOption::VALUE_NONE,
-				'Only updates the value, if it is not set before, it is not being added'
-			)
-		;
-	}
+        $this
+            ->setName('config:system:set')
+            ->setDescription('Set a system config value')
+            ->addArgument(
+                'name',
+                InputArgument::REQUIRED | InputArgument::IS_ARRAY,
+                'Name of the config parameter, specify multiple for array parameter'
+            )
+            ->addOption(
+                'type',
+                null,
+                InputOption::VALUE_REQUIRED,
+                'Value type [string, integer, double, boolean]',
+                'string'
+            )
+            ->addOption(
+                'value',
+                null,
+                InputOption::VALUE_REQUIRED,
+                'The new value of the config'
+            )
+            ->addOption(
+                'update-only',
+                null,
+                InputOption::VALUE_NONE,
+                'Only updates the value, if it is not set before, it is not being added'
+            )
+        ;
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$configNames = $input->getArgument('name');
-		$configName = $configNames[0];
-		$configValue = $this->castHelper->castValue($input->getOption('value'), $input->getOption('type'));
-		$updateOnly = $input->getOption('update-only');
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $configNames = $input->getArgument('name');
+        $configName = $configNames[0];
+        $configValue = $this->castHelper->castValue($input->getOption('value'), $input->getOption('type'));
+        $updateOnly = $input->getOption('update-only');
 
-		if (count($configNames) > 1) {
-			$existingValue = $this->systemConfig->getValue($configName);
+        if (count($configNames) > 1) {
+            $existingValue = $this->systemConfig->getValue($configName);
 
-			$newValue = $this->mergeArrayValue(
-				array_slice($configNames, 1), $existingValue, $configValue['value'], $updateOnly
-			);
+            $newValue = $this->mergeArrayValue(
+                array_slice($configNames, 1), $existingValue, $configValue['value'], $updateOnly
+            );
 
-			$this->systemConfig->setValue($configName, $newValue);
-		} else {
-			if ($updateOnly && !in_array($configName, $this->systemConfig->getKeys(), true)) {
-				throw new \UnexpectedValueException('Config parameter does not exist');
-			}
+            $this->systemConfig->setValue($configName, $newValue);
+        } else {
+            if ($updateOnly && !in_array($configName, $this->systemConfig->getKeys(), true)) {
+                throw new \UnexpectedValueException('Config parameter does not exist');
+            }
 
-			$this->systemConfig->setValue($configName, $configValue['value']);
-		}
+            $this->systemConfig->setValue($configName, $configValue['value']);
+        }
 
-		$output->writeln('<info>System config value ' . implode(' => ', $configNames) . ' set to ' . $configValue['readable-value'] . '</info>');
-		return 0;
-	}
+        $output->writeln('<info>System config value ' . implode(' => ', $configNames) . ' set to ' . $configValue['readable-value'] . '</info>');
+        return 0;
+    }
 
-	/**
-	 * @param array $configNames
-	 * @param mixed $existingValues
-	 * @param mixed $value
-	 * @param bool $updateOnly
-	 * @return array merged value
-	 * @throws \UnexpectedValueException
-	 */
-	protected function mergeArrayValue(array $configNames, $existingValues, $value, $updateOnly) {
-		$configName = array_shift($configNames);
-		if (!is_array($existingValues)) {
-			$existingValues = [];
-		}
-		if (!empty($configNames)) {
-			if (isset($existingValues[$configName])) {
-				$existingValue = $existingValues[$configName];
-			} else {
-				$existingValue = [];
-			}
-			$existingValues[$configName] = $this->mergeArrayValue($configNames, $existingValue, $value, $updateOnly);
-		} else {
-			if (!isset($existingValues[$configName]) && $updateOnly) {
-				throw new \UnexpectedValueException('Config parameter does not exist');
-			}
-			$existingValues[$configName] = $value;
-		}
-		return $existingValues;
-	}
+    /**
+     * @param array $configNames
+     * @param mixed $existingValues
+     * @param mixed $value
+     * @param bool $updateOnly
+     * @return array merged value
+     * @throws \UnexpectedValueException
+     */
+    protected function mergeArrayValue(array $configNames, $existingValues, $value, $updateOnly) {
+        $configName = array_shift($configNames);
+        if (!is_array($existingValues)) {
+            $existingValues = [];
+        }
+        if (!empty($configNames)) {
+            if (isset($existingValues[$configName])) {
+                $existingValue = $existingValues[$configName];
+            } else {
+                $existingValue = [];
+            }
+            $existingValues[$configName] = $this->mergeArrayValue($configNames, $existingValue, $value, $updateOnly);
+        } else {
+            if (!isset($existingValues[$configName]) && $updateOnly) {
+                throw new \UnexpectedValueException('Config parameter does not exist');
+            }
+            $existingValues[$configName] = $value;
+        }
+        return $existingValues;
+    }
 
-	/**
-	 * @param string $optionName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeOptionValues($optionName, CompletionContext $context) {
-		if ($optionName === 'type') {
-			return ['string', 'integer', 'double', 'boolean', 'json', 'null'];
-		}
-		return parent::completeOptionValues($optionName, $context);
-	}
+    /**
+     * @param string $optionName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeOptionValues($optionName, CompletionContext $context) {
+        if ($optionName === 'type') {
+            return ['string', 'integer', 'double', 'boolean', 'json', 'null'];
+        }
+        return parent::completeOptionValues($optionName, $context);
+    }
 }

core/Command/Config/System/CastHelper.php

Indentation +58 added lines, -58 removed lines

@@ -9,68 +9,68 @@
 namespace OC\Core\Command\Config\System;
 
 class CastHelper {
-	/**
-	 * @return array{value: mixed, readable-value: string}
-	 */
-	public function castValue(?string $value, string $type): array {
-		switch ($type) {
-			case 'integer':
-			case 'int':
-				if (!is_numeric($value)) {
-					throw new \InvalidArgumentException('Non-numeric value specified');
-				}
-				return [
-					'value' => (int)$value,
-					'readable-value' => 'integer ' . (int)$value,
-				];
+    /**
+     * @return array{value: mixed, readable-value: string}
+     */
+    public function castValue(?string $value, string $type): array {
+        switch ($type) {
+            case 'integer':
+            case 'int':
+                if (!is_numeric($value)) {
+                    throw new \InvalidArgumentException('Non-numeric value specified');
+                }
+                return [
+                    'value' => (int)$value,
+                    'readable-value' => 'integer ' . (int)$value,
+                ];
 
-			case 'double':
-			case 'float':
-				if (!is_numeric($value)) {
-					throw new \InvalidArgumentException('Non-numeric value specified');
-				}
-				return [
-					'value' => (float)$value,
-					'readable-value' => 'double ' . (float)$value,
-				];
+            case 'double':
+            case 'float':
+                if (!is_numeric($value)) {
+                    throw new \InvalidArgumentException('Non-numeric value specified');
+                }
+                return [
+                    'value' => (float)$value,
+                    'readable-value' => 'double ' . (float)$value,
+                ];
 
-			case 'boolean':
-			case 'bool':
-				$value = strtolower($value);
-				return match ($value) {
-					'true' => [
-						'value' => true,
-						'readable-value' => 'boolean ' . $value,
-					],
-					'false' => [
-						'value' => false,
-						'readable-value' => 'boolean ' . $value,
-					],
-					default => throw new \InvalidArgumentException('Unable to parse value as boolean'),
-				};
+            case 'boolean':
+            case 'bool':
+                $value = strtolower($value);
+                return match ($value) {
+                    'true' => [
+                        'value' => true,
+                        'readable-value' => 'boolean ' . $value,
+                    ],
+                    'false' => [
+                        'value' => false,
+                        'readable-value' => 'boolean ' . $value,
+                    ],
+                    default => throw new \InvalidArgumentException('Unable to parse value as boolean'),
+                };
 
-			case 'null':
-				return [
-					'value' => null,
-					'readable-value' => 'null',
-				];
+            case 'null':
+                return [
+                    'value' => null,
+                    'readable-value' => 'null',
+                ];
 
-			case 'string':
-				$value = (string)$value;
-				return [
-					'value' => $value,
-					'readable-value' => ($value === '') ? 'empty string' : 'string ' . $value,
-				];
+            case 'string':
+                $value = (string)$value;
+                return [
+                    'value' => $value,
+                    'readable-value' => ($value === '') ? 'empty string' : 'string ' . $value,
+                ];
 
-			case 'json':
-				$value = json_decode($value, true);
-				return [
-					'value' => $value,
-					'readable-value' => 'json ' . json_encode($value),
-				];
+            case 'json':
+                $value = json_decode($value, true);
+                return [
+                    'value' => $value,
+                    'readable-value' => 'json ' . json_encode($value),
+                ];
 
-			default:
-				throw new \InvalidArgumentException('Invalid type');
-		}
-	}
+            default:
+                throw new \InvalidArgumentException('Invalid type');
+        }
+    }
 }