nextcloud / server

apps/dav/lib/BackgroundJob/EventReminderJob.php

Indentation +31 added lines, -31 removed lines

@@ -25,40 +25,40 @@
 
 class EventReminderJob extends TimedJob {
 
-	/** @var ReminderService */
-	private $reminderService;
+    /** @var ReminderService */
+    private $reminderService;
 
-	/** @var IConfig */
-	private $config;
+    /** @var IConfig */
+    private $config;
 
-	/**
-	 * EventReminderJob constructor.
-	 *
-	 * @param ReminderService $reminderService
-	 * @param IConfig $config
-	 */
-	public function __construct(ReminderService $reminderService, IConfig $config) {
-		$this->reminderService = $reminderService;
-		$this->config = $config;
-		/** Run every 5 minutes */
-		$this->setInterval(5);
-	}
+    /**
+     * EventReminderJob constructor.
+     *
+     * @param ReminderService $reminderService
+     * @param IConfig $config
+     */
+    public function __construct(ReminderService $reminderService, IConfig $config) {
+        $this->reminderService = $reminderService;
+        $this->config = $config;
+        /** Run every 5 minutes */
+        $this->setInterval(5);
+    }
 
-	/**
-	 * @param $arg
-	 * @throws \OCA\DAV\CalDAV\Reminder\NotificationProvider\ProviderNotAvailableException
-	 * @throws \OCA\DAV\CalDAV\Reminder\NotificationTypeDoesNotExistException
-	 * @throws \OC\User\NoUserException
-	 */
-	public function run($arg):void {
-		if ($this->config->getAppValue('dav', 'sendEventReminders', 'yes') !== 'yes') {
-			return;
-		}
+    /**
+     * @param $arg
+     * @throws \OCA\DAV\CalDAV\Reminder\NotificationProvider\ProviderNotAvailableException
+     * @throws \OCA\DAV\CalDAV\Reminder\NotificationTypeDoesNotExistException
+     * @throws \OC\User\NoUserException
+     */
+    public function run($arg):void {
+        if ($this->config->getAppValue('dav', 'sendEventReminders', 'yes') !== 'yes') {
+            return;
+        }
 
-		if ($this->config->getAppValue('dav', 'sendEventRemindersMode', 'backgroundjob') !== 'backgroundjob') {
-			return;
-		}
+        if ($this->config->getAppValue('dav', 'sendEventRemindersMode', 'backgroundjob') !== 'backgroundjob') {
+            return;
+        }
 
-		$this->reminderService->processReminders();
-	}
+        $this->reminderService->processReminders();
+    }
 }

apps/dav/lib/CalDAV/Reminder/NotificationProviderManager.php

Indentation +42 added lines, -42 removed lines

