nextcloud / server

apps/dav/appinfo/v2/direct.php

Indentation +8 added lines, -8 removed lines

@@ -24,7 +24,7 @@ 
 
 // no php execution timeout for webdav
 if (strpos(@ini_get('disable_functions'), 'set_time_limit') === false) {
-	@set_time_limit(0);
+    @set_time_limit(0);
 }
 ignore_user_abort(true);
 
@@ -35,13 +35,13 @@ 
 
 $serverFactory = new \OCA\DAV\Direct\ServerFactory(\OC::$server->getConfig());
 $server = $serverFactory->createServer(
-	$baseuri,
-	$requestUri,
-	\OC::$server->getRootFolder(),
-	\OC::$server->query(\OCA\DAV\Db\DirectMapper::class),
-	\OC::$server->query(\OCP\AppFramework\Utility\ITimeFactory::class),
-	\OC::$server->getBruteForceThrottler(),
-	\OC::$server->getRequest()
+    $baseuri,
+    $requestUri,
+    \OC::$server->getRootFolder(),
+    \OC::$server->query(\OCA\DAV\Db\DirectMapper::class),
+    \OC::$server->query(\OCP\AppFramework\Utility\ITimeFactory::class),
+    \OC::$server->getBruteForceThrottler(),
+    \OC::$server->getRequest()
 );
 
 $server->exec();

lib/private/Log/Errorlog.php

Indentation +9 added lines, -9 removed lines

@@ -29,14 +29,14 @@
 
 class Errorlog implements IWriter {
 
-	/**
-	 * write a message in the log
-	 * @param string $app
-	 * @param string $message
-	 * @param int $level
-	 */
-	public function write(string $app, $message, int $level) {
-		error_log('[owncloud]['.$app.']['.$level.'] '.$message);
-	}
+    /**
+     * write a message in the log
+     * @param string $app
+     * @param string $message
+     * @param int $level
+     */
+    public function write(string $app, $message, int $level) {
+        error_log('[owncloud]['.$app.']['.$level.'] '.$message);
+    }
 }
 

lib/public/Log/IWriter.php

Indentation +4 added lines, -4 removed lines

@@ -30,8 +30,8 @@
  * @since 14.0.0
  */
 interface IWriter {
-	/**
-	 * @since 14.0.0
-	 */
-	public function write(string $app, $message, int $level);
+    /**
+     * @since 14.0.0
+     */
+    public function write(string $app, $message, int $level);
 }

lib/public/Log/IFileBased.php

Indentation +8 added lines, -8 removed lines

@@ -31,13 +31,13 @@
  * @since 14.0.0
  */
 interface IFileBased {
-	/**
-	 * @since 14.0.0
-	 */
-	public function getLogFilePath():string;
+    /**
+     * @since 14.0.0
+     */
+    public function getLogFilePath():string;
 
-	/**
-	 * @since 14.0.0
-	 */
-	public function getEntries(int $limit=50, int $offset=0): array;
+    /**
+     * @since 14.0.0
+     */
+    public function getEntries(int $limit=50, int $offset=0): array;
 }

lib/public/Log/RotationTrait.php

Indentation +32 added lines, -32 removed lines

@@ -32,40 +32,40 @@
  */
 trait RotationTrait {
 
-	/**
-	 * @var string
-	 * @since 14.0.0
-	 */
-	protected $filePath;
+    /**
+     * @var string
+     * @since 14.0.0
+     */
+    protected $filePath;
 
-	/**
-	 * @var int
-	 * @since 14.0.0
-	 */
-	protected $maxSize;
+    /**
+     * @var int
+     * @since 14.0.0
+     */
+    protected $maxSize;
 
-	/**
-	 * @return string the resulting new filepath
-	 * @since 14.0.0
-	 */
-	protected function rotate():string {
-		$rotatedFile = $this->filePath.'.1';
-		rename($this->filePath, $rotatedFile);
-		return $rotatedFile;
-	}
+    /**
+     * @return string the resulting new filepath
+     * @since 14.0.0
+     */
+    protected function rotate():string {
+        $rotatedFile = $this->filePath.'.1';
+        rename($this->filePath, $rotatedFile);
+        return $rotatedFile;
+    }
 
-	/**
-	 * @return bool
-	 * @since 14.0.0
-	 */
-	protected function shouldRotateBySize():bool {
-		if ((int)$this->maxSize > 0) {
-			$filesize = @filesize($this->filePath);
-			if ($filesize >= (int)$this->maxSize) {
-				return true;
-			}
-		}
-		return false;
-	}
+    /**
+     * @return bool
+     * @since 14.0.0
+     */
+    protected function shouldRotateBySize():bool {
+        if ((int)$this->maxSize > 0) {
+            $filesize = @filesize($this->filePath);
+            if ($filesize >= (int)$this->maxSize) {
+                return true;
+            }
+        }
+        return false;
+    }
 
 }

