nextcloud / server

apps/dav/lib/Events/CardDeletedEvent.php

Indentation +55 added lines, -55 removed lines

@@ -34,67 +34,67 @@
  */
 class CardDeletedEvent extends Event {
 
-	/** @var int */
-	private $addressBookId;
+    /** @var int */
+    private $addressBookId;
 
-	/** @var array */
-	private $addressBookData;
+    /** @var array */
+    private $addressBookData;
 
-	/** @var array */
-	private $shares;
+    /** @var array */
+    private $shares;
 
-	/** @var array */
-	private $cardData;
+    /** @var array */
+    private $cardData;
 
-	/**
-	 * CardDeletedEvent constructor.
-	 *
-	 * @param int $addressBookId
-	 * @param array $addressBookData
-	 * @param array $shares
-	 * @param array $cardData
-	 * @since 20.0.0
-	 */
-	public function __construct(int $addressBookId,
-								array $addressBookData,
-								array $shares,
-								array $cardData) {
-		parent::__construct();
-		$this->addressBookId = $addressBookId;
-		$this->addressBookData = $addressBookData;
-		$this->shares = $shares;
-		$this->cardData = $cardData;
-	}
+    /**
+     * CardDeletedEvent constructor.
+     *
+     * @param int $addressBookId
+     * @param array $addressBookData
+     * @param array $shares
+     * @param array $cardData
+     * @since 20.0.0
+     */
+    public function __construct(int $addressBookId,
+                                array $addressBookData,
+                                array $shares,
+                                array $cardData) {
+        parent::__construct();
+        $this->addressBookId = $addressBookId;
+        $this->addressBookData = $addressBookData;
+        $this->shares = $shares;
+        $this->cardData = $cardData;
+    }
 
-	/**
-	 * @return int
-	 * @since 20.0.0
-	 */
-	public function getAddressBookId(): int {
-		return $this->addressBookId;
-	}
+    /**
+     * @return int
+     * @since 20.0.0
+     */
+    public function getAddressBookId(): int {
+        return $this->addressBookId;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getAddressBookData(): array {
-		return $this->addressBookData;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getAddressBookData(): array {
+        return $this->addressBookData;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getShares(): array {
-		return $this->shares;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getShares(): array {
+        return $this->shares;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getCardData(): array {
-		return $this->cardData;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getCardData(): array {
+        return $this->cardData;
+    }
 }

apps/dav/lib/Events/AddressBookShareUpdatedEvent.php

Indentation +67 added lines, -67 removed lines

@@ -34,81 +34,81 @@
  */
 class AddressBookShareUpdatedEvent extends Event {
 
-	/** @var int */
-	private $addressBookId;
+    /** @var int */
+    private $addressBookId;
 
-	/** @var array */
-	private $addressBookData;
+    /** @var array */
+    private $addressBookData;
 
-	/** @var array */
-	private $oldShares;
+    /** @var array */
+    private $oldShares;
 
-	/** @var array */
-	private $added;
+    /** @var array */
+    private $added;
 
-	/** @var array */
-	private $removed;
+    /** @var array */
+    private $removed;
 
-	/**
-	 * AddressBookShareUpdatedEvent constructor.
-	 *
-	 * @param int $addressBookId
-	 * @param array $addressBookData
-	 * @param array $oldShares
-	 * @param array $added
-	 * @param array $removed
-	 * @since 20.0.0
-	 */
-	public function __construct(int $addressBookId,
-								array $addressBookData,
-								array $oldShares,
-								array $added,
-								array $removed) {
-		parent::__construct();
-		$this->addressBookId = $addressBookId;
-		$this->addressBookData = $addressBookData;
-		$this->oldShares = $oldShares;
-		$this->added = $added;
-		$this->removed = $removed;
-	}
+    /**
+     * AddressBookShareUpdatedEvent constructor.
+     *
+     * @param int $addressBookId
+     * @param array $addressBookData
+     * @param array $oldShares
+     * @param array $added
+     * @param array $removed
+     * @since 20.0.0
+     */
+    public function __construct(int $addressBookId,
+                                array $addressBookData,
+                                array $oldShares,
+                                array $added,
+                                array $removed) {
+        parent::__construct();
+        $this->addressBookId = $addressBookId;
+        $this->addressBookData = $addressBookData;
+        $this->oldShares = $oldShares;
+        $this->added = $added;
+        $this->removed = $removed;
+    }
 
-	/**
-	 * @return int
-	 * @since 20.0.0
-	 */
-	public function getAddressBookId(): int {
-		return $this->addressBookId;
-	}
+    /**
+     * @return int
+     * @since 20.0.0
+     */
+    public function getAddressBookId(): int {
+        return $this->addressBookId;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getAddressBookData(): array {
-		return $this->addressBookData;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getAddressBookData(): array {
+        return $this->addressBookData;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getOldShares(): array {
-		return $this->oldShares;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getOldShares(): array {
+        return $this->oldShares;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getAdded(): array {
-		return $this->added;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getAdded(): array {
+        return $this->added;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getRemoved(): array {
-		return $this->removed;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getRemoved(): array {
+        return $this->removed;
+    }
 }

apps/dav/lib/Events/AddressBookCreatedEvent.php

Indentation +31 added lines, -31 removed lines

@@ -34,39 +34,39 @@
  */
 class AddressBookCreatedEvent extends Event {
 
-	/** @var int */
-	private $addressBookId;
+    /** @var int */
+    private $addressBookId;
 
-	/** @var array */
-	private $addressBookData;
+    /** @var array */
+    private $addressBookData;
 
-	/**
-	 * AddressBookCreatedEvent constructor.
-	 *
-	 * @param int $addressBookId
-	 * @param array $addressBookData
-	 * @since 20.0.0
-	 */
-	public function __construct(int $addressBookId,
-								array $addressBookData) {
-		parent::__construct();
-		$this->addressBookId = $addressBookId;
-		$this->addressBookData = $addressBookData;
-	}
+    /**
+     * AddressBookCreatedEvent constructor.
+     *
+     * @param int $addressBookId
+     * @param array $addressBookData
+     * @since 20.0.0
+     */
+    public function __construct(int $addressBookId,
+                                array $addressBookData) {
+        parent::__construct();
+        $this->addressBookId = $addressBookId;
+        $this->addressBookData = $addressBookData;
+    }
 
-	/**
-	 * @return int
-	 * @since 20.0.0
-	 */
-	public function getAddressBookId(): int {
-		return $this->addressBookId;
-	}
+    /**
+     * @return int
+     * @since 20.0.0
+     */
+    public function getAddressBookId(): int {
+        return $this->addressBookId;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getAddressBookData(): array {
-		return $this->addressBookData;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getAddressBookData(): array {
+        return $this->addressBookData;
+    }
 }

apps/dav/lib/Events/CalendarDeletedEvent.php

Indentation +43 added lines, -43 removed lines

@@ -34,53 +34,53 @@
  */
 class CalendarDeletedEvent extends Event {
 
-	/** @var int */
-	private $calendarId;
+    /** @var int */
+    private $calendarId;
 
-	/** @var array */
-	private $calendarData;
+    /** @var array */
+    private $calendarData;
 
-	/** @var array  */
-	private $shares;
+    /** @var array  */
+    private $shares;
 
-	/**
-	 * CalendarDeletedEvent constructor.
-	 *
-	 * @param int $calendarId
-	 * @param array $calendarData
-	 * @param array $shares
-	 * @since 20.0.0
-	 */
-	public function __construct(int $calendarId,
-								array $calendarData,
-								array $shares) {
-		parent::__construct();
-		$this->calendarId = $calendarId;
-		$this->calendarData = $calendarData;
-		$this->shares = $shares;
-	}
+    /**
+     * CalendarDeletedEvent constructor.
+     *
+     * @param int $calendarId
+     * @param array $calendarData
+     * @param array $shares
+     * @since 20.0.0
+     */
+    public function __construct(int $calendarId,
+                                array $calendarData,
+                                array $shares) {
+        parent::__construct();
+        $this->calendarId = $calendarId;
+        $this->calendarData = $calendarData;
+        $this->shares = $shares;
+    }
 
-	/**
-	 * @return int
-	 * @since 20.0.0
-	 */
-	public function getCalendarId(): int {
-		return $this->calendarId;
-	}
+    /**
+     * @return int
+     * @since 20.0.0
+     */
+    public function getCalendarId(): int {
+        return $this->calendarId;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getCalendarData(): array {
-		return $this->calendarData;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getCalendarData(): array {
+        return $this->calendarData;
+    }
 
-	/**
-	 * @return array
-	 * @since 20.0.0
-	 */
-	public function getShares(): array {
-		return $this->shares;
-	}
+    /**
+     * @return array
+     * @since 20.0.0
+     */
+    public function getShares(): array {
+        return $this->shares;
+    }
 }

apps/settings/lib/SetupChecks/LegacySSEKeyFormat.php

Indentation +28 added lines, -28 removed lines

@@ -32,32 +32,32 @@
 use OCP\IURLGenerator;
 
 class LegacySSEKeyFormat {
-	/** @var IL10N */
-	private $l10n;
-	/** @var IConfig */
-	private $config;
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
-	public function __construct(IL10N $l10n, IConfig $config, IURLGenerator $urlGenerator) {
-		$this->l10n = $l10n;
-		$this->config = $config;
-		$this->urlGenerator = $urlGenerator;
-	}
-
-	public function description(): string {
-		return $this->l10n->t('The old server-side-encryption format is enabled. We recommend disabling this.');
-	}
-
-	public function severity(): string {
-		return 'warning';
-	}
-
-	public function run(): bool {
-		return $this->config->getSystemValueBool('encryption.legacy_format_support', false) === false;
-	}
-
-	public function linkToDocumentation(): string {
-		return $this->urlGenerator->linkToDocs('admin-sse-legacy-format');
-	}
+    /** @var IL10N */
+    private $l10n;
+    /** @var IConfig */
+    private $config;
+    /** @var IURLGenerator */
+    private $urlGenerator;
+
+    public function __construct(IL10N $l10n, IConfig $config, IURLGenerator $urlGenerator) {
+        $this->l10n = $l10n;
+        $this->config = $config;
+        $this->urlGenerator = $urlGenerator;
+    }
+
+    public function description(): string {
+        return $this->l10n->t('The old server-side-encryption format is enabled. We recommend disabling this.');
+    }
+
+    public function severity(): string {
+        return 'warning';
+    }
+
+    public function run(): bool {
+        return $this->config->getSystemValueBool('encryption.legacy_format_support', false) === false;
+    }
+
+    public function linkToDocumentation(): string {
+        return $this->urlGenerator->linkToDocs('admin-sse-legacy-format');
+    }
 }

apps/weather_status/lib/AppInfo/Application.php

Indentation +23 added lines, -23 removed lines

@@ -42,31 +42,31 @@
  */
 class Application extends App implements IBootstrap {
 
-	/** @var string */
-	public const APP_ID = 'weather_status';
+    /** @var string */
+    public const APP_ID = 'weather_status';
 
-	/**
-	 * Application constructor.
-	 *
-	 * @param array $urlParams
-	 */
-	public function __construct(array $urlParams = []) {
-		parent::__construct(self::APP_ID, $urlParams);
+    /**
+     * Application constructor.
+     *
+     * @param array $urlParams
+     */
+    public function __construct(array $urlParams = []) {
+        parent::__construct(self::APP_ID, $urlParams);
 
-		$dispatcher = $this->getContainer()->query(IEventDispatcher::class);
-		$dispatcher->addListener(RegisterWidgetEvent::class, function (Event $e) {
-			Util::addScript(self::APP_ID, 'weather-status');
-		});
-	}
+        $dispatcher = $this->getContainer()->query(IEventDispatcher::class);
+        $dispatcher->addListener(RegisterWidgetEvent::class, function (Event $e) {
+            Util::addScript(self::APP_ID, 'weather-status');
+        });
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function register(IRegistrationContext $context): void {
-		// Register OCS Capabilities
-		$context->registerCapability(Capabilities::class);
-	}
+    /**
+     * @inheritDoc
+     */
+    public function register(IRegistrationContext $context): void {
+        // Register OCS Capabilities
+        $context->registerCapability(Capabilities::class);
+    }
 
-	public function boot(IBootContext $context): void {
-	}
+    public function boot(IBootContext $context): void {
+    }
 }

apps/weather_status/lib/Capabilities.php

Indentation +16 added lines, -16 removed lines

@@ -35,21 +35,21 @@
  */
 class Capabilities implements ICapability {
 
-	/**
-	 * Capabilities constructor.
-	 *
-	 */
-	public function __construct() {
-	}
+    /**
+     * Capabilities constructor.
+     *
+     */
+    public function __construct() {
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function getCapabilities() {
-		return [
-			Application::APP_ID => [
-				'enabled' => true,
-			],
-		];
-	}
+    /**
+     * @inheritDoc
+     */
+    public function getCapabilities() {
+        return [
+            Application::APP_ID => [
+                'enabled' => true,
+            ],
+        ];
+    }
 }

apps/dav/lib/Migration/BuildCalendarSearchIndexBackgroundJob.php

Indentation +87 added lines, -87 removed lines

@@ -34,91 +34,91 @@
 
 class BuildCalendarSearchIndexBackgroundJob extends QueuedJob {
 
-	/** @var IDBConnection */
-	private $db;
-
-	/** @var CalDavBackend */
-	private $calDavBackend;
-
-	/** @var ILogger */
-	private $logger;
-
-	/** @var IJobList */
-	private $jobList;
-
-	/** @var ITimeFactory */
-	private $timeFactory;
-
-	/**
-	 * @param IDBConnection $db
-	 * @param CalDavBackend $calDavBackend
-	 * @param ILogger $logger
-	 * @param IJobList $jobList
-	 * @param ITimeFactory $timeFactory
-	 */
-	public function __construct(IDBConnection $db,
-								CalDavBackend $calDavBackend,
-								ILogger $logger,
-								IJobList $jobList,
-								ITimeFactory $timeFactory) {
-		$this->db = $db;
-		$this->calDavBackend = $calDavBackend;
-		$this->logger = $logger;
-		$this->jobList = $jobList;
-		$this->timeFactory = $timeFactory;
-	}
-
-	public function run($arguments) {
-		$offset = (int) $arguments['offset'];
-		$stopAt = (int) $arguments['stopAt'];
-
-		$this->logger->info('Building calendar index (' . $offset .'/' . $stopAt . ')');
-
-		$startTime = $this->timeFactory->getTime();
-		while (($this->timeFactory->getTime() - $startTime) < 15) {
-			$offset = $this->buildIndex($offset, $stopAt);
-			if ($offset >= $stopAt) {
-				break;
-			}
-		}
-
-		if ($offset >= $stopAt) {
-			$this->logger->info('Building calendar index done');
-		} else {
-			$this->jobList->add(self::class, [
-				'offset' => $offset,
-				'stopAt' => $stopAt
-			]);
-			$this->logger->info('New building calendar index job scheduled with offset ' . $offset);
-		}
-	}
-
-	/**
-	 * @param int $offset
-	 * @param int $stopAt
-	 * @return int
-	 */
-	private function buildIndex(int $offset, int $stopAt): int {
-		$query = $this->db->getQueryBuilder();
-		$query->select(['id', 'calendarid', 'uri', 'calendardata'])
-			->from('calendarobjects')
-			->where($query->expr()->lte('id', $query->createNamedParameter($stopAt)))
-			->andWhere($query->expr()->gt('id', $query->createNamedParameter($offset)))
-			->orderBy('id', 'ASC')
-			->setMaxResults(500);
-
-		$result = $query->execute();
-		while ($row = $result->fetch(\PDO::FETCH_ASSOC)) {
-			$offset = $row['id'];
-
-			$calendarData = $row['calendardata'];
-			if (is_resource($calendarData)) {
-				$calendarData = stream_get_contents($calendarData);
-			}
-
-			$this->calDavBackend->updateProperties($row['calendarid'], $row['uri'], $calendarData);
-		}
-
-		return $offset;
-	}
+    /** @var IDBConnection */
+    private $db;
+
+    /** @var CalDavBackend */
+    private $calDavBackend;
+
+    /** @var ILogger */
+    private $logger;
+
+    /** @var IJobList */
+    private $jobList;
+
+    /** @var ITimeFactory */
+    private $timeFactory;
+
+    /**
+     * @param IDBConnection $db
+     * @param CalDavBackend $calDavBackend
+     * @param ILogger $logger
+     * @param IJobList $jobList
+     * @param ITimeFactory $timeFactory
+     */
+    public function __construct(IDBConnection $db,
+                                CalDavBackend $calDavBackend,
+                                ILogger $logger,
+                                IJobList $jobList,
+                                ITimeFactory $timeFactory) {
+        $this->db = $db;
+        $this->calDavBackend = $calDavBackend;
+        $this->logger = $logger;
+        $this->jobList = $jobList;
+        $this->timeFactory = $timeFactory;
+    }
+
+    public function run($arguments) {
+        $offset = (int) $arguments['offset'];
+        $stopAt = (int) $arguments['stopAt'];
+
+        $this->logger->info('Building calendar index (' . $offset .'/' . $stopAt . ')');
+
+        $startTime = $this->timeFactory->getTime();
+        while (($this->timeFactory->getTime() - $startTime) < 15) {
+            $offset = $this->buildIndex($offset, $stopAt);
+            if ($offset >= $stopAt) {
+                break;
+            }
+        }
+
+        if ($offset >= $stopAt) {
+            $this->logger->info('Building calendar index done');
+        } else {
+            $this->jobList->add(self::class, [
+                'offset' => $offset,
+                'stopAt' => $stopAt
+            ]);
+            $this->logger->info('New building calendar index job scheduled with offset ' . $offset);
+        }
+    }
+
+    /**
+     * @param int $offset
+     * @param int $stopAt
+     * @return int
+     */
+    private function buildIndex(int $offset, int $stopAt): int {
+        $query = $this->db->getQueryBuilder();
+        $query->select(['id', 'calendarid', 'uri', 'calendardata'])
+            ->from('calendarobjects')
+            ->where($query->expr()->lte('id', $query->createNamedParameter($stopAt)))
+            ->andWhere($query->expr()->gt('id', $query->createNamedParameter($offset)))
+            ->orderBy('id', 'ASC')
+            ->setMaxResults(500);
+
+        $result = $query->execute();
+        while ($row = $result->fetch(\PDO::FETCH_ASSOC)) {
+            $offset = $row['id'];
+
+            $calendarData = $row['calendardata'];
+            if (is_resource($calendarData)) {
+                $calendarData = stream_get_contents($calendarData);
+            }
+
+            $this->calDavBackend->updateProperties($row['calendarid'], $row['uri'], $calendarData);
+        }
+
+        return $offset;
+    }
 }

lib/private/TempManager.php

Indentation +220 added lines, -220 removed lines

@@ -37,244 +37,244 @@
 use Psr\Log\LoggerInterface;
 
 class TempManager implements ITempManager {
-	/** @var string[] Current temporary files and folders, used for cleanup */
-	protected $current = [];
-	/** @var string i.e. /tmp on linux systems */
-	protected $tmpBaseDir;
-	/** @var LoggerInterface */
-	protected $log;
-	/** @var IConfig */
-	protected $config;
-	/** @var IniGetWrapper */
-	protected $iniGetWrapper;
+    /** @var string[] Current temporary files and folders, used for cleanup */
+    protected $current = [];
+    /** @var string i.e. /tmp on linux systems */
+    protected $tmpBaseDir;
+    /** @var LoggerInterface */
+    protected $log;
+    /** @var IConfig */
+    protected $config;
+    /** @var IniGetWrapper */
+    protected $iniGetWrapper;
 
-	/** Prefix */
-	public const TMP_PREFIX = 'oc_tmp_';
+    /** Prefix */
+    public const TMP_PREFIX = 'oc_tmp_';
 
-	public function __construct(LoggerInterface $logger, IConfig $config, IniGetWrapper $iniGetWrapper) {
-		$this->log = $logger;
-		$this->config = $config;
-		$this->iniGetWrapper = $iniGetWrapper;
-		$this->tmpBaseDir = $this->getTempBaseDir();
-	}
+    public function __construct(LoggerInterface $logger, IConfig $config, IniGetWrapper $iniGetWrapper) {
+        $this->log = $logger;
+        $this->config = $config;
+        $this->iniGetWrapper = $iniGetWrapper;
+        $this->tmpBaseDir = $this->getTempBaseDir();
+    }
 
-	/**
-	 * Builds the filename with suffix and removes potential dangerous characters
-	 * such as directory separators.
-	 *
-	 * @param string $absolutePath Absolute path to the file / folder
-	 * @param string $postFix Postfix appended to the temporary file name, may be user controlled
-	 * @return string
-	 */
-	private function buildFileNameWithSuffix($absolutePath, $postFix = '') {
-		if ($postFix !== '') {
-			$postFix = '.' . ltrim($postFix, '.');
-			$postFix = str_replace(['\\', '/'], '', $postFix);
-			$absolutePath .= '-';
-		}
+    /**
+     * Builds the filename with suffix and removes potential dangerous characters
+     * such as directory separators.
+     *
+     * @param string $absolutePath Absolute path to the file / folder
+     * @param string $postFix Postfix appended to the temporary file name, may be user controlled
+     * @return string
+     */
+    private function buildFileNameWithSuffix($absolutePath, $postFix = '') {
+        if ($postFix !== '') {
+            $postFix = '.' . ltrim($postFix, '.');
+            $postFix = str_replace(['\\', '/'], '', $postFix);
+            $absolutePath .= '-';
+        }
 
-		return $absolutePath . $postFix;
-	}
+        return $absolutePath . $postFix;
+    }
 
-	/**
-	 * Create a temporary file and return the path
-	 *
-	 * @param string $postFix Postfix appended to the temporary file name
-	 * @return string
-	 */
-	public function getTemporaryFile($postFix = '') {
-		if (is_writable($this->tmpBaseDir)) {
-			// To create an unique file and prevent the risk of race conditions
-			// or duplicated temporary files by other means such as collisions
-			// we need to create the file using `tempnam` and append a possible
-			// postfix to it later
-			$file = tempnam($this->tmpBaseDir, self::TMP_PREFIX);
-			$this->current[] = $file;
+    /**
+     * Create a temporary file and return the path
+     *
+     * @param string $postFix Postfix appended to the temporary file name
+     * @return string
+     */
+    public function getTemporaryFile($postFix = '') {
+        if (is_writable($this->tmpBaseDir)) {
+            // To create an unique file and prevent the risk of race conditions
+            // or duplicated temporary files by other means such as collisions
+            // we need to create the file using `tempnam` and append a possible
+            // postfix to it later
+            $file = tempnam($this->tmpBaseDir, self::TMP_PREFIX);
+            $this->current[] = $file;
 
-			// If a postfix got specified sanitize it and create a postfixed
-			// temporary file
-			if ($postFix !== '') {
-				$fileNameWithPostfix = $this->buildFileNameWithSuffix($file, $postFix);
-				touch($fileNameWithPostfix);
-				chmod($fileNameWithPostfix, 0600);
-				$this->current[] = $fileNameWithPostfix;
-				return $fileNameWithPostfix;
-			}
+            // If a postfix got specified sanitize it and create a postfixed
+            // temporary file
+            if ($postFix !== '') {
+                $fileNameWithPostfix = $this->buildFileNameWithSuffix($file, $postFix);
+                touch($fileNameWithPostfix);
+                chmod($fileNameWithPostfix, 0600);
+                $this->current[] = $fileNameWithPostfix;
+                return $fileNameWithPostfix;
+            }
 
-			return $file;
-		} else {
-			$this->log->warning(
-				'Can not create a temporary file in directory {dir}. Check it exists and has correct permissions',
-				[
-					'dir' => $this->tmpBaseDir,
-				]
-			);
-			return false;
-		}
-	}
+            return $file;
+        } else {
+            $this->log->warning(
+                'Can not create a temporary file in directory {dir}. Check it exists and has correct permissions',
+                [
+                    'dir' => $this->tmpBaseDir,
+                ]
+            );
+            return false;
+        }
+    }
 
-	/**
-	 * Create a temporary folder and return the path
-	 *
-	 * @param string $postFix Postfix appended to the temporary folder name
-	 * @return string
-	 */
-	public function getTemporaryFolder($postFix = '') {
-		if (is_writable($this->tmpBaseDir)) {
-			// To create an unique directory and prevent the risk of race conditions
-			// or duplicated temporary files by other means such as collisions
-			// we need to create the file using `tempnam` and append a possible
-			// postfix to it later
-			$uniqueFileName = tempnam($this->tmpBaseDir, self::TMP_PREFIX);
-			$this->current[] = $uniqueFileName;
+    /**
+     * Create a temporary folder and return the path
+     *
+     * @param string $postFix Postfix appended to the temporary folder name
+     * @return string
+     */
+    public function getTemporaryFolder($postFix = '') {
+        if (is_writable($this->tmpBaseDir)) {
+            // To create an unique directory and prevent the risk of race conditions
+            // or duplicated temporary files by other means such as collisions
+            // we need to create the file using `tempnam` and append a possible
+            // postfix to it later
+            $uniqueFileName = tempnam($this->tmpBaseDir, self::TMP_PREFIX);
+            $this->current[] = $uniqueFileName;
 
-			// Build a name without postfix
-			$path = $this->buildFileNameWithSuffix($uniqueFileName . '-folder', $postFix);
-			mkdir($path, 0700);
-			$this->current[] = $path;
+            // Build a name without postfix
+            $path = $this->buildFileNameWithSuffix($uniqueFileName . '-folder', $postFix);
+            mkdir($path, 0700);
+            $this->current[] = $path;
 
-			return $path . '/';
-		} else {
-			$this->log->warning(
-				'Can not create a temporary folder in directory {dir}. Check it exists and has correct permissions',
-				[
-					'dir' => $this->tmpBaseDir,
-				]
-			);
-			return false;
-		}
-	}
+            return $path . '/';
+        } else {
+            $this->log->warning(
+                'Can not create a temporary folder in directory {dir}. Check it exists and has correct permissions',
+                [
+                    'dir' => $this->tmpBaseDir,
+                ]
+            );
+            return false;
+        }
+    }
 
-	/**
-	 * Remove the temporary files and folders generated during this request
-	 */
-	public function clean() {
-		$this->cleanFiles($this->current);
-	}
+    /**
+     * Remove the temporary files and folders generated during this request
+     */
+    public function clean() {
+        $this->cleanFiles($this->current);
+    }
 
-	/**
-	 * @param string[] $files
-	 */
-	protected function cleanFiles($files) {
-		foreach ($files as $file) {
-			if (file_exists($file)) {
-				try {
-					\OC_Helper::rmdirr($file);
-				} catch (\UnexpectedValueException $ex) {
-					$this->log->warning(
-						"Error deleting temporary file/folder: {file} - Reason: {error}",
-						[
-							'file' => $file,
-							'error' => $ex->getMessage(),
-						]
-					);
-				}
-			}
-		}
-	}
+    /**
+     * @param string[] $files
+     */
+    protected function cleanFiles($files) {
+        foreach ($files as $file) {
+            if (file_exists($file)) {
+                try {
+                    \OC_Helper::rmdirr($file);
+                } catch (\UnexpectedValueException $ex) {
+                    $this->log->warning(
+                        "Error deleting temporary file/folder: {file} - Reason: {error}",
+                        [
+                            'file' => $file,
+                            'error' => $ex->getMessage(),
+                        ]
+                    );
+                }
+            }
+        }
+    }
 
-	/**
-	 * Remove old temporary files and folders that were failed to be cleaned
-	 */
-	public function cleanOld() {
-		$this->cleanFiles($this->getOldFiles());
-	}
+    /**
+     * Remove old temporary files and folders that were failed to be cleaned
+     */
+    public function cleanOld() {
+        $this->cleanFiles($this->getOldFiles());
+    }
 
-	/**
-	 * Get all temporary files and folders generated by oc older than an hour
-	 *
-	 * @return string[]
-	 */
-	protected function getOldFiles() {
-		$cutOfTime = time() - 3600;
-		$files = [];
-		$dh = opendir($this->tmpBaseDir);
-		if ($dh) {
-			while (($file = readdir($dh)) !== false) {
-				if (substr($file, 0, 7) === self::TMP_PREFIX) {
-					$path = $this->tmpBaseDir . '/' . $file;
-					$mtime = filemtime($path);
-					if ($mtime < $cutOfTime) {
-						$files[] = $path;
-					}
-				}
-			}
-		}
-		return $files;
-	}
+    /**
+     * Get all temporary files and folders generated by oc older than an hour
+     *
+     * @return string[]
+     */
+    protected function getOldFiles() {
+        $cutOfTime = time() - 3600;
+        $files = [];
+        $dh = opendir($this->tmpBaseDir);
+        if ($dh) {
+            while (($file = readdir($dh)) !== false) {
+                if (substr($file, 0, 7) === self::TMP_PREFIX) {
+                    $path = $this->tmpBaseDir . '/' . $file;
+                    $mtime = filemtime($path);
+                    if ($mtime < $cutOfTime) {
+                        $files[] = $path;
+                    }
+                }
+            }
+        }
+        return $files;
+    }
 
-	/**
-	 * Get the temporary base directory configured on the server
-	 *
-	 * @return string Path to the temporary directory or null
-	 * @throws \UnexpectedValueException
-	 */
-	public function getTempBaseDir() {
-		if ($this->tmpBaseDir) {
-			return $this->tmpBaseDir;
-		}
+    /**
+     * Get the temporary base directory configured on the server
+     *
+     * @return string Path to the temporary directory or null
+     * @throws \UnexpectedValueException
+     */
+    public function getTempBaseDir() {
+        if ($this->tmpBaseDir) {
+            return $this->tmpBaseDir;
+        }
 
-		$directories = [];
-		if ($temp = $this->config->getSystemValue('tempdirectory', null)) {
-			$directories[] = $temp;
-		}
-		if ($temp = $this->iniGetWrapper->get('upload_tmp_dir')) {
-			$directories[] = $temp;
-		}
-		if ($temp = getenv('TMP')) {
-			$directories[] = $temp;
-		}
-		if ($temp = getenv('TEMP')) {
-			$directories[] = $temp;
-		}
-		if ($temp = getenv('TMPDIR')) {
-			$directories[] = $temp;
-		}
-		if ($temp = sys_get_temp_dir()) {
-			$directories[] = $temp;
-		}
+        $directories = [];
+        if ($temp = $this->config->getSystemValue('tempdirectory', null)) {
+            $directories[] = $temp;
+        }
+        if ($temp = $this->iniGetWrapper->get('upload_tmp_dir')) {
+            $directories[] = $temp;
+        }
+        if ($temp = getenv('TMP')) {
+            $directories[] = $temp;
+        }
+        if ($temp = getenv('TEMP')) {
+            $directories[] = $temp;
+        }
+        if ($temp = getenv('TMPDIR')) {
+            $directories[] = $temp;
+        }
+        if ($temp = sys_get_temp_dir()) {
+            $directories[] = $temp;
+        }
 
-		foreach ($directories as $dir) {
-			if ($this->checkTemporaryDirectory($dir)) {
-				return $dir;
-			}
-		}
+        foreach ($directories as $dir) {
+            if ($this->checkTemporaryDirectory($dir)) {
+                return $dir;
+            }
+        }
 
-		$temp = tempnam(dirname(__FILE__), '');
-		if (file_exists($temp)) {
-			unlink($temp);
-			return dirname($temp);
-		}
-		throw new \UnexpectedValueException('Unable to detect system temporary directory');
-	}
+        $temp = tempnam(dirname(__FILE__), '');
+        if (file_exists($temp)) {
+            unlink($temp);
+            return dirname($temp);
+        }
+        throw new \UnexpectedValueException('Unable to detect system temporary directory');
+    }
 
-	/**
-	 * Check if a temporary directory is ready for use
-	 *
-	 * @param mixed $directory
-	 * @return bool
-	 */
-	private function checkTemporaryDirectory($directory) {
-		// suppress any possible errors caused by is_writable
-		// checks missing or invalid path or characters, wrong permissions etc
-		try {
-			if (is_writable($directory)) {
-				return true;
-			}
-		} catch (\Exception $e) {
-		}
-		$this->log->warning('Temporary directory {dir} is not present or writable',
-			['dir' => $directory]
-		);
-		return false;
-	}
+    /**
+     * Check if a temporary directory is ready for use
+     *
+     * @param mixed $directory
+     * @return bool
+     */
+    private function checkTemporaryDirectory($directory) {
+        // suppress any possible errors caused by is_writable
+        // checks missing or invalid path or characters, wrong permissions etc
+        try {
+            if (is_writable($directory)) {
+                return true;
+            }
+        } catch (\Exception $e) {
+        }
+        $this->log->warning('Temporary directory {dir} is not present or writable',
+            ['dir' => $directory]
+        );
+        return false;
+    }
 
-	/**
-	 * Override the temporary base directory
-	 *
-	 * @param string $directory
-	 */
-	public function overrideTempBaseDir($directory) {
-		$this->tmpBaseDir = $directory;
-	}
+    /**
+     * Override the temporary base directory
+     *
+     * @param string $directory
+     */
+    public function overrideTempBaseDir($directory) {
+        $this->tmpBaseDir = $directory;
+    }
 }