@@ -31,51 +31,51 @@
  */
 class NotificationProviderManager {
 
-	/** @var INotificationProvider[] */
-	private $providers = [];
+    /** @var INotificationProvider[] */
+    private $providers = [];
 
-	/**
-	 * Checks whether a provider for a given ACTION exists
-	 *
-	 * @param string $type
-	 * @return bool
-	 */
-	public function hasProvider(string $type):bool {
-		return (\in_array($type, ReminderService::REMINDER_TYPES, true)
-			&& isset($this->providers[$type]));
-	}
+    /**
+     * Checks whether a provider for a given ACTION exists
+     *
+     * @param string $type
+     * @return bool
+     */
+    public function hasProvider(string $type):bool {
+        return (\in_array($type, ReminderService::REMINDER_TYPES, true)
+            && isset($this->providers[$type]));
+    }
 
-	/**
-	 * Get provider for a given ACTION
-	 *
-	 * @param string $type
-	 * @return INotificationProvider
-	 * @throws NotificationProvider\ProviderNotAvailableException
-	 * @throws NotificationTypeDoesNotExistException
-	 */
-	public function getProvider(string $type):INotificationProvider {
-		if (in_array($type, ReminderService::REMINDER_TYPES, true)) {
-			if (isset($this->providers[$type])) {
-				return $this->providers[$type];
-			}
-			throw new NotificationProvider\ProviderNotAvailableException($type);
-		}
-		throw new NotificationTypeDoesNotExistException($type);
-	}
+    /**
+     * Get provider for a given ACTION
+     *
+     * @param string $type
+     * @return INotificationProvider
+     * @throws NotificationProvider\ProviderNotAvailableException
+     * @throws NotificationTypeDoesNotExistException
+     */
+    public function getProvider(string $type):INotificationProvider {
+        if (in_array($type, ReminderService::REMINDER_TYPES, true)) {
+            if (isset($this->providers[$type])) {
+                return $this->providers[$type];
+            }
+            throw new NotificationProvider\ProviderNotAvailableException($type);
+        }
+        throw new NotificationTypeDoesNotExistException($type);
+    }
 
-	/**
-	 * Registers a new provider
-	 *
-	 * @param string $providerClassName
-	 * @throws \OCP\AppFramework\QueryException
-	 */
-	public function registerProvider(string $providerClassName):void {
-		$provider = \OC::$server->query($providerClassName);
+    /**
+     * Registers a new provider
+     *
+     * @param string $providerClassName
+     * @throws \OCP\AppFramework\QueryException
+     */
+    public function registerProvider(string $providerClassName):void {
+        $provider = \OC::$server->query($providerClassName);
 
-		if (!$provider instanceof INotificationProvider) {
-			throw new \InvalidArgumentException('Invalid notification provider registered');
-		}
+        if (!$provider instanceof INotificationProvider) {
+            throw new \InvalidArgumentException('Invalid notification provider registered');
+        }
 
-		$this->providers[$provider::NOTIFICATION_TYPE] = $provider;
-	}
+        $this->providers[$provider::NOTIFICATION_TYPE] = $provider;
+    }
 }

apps/dav/lib/CalDAV/Reminder/NotificationTypeDoesNotExistException.php

Indentation +10 added lines, -10 removed lines

@@ -26,15 +26,15 @@
 
 class NotificationTypeDoesNotExistException extends \Exception {
 
-	/**
-	 * NotificationTypeDoesNotExistException constructor.
-	 *
-	 * @since 16.0.0
-	 *
-	 * @param string $type ReminderType
-	 */
-	public function __construct(string $type) {
-		parent::__construct("Type $type is not an accepted type of notification");
-	}
+    /**
+     * NotificationTypeDoesNotExistException constructor.
+     *
+     * @since 16.0.0
+     *
+     * @param string $type ReminderType
+     */
+    public function __construct(string $type) {
+        parent::__construct("Type $type is not an accepted type of notification");
+    }
 
 }

apps/dav/lib/CalDAV/Reminder/NotificationProvider/ProviderNotAvailableException.php

Indentation +10 added lines, -10 removed lines

@@ -25,15 +25,15 @@
 
 class ProviderNotAvailableException extends \Exception {
 
-	/**
-	 * ProviderNotAvailableException constructor.
-	 *
-	 * @since 16.0.0
-	 *
-	 * @param string $type ReminderType
-	 */
-	public function __construct(string $type) {
-		parent::__construct("No notification provider for type $type available");
-	}
+    /**
+     * ProviderNotAvailableException constructor.
+     *
+     * @since 16.0.0
+     *
+     * @param string $type ReminderType
+     */
+    public function __construct(string $type) {
+        parent::__construct("No notification provider for type $type available");
+    }
 
 }

apps/dav/lib/CalDAV/Reminder/NotificationProvider/AudioProvider.php

Indentation +2 added lines, -2 removed lines

@@ -32,6 +32,6 @@
  */
 class AudioProvider extends PushProvider {
 
-	/** @var string */
-	public const NOTIFICATION_TYPE = 'AUDIO';
+    /** @var string */
+    public const NOTIFICATION_TYPE = 'AUDIO';
 }

apps/dav/lib/CalDAV/Reminder/Backend.php

Indentation +181 added lines, -181 removed lines