apps/dav/lib/Controller/DirectController.php

Indentation +54 added lines, -54 removed lines

@@ -39,75 +39,75 @@
 
 class DirectController extends OCSController {
 
-	/** @var IRootFolder */
-	private $rootFolder;
+    /** @var IRootFolder */
+    private $rootFolder;
 
-	/** @var string */
-	private $userId;
+    /** @var string */
+    private $userId;
 
-	/** @var DirectMapper */
-	private $mapper;
+    /** @var DirectMapper */
+    private $mapper;
 
-	/** @var ISecureRandom */
-	private $random;
+    /** @var ISecureRandom */
+    private $random;
 
-	/** @var ITimeFactory */
-	private $timeFactory;
+    /** @var ITimeFactory */
+    private $timeFactory;
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
+    /** @var IURLGenerator */
+    private $urlGenerator;
 
 
-	public function __construct(string $appName,
-								IRequest $request,
-								IRootFolder $rootFolder,
-								string $userId,
-								DirectMapper $mapper,
-								ISecureRandom $random,
-								ITimeFactory $timeFactory,
-								IURLGenerator $urlGenerator) {
-		parent::__construct($appName, $request);
+    public function __construct(string $appName,
+                                IRequest $request,
+                                IRootFolder $rootFolder,
+                                string $userId,
+                                DirectMapper $mapper,
+                                ISecureRandom $random,
+                                ITimeFactory $timeFactory,
+                                IURLGenerator $urlGenerator) {
+        parent::__construct($appName, $request);
 
-		$this->rootFolder = $rootFolder;
-		$this->userId = $userId;
-		$this->mapper = $mapper;
-		$this->random = $random;
-		$this->timeFactory = $timeFactory;
-		$this->urlGenerator = $urlGenerator;
-	}
+        $this->rootFolder = $rootFolder;
+        $this->userId = $userId;
+        $this->mapper = $mapper;
+        $this->random = $random;
+        $this->timeFactory = $timeFactory;
+        $this->urlGenerator = $urlGenerator;
+    }
 
-	/**
-	 * @NoAdminRequired
-	 */
-	public function getUrl(int $fileId): DataResponse {
-		$userFolder = $this->rootFolder->getUserFolder($this->userId);
+    /**
+     * @NoAdminRequired
+     */
+    public function getUrl(int $fileId): DataResponse {
+        $userFolder = $this->rootFolder->getUserFolder($this->userId);
 
-		$files = $userFolder->getById($fileId);
+        $files = $userFolder->getById($fileId);
 
-		if ($files === []) {
-			throw new OCSNotFoundException();
-		}
+        if ($files === []) {
+            throw new OCSNotFoundException();
+        }
 
-		$file = array_shift($files);
-		if (!($file instanceof File)) {
-			throw new OCSBadRequestException('Direct download only works for files');
-		}
+        $file = array_shift($files);
+        if (!($file instanceof File)) {
+            throw new OCSBadRequestException('Direct download only works for files');
+        }
 
-		//TODO: at some point we should use the directdownlaod function of storages
-		$direct = new Direct();
-		$direct->setUserId($this->userId);
-		$direct->setFileId($fileId);
+        //TODO: at some point we should use the directdownlaod function of storages
+        $direct = new Direct();
+        $direct->setUserId($this->userId);
+        $direct->setFileId($fileId);
 
-		$token = $this->random->generate(60, ISecureRandom::CHAR_UPPER . ISecureRandom::CHAR_LOWER . ISecureRandom::CHAR_DIGITS);
-		$direct->setToken($token);
-		$direct->setExpiration($this->timeFactory->getTime() + 60 * 60 * 8);
+        $token = $this->random->generate(60, ISecureRandom::CHAR_UPPER . ISecureRandom::CHAR_LOWER . ISecureRandom::CHAR_DIGITS);
+        $direct->setToken($token);
+        $direct->setExpiration($this->timeFactory->getTime() + 60 * 60 * 8);
 
-		$this->mapper->insert($direct);
+        $this->mapper->insert($direct);
 
-		$url = $this->urlGenerator->getAbsoluteURL('remote.php/direct/'.$token);
+        $url = $this->urlGenerator->getAbsoluteURL('remote.php/direct/'.$token);
 
-		return new DataResponse([
-			'url' => $url,
-		]);
-	}
+        return new DataResponse([
+            'url' => $url,
+        ]);
+    }
 }

apps/dav/lib/Direct/DirectHome.php

Indentation +77 added lines, -77 removed lines