@@ -35,185 +35,185 @@
  */
 class Backend {
 
-	/** @var IDBConnection */
-	protected $db;
-
-	/** @var ITimeFactory */
-	private $timeFactory;
-
-	/**
-	 * Backend constructor.
-	 *
-	 * @param IDBConnection $db
-	 * @param ITimeFactory $timeFactory
-	 */
-	public function __construct(IDBConnection $db,
-								ITimeFactory $timeFactory) {
-		$this->db = $db;
-		$this->timeFactory = $timeFactory;
-	}
-
-	/**
-	 * Get all reminders with a notification date before now
-	 *
-	 * @return array
-	 * @throws \Exception
-	 */
-	public function getRemindersToProcess():array {
-		$query = $this->db->getQueryBuilder();
-		$query->select(['cr.*', 'co.calendardata', 'c.displayname', 'c.principaluri'])
-			->from('calendar_reminders', 'cr')
-			->where($query->expr()->lte('cr.notification_date', $query->createNamedParameter($this->timeFactory->getTime())))
-			->leftJoin('cr', 'calendarobjects', 'co', $query->expr()->eq('cr.object_id', 'co.id'))
-			->leftJoin('cr', 'calendars', 'c', $query->expr()->eq('cr.calendar_id', 'c.id'));
-		$stmt = $query->execute();
-
-		return array_map(
-			[$this, 'fixRowTyping'],
-			$stmt->fetchAll()
-		);
-	}
-
-	/**
-	 * Get all scheduled reminders for an event
-	 *
-	 * @param int $objectId
-	 * @return array
-	 */
-	public function getAllScheduledRemindersForEvent(int $objectId):array {
-		$query = $this->db->getQueryBuilder();
-		$query->select('*')
-			->from('calendar_reminders')
-			->where($query->expr()->eq('object_id', $query->createNamedParameter($objectId)));
-		$stmt = $query->execute();
-
-		return array_map(
-			[$this, 'fixRowTyping'],
-			$stmt->fetchAll()
-		);
-	}
-
-	/**
-	 * Insert a new reminder into the database
-	 *
-	 * @param int $calendarId
-	 * @param int $objectId
-	 * @param string $uid
-	 * @param bool $isRecurring
-	 * @param int $recurrenceId
-	 * @param bool $isRecurrenceException
-	 * @param string $eventHash
-	 * @param string $alarmHash
-	 * @param string $type
-	 * @param bool $isRelative
-	 * @param int $notificationDate
-	 * @param bool $isRepeatBased
-	 * @return int The insert id
-	 */
-	public function insertReminder(int $calendarId,
-								   int $objectId,
-								   string $uid,
-								   bool $isRecurring,
-								   int $recurrenceId,
-								   bool $isRecurrenceException,
-								   string $eventHash,
-								   string $alarmHash,
-								   string $type,
-								   bool $isRelative,
-								   int $notificationDate,
-								   bool $isRepeatBased):int {
-		$query = $this->db->getQueryBuilder();
-		$query->insert('calendar_reminders')
-			->values([
-				'calendar_id' => $query->createNamedParameter($calendarId),
-				'object_id' => $query->createNamedParameter($objectId),
-				'uid' => $query->createNamedParameter($uid),
-				'is_recurring' => $query->createNamedParameter($isRecurring ? 1 : 0),
-				'recurrence_id' => $query->createNamedParameter($recurrenceId),
-				'is_recurrence_exception' => $query->createNamedParameter($isRecurrenceException ? 1 : 0),
-				'event_hash' => $query->createNamedParameter($eventHash),
-				'alarm_hash' => $query->createNamedParameter($alarmHash),
-				'type' => $query->createNamedParameter($type),
-				'is_relative' => $query->createNamedParameter($isRelative ? 1 : 0),
-				'notification_date' => $query->createNamedParameter($notificationDate),
-				'is_repeat_based' => $query->createNamedParameter($isRepeatBased ? 1 : 0),
-			])
-			->execute();
-
-		return $query->getLastInsertId();
-	}
-
-	/**
-	 * Sets a new notificationDate on an existing reminder
-	 *
-	 * @param int $reminderId
-	 * @param int $newNotificationDate
-	 */
-	public function updateReminder(int $reminderId,
-								   int $newNotificationDate):void {
-		$query = $this->db->getQueryBuilder();
-		$query->update('calendar_reminders')
-			->set('notification_date', $query->createNamedParameter($newNotificationDate))
-			->where($query->expr()->eq('id', $query->createNamedParameter($reminderId)))
-			->execute();
-	}
-
-	/**
-	 * Remove a reminder by it's id
-	 *
-	 * @param integer $reminderId
-	 * @return void
-	 */
-	public function removeReminder(int $reminderId):void {
-		$query = $this->db->getQueryBuilder();
-
-		$query->delete('calendar_reminders')
-			->where($query->expr()->eq('id', $query->createNamedParameter($reminderId)))
-			->execute();
-	}
-
-	/**
-	 * Cleans reminders in database
-	 *
-	 * @param int $objectId
-	 */
-	public function cleanRemindersForEvent(int $objectId):void {
-		$query = $this->db->getQueryBuilder();
-
-		$query->delete('calendar_reminders')
-			->where($query->expr()->eq('object_id', $query->createNamedParameter($objectId)))
-			->execute();
-	}
-
-	/**
-	 * Remove all reminders for a calendar
-	 *
-	 * @param int $calendarId
-	 * @return void
-	 */
-	public function cleanRemindersForCalendar(int $calendarId):void {
-		$query = $this->db->getQueryBuilder();
-
-		$query->delete('calendar_reminders')
-			->where($query->expr()->eq('calendar_id', $query->createNamedParameter($calendarId)))
-			->execute();
-	}
-
-	/**
-	 * @param array $row
-	 * @return array
-	 */
-	private function fixRowTyping(array $row): array {
-		$row['id'] = (int) $row['id'];
-		$row['calendar_id'] = (int) $row['calendar_id'];
-		$row['object_id'] = (int) $row['object_id'];
-		$row['is_recurring'] = (bool) $row['is_recurring'];
-		$row['recurrence_id'] = (int) $row['recurrence_id'];
-		$row['is_recurrence_exception'] = (bool) $row['is_recurrence_exception'];
-		$row['is_relative'] = (bool) $row['is_relative'];
-		$row['notification_date'] = (int) $row['notification_date'];
-		$row['is_repeat_based'] = (bool) $row['is_repeat_based'];
-
-		return $row;
-	}
+    /** @var IDBConnection */
+    protected $db;
+
+    /** @var ITimeFactory */
+    private $timeFactory;
+
+    /**
+     * Backend constructor.
+     *
+     * @param IDBConnection $db
+     * @param ITimeFactory $timeFactory
+     */
+    public function __construct(IDBConnection $db,
+                                ITimeFactory $timeFactory) {
+        $this->db = $db;
+        $this->timeFactory = $timeFactory;
+    }
+
+    /**
+     * Get all reminders with a notification date before now
+     *
+     * @return array
+     * @throws \Exception
+     */
+    public function getRemindersToProcess():array {
+        $query = $this->db->getQueryBuilder();
+        $query->select(['cr.*', 'co.calendardata', 'c.displayname', 'c.principaluri'])
+            ->from('calendar_reminders', 'cr')
+            ->where($query->expr()->lte('cr.notification_date', $query->createNamedParameter($this->timeFactory->getTime())))
+            ->leftJoin('cr', 'calendarobjects', 'co', $query->expr()->eq('cr.object_id', 'co.id'))
+            ->leftJoin('cr', 'calendars', 'c', $query->expr()->eq('cr.calendar_id', 'c.id'));
+        $stmt = $query->execute();
+
+        return array_map(
+            [$this, 'fixRowTyping'],
+            $stmt->fetchAll()
+        );
+    }
+
+    /**
+     * Get all scheduled reminders for an event
+     *
+     * @param int $objectId
+     * @return array
+     */
+    public function getAllScheduledRemindersForEvent(int $objectId):array {
+        $query = $this->db->getQueryBuilder();
+        $query->select('*')
+            ->from('calendar_reminders')
+            ->where($query->expr()->eq('object_id', $query->createNamedParameter($objectId)));
+        $stmt = $query->execute();
+
+        return array_map(
+            [$this, 'fixRowTyping'],
+            $stmt->fetchAll()
+        );
+    }
+
+    /**
+     * Insert a new reminder into the database
+     *
+     * @param int $calendarId
+     * @param int $objectId
+     * @param string $uid
+     * @param bool $isRecurring
+     * @param int $recurrenceId
+     * @param bool $isRecurrenceException
+     * @param string $eventHash
+     * @param string $alarmHash
+     * @param string $type
+     * @param bool $isRelative
+     * @param int $notificationDate
+     * @param bool $isRepeatBased
+     * @return int The insert id
+     */
+    public function insertReminder(int $calendarId,
+                                    int $objectId,
+                                    string $uid,
+                                    bool $isRecurring,
+                                    int $recurrenceId,
+                                    bool $isRecurrenceException,
+                                    string $eventHash,
+                                    string $alarmHash,
+                                    string $type,
+                                    bool $isRelative,
+                                    int $notificationDate,
+                                    bool $isRepeatBased):int {
+        $query = $this->db->getQueryBuilder();
+        $query->insert('calendar_reminders')
+            ->values([
+                'calendar_id' => $query->createNamedParameter($calendarId),
+                'object_id' => $query->createNamedParameter($objectId),
+                'uid' => $query->createNamedParameter($uid),
+                'is_recurring' => $query->createNamedParameter($isRecurring ? 1 : 0),
+                'recurrence_id' => $query->createNamedParameter($recurrenceId),
+                'is_recurrence_exception' => $query->createNamedParameter($isRecurrenceException ? 1 : 0),
+                'event_hash' => $query->createNamedParameter($eventHash),
+                'alarm_hash' => $query->createNamedParameter($alarmHash),
+                'type' => $query->createNamedParameter($type),
+                'is_relative' => $query->createNamedParameter($isRelative ? 1 : 0),
+                'notification_date' => $query->createNamedParameter($notificationDate),
+                'is_repeat_based' => $query->createNamedParameter($isRepeatBased ? 1 : 0),
+            ])
+            ->execute();
+
+        return $query->getLastInsertId();
+    }
+
+    /**
+     * Sets a new notificationDate on an existing reminder
+     *
+     * @param int $reminderId
+     * @param int $newNotificationDate
+     */
+    public function updateReminder(int $reminderId,
+                                    int $newNotificationDate):void {
+        $query = $this->db->getQueryBuilder();
+        $query->update('calendar_reminders')
+            ->set('notification_date', $query->createNamedParameter($newNotificationDate))
+            ->where($query->expr()->eq('id', $query->createNamedParameter($reminderId)))
+            ->execute();
+    }
+
+    /**
+     * Remove a reminder by it's id
+     *
+     * @param integer $reminderId
+     * @return void
+     */
+    public function removeReminder(int $reminderId):void {
+        $query = $this->db->getQueryBuilder();
+
+        $query->delete('calendar_reminders')
+            ->where($query->expr()->eq('id', $query->createNamedParameter($reminderId)))
+            ->execute();
+    }
+
+    /**
+     * Cleans reminders in database
+     *
+     * @param int $objectId
+     */
+    public function cleanRemindersForEvent(int $objectId):void {
+        $query = $this->db->getQueryBuilder();
+
+        $query->delete('calendar_reminders')
+            ->where($query->expr()->eq('object_id', $query->createNamedParameter($objectId)))
+            ->execute();
+    }
+
+    /**
+     * Remove all reminders for a calendar
+     *
+     * @param int $calendarId
+     * @return void
+     */
+    public function cleanRemindersForCalendar(int $calendarId):void {
+        $query = $this->db->getQueryBuilder();
+
+        $query->delete('calendar_reminders')
+            ->where($query->expr()->eq('calendar_id', $query->createNamedParameter($calendarId)))
+            ->execute();
+    }
+
+    /**
+     * @param array $row
+     * @return array
+     */
+    private function fixRowTyping(array $row): array {
+        $row['id'] = (int) $row['id'];
+        $row['calendar_id'] = (int) $row['calendar_id'];
+        $row['object_id'] = (int) $row['object_id'];
+        $row['is_recurring'] = (bool) $row['is_recurring'];
+        $row['recurrence_id'] = (int) $row['recurrence_id'];
+        $row['is_recurrence_exception'] = (bool) $row['is_recurrence_exception'];
+        $row['is_relative'] = (bool) $row['is_relative'];
+        $row['notification_date'] = (int) $row['notification_date'];
+        $row['is_repeat_based'] = (bool) $row['is_repeat_based'];
+
+        return $row;
+    }
 }

lib/private/Repair/NC16/ClearCollectionsAccessCache.php

Indentation +25 added lines, -25 removed lines

@@ -30,29 +30,29 @@
 
 class ClearCollectionsAccessCache implements IRepairStep {
 
-	/** @var IConfig */
-	private $config;
-
-	/** @var IManager|Manager */
-	private $manager;
-
-	public function __construct(IConfig $config, IManager $manager) {
-		$this->config = $config;
-		$this->manager = $manager;
-	}
-
-	public function getName(): string {
-		return 'Clear access cache of projects';
-	}
-
-	private function shouldRun(): bool {
-		$versionFromBeforeUpdate = $this->config->getSystemValue('version', '0.0.0.0');
-		return version_compare($versionFromBeforeUpdate, '17.0.0.3', '<=');
-	}
-
-	public function run(IOutput $output): void {
-		if ($this->shouldRun()) {
-			$this->manager->invalidateAccessCacheForAllCollections();
-		}
-	}
+    /** @var IConfig */
+    private $config;
+
+    /** @var IManager|Manager */
+    private $manager;
+
+    public function __construct(IConfig $config, IManager $manager) {
+        $this->config = $config;
+        $this->manager = $manager;
+    }
+
+    public function getName(): string {
+        return 'Clear access cache of projects';
+    }
+
+    private function shouldRun(): bool {
+        $versionFromBeforeUpdate = $this->config->getSystemValue('version', '0.0.0.0');
+        return version_compare($versionFromBeforeUpdate, '17.0.0.3', '<=');
+    }
+
+    public function run(IOutput $output): void {
+        if ($this->shouldRun()) {
+            $this->manager->invalidateAccessCacheForAllCollections();
+        }
+    }
 }