@@ -37,82 +37,82 @@
 
 class DirectHome implements ICollection {
 
-	/** @var IRootFolder */
-	private $rootFolder;
-
-	/** @var DirectMapper */
-	private $mapper;
-
-	/** @var ITimeFactory */
-	private $timeFactory;
-
-	/** @var Throttler */
-	private $throttler;
-
-	/** @var IRequest */
-	private $request;
-
-	public function __construct(IRootFolder $rootFolder,
-								DirectMapper $mapper,
-								ITimeFactory $timeFactory,
-								Throttler $throttler,
-								IRequest $request) {
-		$this->rootFolder = $rootFolder;
-		$this->mapper = $mapper;
-		$this->timeFactory = $timeFactory;
-		$this->throttler = $throttler;
-		$this->request = $request;
-	}
-
-	public function createFile($name, $data = null) {
-		throw new Forbidden();
-	}
-
-	public function createDirectory($name) {
-		throw new Forbidden();
-	}
-
-	public function getChild($name): DirectFile {
-		try {
-			$direct = $this->mapper->getByToken($name);
-
-			// Expired
-			if ($direct->getExpiration() < $this->timeFactory->getTime()) {
-				throw new NotFound();
-			}
-
-			return new DirectFile($direct, $this->rootFolder);
-		} catch (DoesNotExistException $e) {
-			// Since the token space is so huge only throttle on non exsisting token
-			$this->throttler->registerAttempt('directlink', $this->request->getRemoteAddress());
-			$this->throttler->sleepDelay($this->request->getRemoteAddress(), 'directlink');
-
-			throw new NotFound();
-		}
-	}
-
-	public function getChildren() {
-		throw new MethodNotAllowed('Listing members of this collection is disabled');
-	}
-
-	public function childExists($name): bool {
-		return false;
-	}
-
-	public function delete() {
-		throw new Forbidden();
-	}
-
-	public function getName(): string {
-		return 'direct';
-	}
-
-	public function setName($name) {
-		throw new Forbidden();
-	}
-
-	public function getLastModified(): int {
-		return 0;
-	}
+    /** @var IRootFolder */
+    private $rootFolder;
+
+    /** @var DirectMapper */
+    private $mapper;
+
+    /** @var ITimeFactory */
+    private $timeFactory;
+
+    /** @var Throttler */
+    private $throttler;
+
+    /** @var IRequest */
+    private $request;
+
+    public function __construct(IRootFolder $rootFolder,
+                                DirectMapper $mapper,
+                                ITimeFactory $timeFactory,
+                                Throttler $throttler,
+                                IRequest $request) {
+        $this->rootFolder = $rootFolder;
+        $this->mapper = $mapper;
+        $this->timeFactory = $timeFactory;
+        $this->throttler = $throttler;
+        $this->request = $request;
+    }
+
+    public function createFile($name, $data = null) {
+        throw new Forbidden();
+    }
+
+    public function createDirectory($name) {
+        throw new Forbidden();
+    }
+
+    public function getChild($name): DirectFile {
+        try {
+            $direct = $this->mapper->getByToken($name);
+
+            // Expired
+            if ($direct->getExpiration() < $this->timeFactory->getTime()) {
+                throw new NotFound();
+            }
+
+            return new DirectFile($direct, $this->rootFolder);
+        } catch (DoesNotExistException $e) {
+            // Since the token space is so huge only throttle on non exsisting token
+            $this->throttler->registerAttempt('directlink', $this->request->getRemoteAddress());
+            $this->throttler->sleepDelay($this->request->getRemoteAddress(), 'directlink');
+
+            throw new NotFound();
+        }
+    }
+
+    public function getChildren() {
+        throw new MethodNotAllowed('Listing members of this collection is disabled');
+    }
+
+    public function childExists($name): bool {
+        return false;
+    }
+
+    public function delete() {
+        throw new Forbidden();
+    }
+
+    public function getName(): string {
+        return 'direct';
+    }
+
+    public function setName($name) {
+        throw new Forbidden();
+    }
+
+    public function getLastModified(): int {
+        return 0;
+    }
 
 }

core/Command/Db/Migrations/ExecuteCommand.php

Indentation +90 added lines, -90 removed lines