lib/private/Security/RateLimiting/Limiter.php

Indentation +63 added lines, -63 removed lines

@@ -31,73 +31,73 @@
 use OCP\IUser;
 
 class Limiter {
-	/** @var IBackend */
-	private $backend;
-	/** @var ITimeFactory */
-	private $timeFactory;
+    /** @var IBackend */
+    private $backend;
+    /** @var ITimeFactory */
+    private $timeFactory;
 
-	/**
-	 * @param ITimeFactory $timeFactory
-	 * @param IBackend $backend
-	 */
-	public function __construct(ITimeFactory $timeFactory,
-								IBackend $backend) {
-		$this->backend = $backend;
-		$this->timeFactory = $timeFactory;
-	}
+    /**
+     * @param ITimeFactory $timeFactory
+     * @param IBackend $backend
+     */
+    public function __construct(ITimeFactory $timeFactory,
+                                IBackend $backend) {
+        $this->backend = $backend;
+        $this->timeFactory = $timeFactory;
+    }
 
-	/**
-	 * @param string $methodIdentifier
-	 * @param string $userIdentifier
-	 * @param int $period
-	 * @param int $limit
-	 * @throws RateLimitExceededException
-	 */
-	private function register(string $methodIdentifier,
-							  string $userIdentifier,
-							  int $period,
-							  int $limit): void {
-		$existingAttempts = $this->backend->getAttempts($methodIdentifier, $userIdentifier, $period);
-		if ($existingAttempts >= $limit) {
-			throw new RateLimitExceededException();
-		}
+    /**
+     * @param string $methodIdentifier
+     * @param string $userIdentifier
+     * @param int $period
+     * @param int $limit
+     * @throws RateLimitExceededException
+     */
+    private function register(string $methodIdentifier,
+                                string $userIdentifier,
+                                int $period,
+                                int $limit): void {
+        $existingAttempts = $this->backend->getAttempts($methodIdentifier, $userIdentifier, $period);
+        if ($existingAttempts >= $limit) {
+            throw new RateLimitExceededException();
+        }
 
-		$this->backend->registerAttempt($methodIdentifier, $userIdentifier, $this->timeFactory->getTime());
-	}
+        $this->backend->registerAttempt($methodIdentifier, $userIdentifier, $this->timeFactory->getTime());
+    }
 
-	/**
-	 * Registers attempt for an anonymous request
-	 *
-	 * @param string $identifier
-	 * @param int $anonLimit
-	 * @param int $anonPeriod
-	 * @param string $ip
-	 * @throws RateLimitExceededException
-	 */
-	public function registerAnonRequest(string $identifier,
-										int $anonLimit,
-										int $anonPeriod,
-										string $ip): void {
-		$ipSubnet = (new IpAddress($ip))->getSubnet();
+    /**
+     * Registers attempt for an anonymous request
+     *
+     * @param string $identifier
+     * @param int $anonLimit
+     * @param int $anonPeriod
+     * @param string $ip
+     * @throws RateLimitExceededException
+     */
+    public function registerAnonRequest(string $identifier,
+                                        int $anonLimit,
+                                        int $anonPeriod,
+                                        string $ip): void {
+        $ipSubnet = (new IpAddress($ip))->getSubnet();
 
-		$anonHashIdentifier = hash('sha512', 'anon::' . $identifier . $ipSubnet);
-		$this->register($identifier, $anonHashIdentifier, $anonPeriod, $anonLimit);
-	}
+        $anonHashIdentifier = hash('sha512', 'anon::' . $identifier . $ipSubnet);
+        $this->register($identifier, $anonHashIdentifier, $anonPeriod, $anonLimit);
+    }
 
-	/**
-	 * Registers attempt for an authenticated request
-	 *
-	 * @param string $identifier
-	 * @param int $userLimit
-	 * @param int $userPeriod
-	 * @param IUser $user
-	 * @throws RateLimitExceededException
-	 */
-	public function registerUserRequest(string $identifier,
-										int $userLimit,
-										int $userPeriod,
-										IUser $user): void {
-		$userHashIdentifier = hash('sha512', 'user::' . $identifier . $user->getUID());
-		$this->register($identifier, $userHashIdentifier, $userPeriod, $userLimit);
-	}
+    /**
+     * Registers attempt for an authenticated request
+     *
+     * @param string $identifier
+     * @param int $userLimit
+     * @param int $userPeriod
+     * @param IUser $user
+     * @throws RateLimitExceededException
+     */
+    public function registerUserRequest(string $identifier,
+                                        int $userLimit,
+                                        int $userPeriod,
+                                        IUser $user): void {
+        $userHashIdentifier = hash('sha512', 'user::' . $identifier . $user->getUID());
+        $this->register($identifier, $userHashIdentifier, $userPeriod, $userLimit);
+    }
 }

lib/public/AppFramework/Utility/IControllerMethodReflector.php

Indentation +31 added lines, -31 removed lines

@@ -35,39 +35,39 @@
  */
 interface IControllerMethodReflector {
 
-	/**
-	 * @param object $object an object or classname
-	 * @param string $method the method which we want to inspect
-	 * @return void
-	 * @since 8.0.0
-	 * @deprecated 17.0.0 Reflect should not be called multiple times and only be used internally. This will be removed in Nextcloud 18
-	 */
-	public function reflect($object, string $method);
+    /**
+     * @param object $object an object or classname
+     * @param string $method the method which we want to inspect
+     * @return void
+     * @since 8.0.0
+     * @deprecated 17.0.0 Reflect should not be called multiple times and only be used internally. This will be removed in Nextcloud 18
+     */
+    public function reflect($object, string $method);
 
-	/**
-	 * Inspects the PHPDoc parameters for types
-	 *
-	 * @param string $parameter the parameter whose type comments should be
-	 * parsed
-	 * @return string|null type in the type parameters (@param int $something)
-	 * would return int or null if not existing
-	 * @since 8.0.0
-	 */
-	public function getType(string $parameter);
+    /**
+     * Inspects the PHPDoc parameters for types
+     *
+     * @param string $parameter the parameter whose type comments should be
+     * parsed
+     * @return string|null type in the type parameters (@param int $something)
+     * would return int or null if not existing
+     * @since 8.0.0
+     */
+    public function getType(string $parameter);
 
-	/**
-	 * @return array the arguments of the method with key => default value
-	 * @since 8.0.0
-	 */
-	public function getParameters(): array;
+    /**
+     * @return array the arguments of the method with key => default value
+     * @since 8.0.0
+     */
+    public function getParameters(): array;
 
-	/**
-	 * Check if a method contains an annotation
-	 *
-	 * @param string $name the name of the annotation
-	 * @return bool true if the annotation is found
-	 * @since 8.0.0
-	 */
-	public function hasAnnotation(string $name): bool;
+    /**
+     * Check if a method contains an annotation
+     *
+     * @param string $name the name of the annotation
+     * @return bool true if the annotation is found
+     * @since 8.0.0
+     */
+    public function hasAnnotation(string $name): bool;
 
 }