@@ -38,95 +38,95 @@
 
 class ExecuteCommand extends Command implements CompletionAwareInterface {
 
-	/** @var IDBConnection */
-	private $connection;
-
-	/** @var IConfig */
-	private $config;
-
-	/** @var IAppManager */
-	protected $appManager;
-
-	/**
-	 * ExecuteCommand constructor.
-	 *
-	 * @param IDBConnection $connection
-	 * @param IConfig $config
-	 * @param IAppManager $appManager
-	 */
-	public function __construct(IDBConnection $connection, IAppManager $appManager, IConfig $config) {
-		$this->connection = $connection;
-		$this->config = $config;
-
-		parent::__construct();
-	}
-
-	protected function configure() {
-		$this
-			->setName('migrations:execute')
-			->setDescription('Execute a single migration version manually.')
-			->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on')
-			->addArgument('version', InputArgument::REQUIRED, 'The version to execute.', null);
-
-		parent::configure();
-	}
-
-	/**
-	 * @param InputInterface $input
-	 * @param OutputInterface $output
-	 * @return int
-	 */
-	public function execute(InputInterface $input, OutputInterface $output) {
-		$appName = $input->getArgument('app');
-		$ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
-		$version = $input->getArgument('version');
-
-		if ($this->config->getSystemValue('debug', false) === false) {
-			$olderVersions = $ms->getMigratedVersions();
-			$olderVersions[] = '0';
-			$olderVersions[] = 'prev';
-			if (in_array($version,  $olderVersions, true)) {
-				$output->writeln('<error>Can not go back to previous migration without debug enabled</error>');
-				return 1;
-			}
-		}
-
-
-		$ms->executeStep($version);
-		return 0;
-	}
-
-	/**
-	 * @param string $optionName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeOptionValues($optionName, CompletionContext $context) {
-		return [];
-	}
-
-	/**
-	 * @param string $argumentName
-	 * @param CompletionContext $context
-	 * @return string[]
-	 */
-	public function completeArgumentValues($argumentName, CompletionContext $context) {
-		if ($argumentName === 'app') {
-			$allApps = \OC_App::getAllApps();
-			return array_diff($allApps, \OC_App::getEnabledApps(true, true));
-		}
-
-		if ($argumentName === 'version') {
-			$appName = $context->getWordAtIndex($context->getWordIndex() - 1);
-
-			$ms = new MigrationService($appName, $this->connection);
-			$migrations = $ms->getAvailableVersions();
-
-			array_unshift($migrations, 'next', 'latest');
-			return $migrations;
-		}
-
-		return [];
-	}
+    /** @var IDBConnection */
+    private $connection;
+
+    /** @var IConfig */
+    private $config;
+
+    /** @var IAppManager */
+    protected $appManager;
+
+    /**
+     * ExecuteCommand constructor.
+     *
+     * @param IDBConnection $connection
+     * @param IConfig $config
+     * @param IAppManager $appManager
+     */
+    public function __construct(IDBConnection $connection, IAppManager $appManager, IConfig $config) {
+        $this->connection = $connection;
+        $this->config = $config;
+
+        parent::__construct();
+    }
+
+    protected function configure() {
+        $this
+            ->setName('migrations:execute')
+            ->setDescription('Execute a single migration version manually.')
+            ->addArgument('app', InputArgument::REQUIRED, 'Name of the app this migration command shall work on')
+            ->addArgument('version', InputArgument::REQUIRED, 'The version to execute.', null);
+
+        parent::configure();
+    }
+
+    /**
+     * @param InputInterface $input
+     * @param OutputInterface $output
+     * @return int
+     */
+    public function execute(InputInterface $input, OutputInterface $output) {
+        $appName = $input->getArgument('app');
+        $ms = new MigrationService($appName, $this->connection, new ConsoleOutput($output));
+        $version = $input->getArgument('version');
+
+        if ($this->config->getSystemValue('debug', false) === false) {
+            $olderVersions = $ms->getMigratedVersions();
+            $olderVersions[] = '0';
+            $olderVersions[] = 'prev';
+            if (in_array($version,  $olderVersions, true)) {
+                $output->writeln('<error>Can not go back to previous migration without debug enabled</error>');
+                return 1;
+            }
+        }
+
+
+        $ms->executeStep($version);
+        return 0;
+    }
+
+    /**
+     * @param string $optionName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeOptionValues($optionName, CompletionContext $context) {
+        return [];
+    }
+
+    /**
+     * @param string $argumentName
+     * @param CompletionContext $context
+     * @return string[]
+     */
+    public function completeArgumentValues($argumentName, CompletionContext $context) {
+        if ($argumentName === 'app') {
+            $allApps = \OC_App::getAllApps();
+            return array_diff($allApps, \OC_App::getEnabledApps(true, true));
+        }
+
+        if ($argumentName === 'version') {
+            $appName = $context->getWordAtIndex($context->getWordIndex() - 1);
+
+            $ms = new MigrationService($appName, $this->connection);
+            $migrations = $ms->getAvailableVersions();
+
+            array_unshift($migrations, 'next', 'latest');
+            return $migrations;
+        }
+
+        return [];
+    }
 
 }

lib/private/Repair/NC14/AddPreviewBackgroundCleanupJob.php

Indentation +11 added lines, -11 removed lines

@@ -30,19 +30,19 @@
 
 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);
+    }
 
 }