nextcloud / server

apps/user_ldap/lib/WizardResult.php

Indentation +40 added lines, -40 removed lines

@@ -30,50 +30,50 @@
 namespace OCA\User_LDAP;
 
 class WizardResult {
-	protected $changes = [];
-	protected $options = [];
-	protected $markedChange = false;
+    protected $changes = [];
+    protected $options = [];
+    protected $markedChange = false;
 
-	/**
-	 * @param string $key
-	 * @param mixed $value
-	 */
-	public function addChange($key, $value) {
-		$this->changes[$key] = $value;
-	}
+    /**
+     * @param string $key
+     * @param mixed $value
+     */
+    public function addChange($key, $value) {
+        $this->changes[$key] = $value;
+    }
 
 	
-	public function markChange() {
-		$this->markedChange = true;
-	}
+    public function markChange() {
+        $this->markedChange = true;
+    }
 
-	/**
-	 * @param string $key
-	 * @param array|string $values
-	 */
-	public function addOptions($key, $values) {
-		if (!is_array($values)) {
-			$values = [$values];
-		}
-		$this->options[$key] = $values;
-	}
+    /**
+     * @param string $key
+     * @param array|string $values
+     */
+    public function addOptions($key, $values) {
+        if (!is_array($values)) {
+            $values = [$values];
+        }
+        $this->options[$key] = $values;
+    }
 
-	/**
-	 * @return bool
-	 */
-	public function hasChanges() {
-		return (count($this->changes) > 0 || $this->markedChange);
-	}
+    /**
+     * @return bool
+     */
+    public function hasChanges() {
+        return (count($this->changes) > 0 || $this->markedChange);
+    }
 
-	/**
-	 * @return array
-	 */
-	public function getResultArray() {
-		$result = [];
-		$result['changes'] = $this->changes;
-		if (count($this->options) > 0) {
-			$result['options'] = $this->options;
-		}
-		return $result;
-	}
+    /**
+     * @return array
+     */
+    public function getResultArray() {
+        $result = [];
+        $result['changes'] = $this->changes;
+        if (count($this->options) > 0) {
+            $result['options'] = $this->options;
+        }
+        return $result;
+    }
 }

apps/user_ldap/ajax/setConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 \OC_JSON::checkAppEnabled('user_ldap');
 \OC_JSON::callCheck();
 
-$prefix = (string)$_POST['ldap_serverconfig_chooser'];
+$prefix = (string) $_POST['ldap_serverconfig_chooser'];
 
 // Checkboxes are not submitted, when they are unchecked. Set them manually.
 // only legacy checkboxes (Advanced and Expert tab) need to be handled here,

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
 // only legacy checkboxes (Advanced and Expert tab) need to be handled here,
 // the Wizard-like tabs handle it on their own
 $chkboxes = ['ldap_configuration_active', 'ldap_override_main_server',
-	'ldap_turn_off_cert_check'];
+    'ldap_turn_off_cert_check'];
 foreach ($chkboxes as $boxid) {
-	if (!isset($_POST[$boxid])) {
-		$_POST[$boxid] = 0;
-	}
+    if (!isset($_POST[$boxid])) {
+        $_POST[$boxid] = 0;
+    }
 }
 
 $ldapWrapper = new LDAP();

apps/dav/lib/Command/ListCalendars.php

Spacing +3 added lines, -3 removed lines

@@ -77,7 +77,7 @@ 
 			}
 
 			$readOnly = false;
-			$readOnlyIndex = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
+			$readOnlyIndex = '{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}read-only';
 			if (isset($calendar[$readOnlyIndex])) {
 				$readOnly = $calendar[$readOnlyIndex];
 			}
@@ -85,8 +85,8 @@ 
 			$calendarTableData[] = [
 				$calendar['uri'],
 				$calendar['{DAV:}displayname'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname'],
+				$calendar['{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}owner-principal'],
+				$calendar['{'.\OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD.'}owner-displayname'],
 				$readOnly ? ' x ' : ' ✓ ',
 			];
 		}

Indentation +49 added lines, -49 removed lines

@@ -15,61 +15,61 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ListCalendars extends Command {
-	public function __construct(
-		protected IUserManager $userManager,
-		private CalDavBackend $caldav,
-	) {
-		parent::__construct();
-	}
+    public function __construct(
+        protected IUserManager $userManager,
+        private CalDavBackend $caldav,
+    ) {
+        parent::__construct();
+    }
 
-	protected function configure(): void {
-		$this
-			->setName('dav:list-calendars')
-			->setDescription('List all calendars of a user')
-			->addArgument('uid',
-				InputArgument::REQUIRED,
-				'User for whom all calendars will be listed');
-	}
+    protected function configure(): void {
+        $this
+            ->setName('dav:list-calendars')
+            ->setDescription('List all calendars of a user')
+            ->addArgument('uid',
+                InputArgument::REQUIRED,
+                'User for whom all calendars will be listed');
+    }
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$user = $input->getArgument('uid');
-		if (!$this->userManager->userExists($user)) {
-			throw new \InvalidArgumentException("User <$user> is unknown.");
-		}
+    protected function execute(InputInterface $input, OutputInterface $output): int {
+        $user = $input->getArgument('uid');
+        if (!$this->userManager->userExists($user)) {
+            throw new \InvalidArgumentException("User <$user> is unknown.");
+        }
 
-		$calendars = $this->caldav->getCalendarsForUser("principals/users/$user");
+        $calendars = $this->caldav->getCalendarsForUser("principals/users/$user");
 
-		$calendarTableData = [];
-		foreach ($calendars as $calendar) {
-			// skip birthday calendar
-			if ($calendar['uri'] === BirthdayService::BIRTHDAY_CALENDAR_URI) {
-				continue;
-			}
+        $calendarTableData = [];
+        foreach ($calendars as $calendar) {
+            // skip birthday calendar
+            if ($calendar['uri'] === BirthdayService::BIRTHDAY_CALENDAR_URI) {
+                continue;
+            }
 
-			$readOnly = false;
-			$readOnlyIndex = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
-			if (isset($calendar[$readOnlyIndex])) {
-				$readOnly = $calendar[$readOnlyIndex];
-			}
+            $readOnly = false;
+            $readOnlyIndex = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
+            if (isset($calendar[$readOnlyIndex])) {
+                $readOnly = $calendar[$readOnlyIndex];
+            }
 
-			$calendarTableData[] = [
-				$calendar['uri'],
-				$calendar['{DAV:}displayname'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'],
-				$calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname'],
-				$readOnly ? ' x ' : ' ✓ ',
-			];
-		}
+            $calendarTableData[] = [
+                $calendar['uri'],
+                $calendar['{DAV:}displayname'],
+                $calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'],
+                $calendar['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname'],
+                $readOnly ? ' x ' : ' ✓ ',
+            ];
+        }
 
-		if (count($calendarTableData) > 0) {
-			$table = new Table($output);
-			$table->setHeaders(['URI', 'Displayname', 'Owner principal', 'Owner displayname', 'Writable'])
-				->setRows($calendarTableData);
+        if (count($calendarTableData) > 0) {
+            $table = new Table($output);
+            $table->setHeaders(['URI', 'Displayname', 'Owner principal', 'Owner displayname', 'Writable'])
+                ->setRows($calendarTableData);
 
-			$table->render();
-		} else {
-			$output->writeln("<info>User <$user> has no calendars</info>");
-		}
-		return self::SUCCESS;
-	}
+            $table->render();
+        } else {
+            $output->writeln("<info>User <$user> has no calendars</info>");
+        }
+        return self::SUCCESS;
+    }
 }

apps/workflowengine/lib/Helper/LogContext.php

Indentation +56 added lines, -56 removed lines

@@ -30,67 +30,67 @@
 use OCP\WorkflowEngine\IOperation;
 
 class LogContext {
-	/** @var array */
-	protected $details;
+    /** @var array */
+    protected $details;
 
-	public function setDescription(string $description): LogContext {
-		$this->details['message'] = $description;
-		return $this;
-	}
+    public function setDescription(string $description): LogContext {
+        $this->details['message'] = $description;
+        return $this;
+    }
 
-	public function setScopes(array $scopes): LogContext {
-		$this->details['scopes'] = [];
-		foreach ($scopes as $scope) {
-			if ($scope instanceof ScopeContext) {
-				switch ($scope->getScope()) {
-					case IManager::SCOPE_ADMIN:
-						$this->details['scopes'][] = ['scope' => 'admin'];
-						break;
-					case IManager::SCOPE_USER:
-						$this->details['scopes'][] = [
-							'scope' => 'user',
-							'uid' => $scope->getScopeId(),
-						];
-						break;
-					default:
-						continue 2;
-				}
-			}
-		}
-		return $this;
-	}
+    public function setScopes(array $scopes): LogContext {
+        $this->details['scopes'] = [];
+        foreach ($scopes as $scope) {
+            if ($scope instanceof ScopeContext) {
+                switch ($scope->getScope()) {
+                    case IManager::SCOPE_ADMIN:
+                        $this->details['scopes'][] = ['scope' => 'admin'];
+                        break;
+                    case IManager::SCOPE_USER:
+                        $this->details['scopes'][] = [
+                            'scope' => 'user',
+                            'uid' => $scope->getScopeId(),
+                        ];
+                        break;
+                    default:
+                        continue 2;
+                }
+            }
+        }
+        return $this;
+    }
 
-	public function setOperation(?IOperation $operation): LogContext {
-		if ($operation instanceof IOperation) {
-			$this->details['operation'] = [
-				'class' => get_class($operation),
-				'name' => $operation->getDisplayName(),
-			];
-		}
-		return $this;
-	}
+    public function setOperation(?IOperation $operation): LogContext {
+        if ($operation instanceof IOperation) {
+            $this->details['operation'] = [
+                'class' => get_class($operation),
+                'name' => $operation->getDisplayName(),
+            ];
+        }
+        return $this;
+    }
 
-	public function setEntity(?IEntity $entity): LogContext {
-		if ($entity instanceof IEntity) {
-			$this->details['entity'] = [
-				'class' => get_class($entity),
-				'name' => $entity->getName(),
-			];
-		}
-		return $this;
-	}
+    public function setEntity(?IEntity $entity): LogContext {
+        if ($entity instanceof IEntity) {
+            $this->details['entity'] = [
+                'class' => get_class($entity),
+                'name' => $entity->getName(),
+            ];
+        }
+        return $this;
+    }
 
-	public function setConfiguration(array $configuration): LogContext {
-		$this->details['configuration'] = $configuration;
-		return $this;
-	}
+    public function setConfiguration(array $configuration): LogContext {
+        $this->details['configuration'] = $configuration;
+        return $this;
+    }
 
-	public function setEventName(string $eventName): LogContext {
-		$this->details['eventName'] = $eventName;
-		return $this;
-	}
+    public function setEventName(string $eventName): LogContext {
+        $this->details['eventName'] = $eventName;
+        return $this;
+    }
 
-	public function getDetails(): array {
-		return $this->details;
-	}
+    public function getDetails(): array {
+        return $this->details;
+    }
 }

apps/workflowengine/lib/Controller/GlobalWorkflowsController.php

Indentation +8 added lines, -8 removed lines

@@ -30,13 +30,13 @@
 
 class GlobalWorkflowsController extends AWorkflowController {
 
-	/** @var ScopeContext */
-	private $scopeContext;
+    /** @var ScopeContext */
+    private $scopeContext;
 
-	protected function getScopeContext(): ScopeContext {
-		if ($this->scopeContext === null) {
-			$this->scopeContext = new ScopeContext(IManager::SCOPE_ADMIN);
-		}
-		return $this->scopeContext;
-	}
+    protected function getScopeContext(): ScopeContext {
+        if ($this->scopeContext === null) {
+            $this->scopeContext = new ScopeContext(IManager::SCOPE_ADMIN);
+        }
+        return $this->scopeContext;
+    }
 }

apps/files_external/lib/Lib/Backend/SFTP_Key.php

Indentation +15 added lines, -15 removed lines

@@ -30,19 +30,19 @@
 use OCP\IL10N;
 
 class SFTP_Key extends Backend {
-	public function __construct(IL10N $l, RSA $legacyAuth, SFTP $sftpBackend) {
-		$this
-			->setIdentifier('\OC\Files\Storage\SFTP_Key')
-			->setStorageClass('\OCA\Files_External\Lib\Storage\SFTP')
-			->setText($l->t('SFTP with secret key login'))
-			->addParameters([
-				new DefinitionParameter('host', $l->t('Host')),
-				(new DefinitionParameter('root', $l->t('Remote subfolder')))
-					->setFlag(DefinitionParameter::FLAG_OPTIONAL),
-			])
-			->addAuthScheme(AuthMechanism::SCHEME_PUBLICKEY)
-			->setLegacyAuthMechanism($legacyAuth)
-			->deprecateTo($sftpBackend)
-		;
-	}
+    public function __construct(IL10N $l, RSA $legacyAuth, SFTP $sftpBackend) {
+        $this
+            ->setIdentifier('\OC\Files\Storage\SFTP_Key')
+            ->setStorageClass('\OCA\Files_External\Lib\Storage\SFTP')
+            ->setText($l->t('SFTP with secret key login'))
+            ->addParameters([
+                new DefinitionParameter('host', $l->t('Host')),
+                (new DefinitionParameter('root', $l->t('Remote subfolder')))
+                    ->setFlag(DefinitionParameter::FLAG_OPTIONAL),
+            ])
+            ->addAuthScheme(AuthMechanism::SCHEME_PUBLICKEY)
+            ->setLegacyAuthMechanism($legacyAuth)
+            ->deprecateTo($sftpBackend)
+        ;
+    }
 }

apps/files_external/lib/Config/UserPlaceholderHandler.php

Indentation +14 added lines, -14 removed lines

@@ -25,19 +25,19 @@
 namespace OCA\Files_External\Config;
 
 class UserPlaceholderHandler extends UserContext implements IConfigHandler {
-	use SimpleSubstitutionTrait;
+    use SimpleSubstitutionTrait;
 
-	/**
-	 * @param mixed $optionValue
-	 * @return mixed the same type as $optionValue
-	 * @since 16.0.0
-	 */
-	public function handle($optionValue) {
-		$this->placeholder = 'user';
-		$uid = $this->getUserId();
-		if ($uid === null) {
-			return $optionValue;
-		}
-		return $this->processInput($optionValue, $uid);
-	}
+    /**
+     * @param mixed $optionValue
+     * @return mixed the same type as $optionValue
+     * @since 16.0.0
+     */
+    public function handle($optionValue) {
+        $this->placeholder = 'user';
+        $uid = $this->getUserId();
+        if ($uid === null) {
+            return $optionValue;
+        }
+        return $this->processInput($optionValue, $uid);
+    }
 }

apps/files_external/lib/Config/SimpleSubstitutionTrait.php

Indentation +49 added lines, -49 removed lines

@@ -31,57 +31,57 @@
  * @since 16.0.0
  */
 trait SimpleSubstitutionTrait {
-	/**
-	 * @var string the placeholder without $ prefix
-	 * @since 16.0.0
-	 */
-	protected $placeholder;
+    /**
+     * @var string the placeholder without $ prefix
+     * @since 16.0.0
+     */
+    protected $placeholder;
 
-	/** @var string */
-	protected $sanitizedPlaceholder;
+    /** @var string */
+    protected $sanitizedPlaceholder;
 
-	/**
-	 * @param mixed $optionValue
-	 * @param string $replacement
-	 * @return mixed
-	 * @since 16.0.0
-	 */
-	private function processInput($optionValue, string $replacement) {
-		$this->checkPlaceholder();
-		if (is_array($optionValue)) {
-			foreach ($optionValue as &$value) {
-				$value = $this->substituteIfString($value, $replacement);
-			}
-		} else {
-			$optionValue = $this->substituteIfString($optionValue, $replacement);
-		}
-		return $optionValue;
-	}
+    /**
+     * @param mixed $optionValue
+     * @param string $replacement
+     * @return mixed
+     * @since 16.0.0
+     */
+    private function processInput($optionValue, string $replacement) {
+        $this->checkPlaceholder();
+        if (is_array($optionValue)) {
+            foreach ($optionValue as &$value) {
+                $value = $this->substituteIfString($value, $replacement);
+            }
+        } else {
+            $optionValue = $this->substituteIfString($optionValue, $replacement);
+        }
+        return $optionValue;
+    }
 
-	/**
-	 * @throws \RuntimeException
-	 */
-	protected function checkPlaceholder(): void {
-		$this->sanitizedPlaceholder = trim(strtolower($this->placeholder));
-		if (!(bool)\preg_match('/^[a-z0-9]*$/', $this->sanitizedPlaceholder)) {
-			throw new \RuntimeException(sprintf(
-				'Invalid placeholder %s, only [a-z0-9] are allowed', $this->sanitizedPlaceholder
-			));
-		}
-		if ($this->sanitizedPlaceholder === '') {
-			throw new \RuntimeException('Invalid empty placeholder');
-		}
-	}
+    /**
+     * @throws \RuntimeException
+     */
+    protected function checkPlaceholder(): void {
+        $this->sanitizedPlaceholder = trim(strtolower($this->placeholder));
+        if (!(bool)\preg_match('/^[a-z0-9]*$/', $this->sanitizedPlaceholder)) {
+            throw new \RuntimeException(sprintf(
+                'Invalid placeholder %s, only [a-z0-9] are allowed', $this->sanitizedPlaceholder
+            ));
+        }
+        if ($this->sanitizedPlaceholder === '') {
+            throw new \RuntimeException('Invalid empty placeholder');
+        }
+    }
 
-	/**
-	 * @param mixed $value
-	 * @param string $replacement
-	 * @return mixed
-	 */
-	protected function substituteIfString($value, string $replacement) {
-		if (is_string($value)) {
-			return str_ireplace('$' . $this->sanitizedPlaceholder, $replacement, $value);
-		}
-		return $value;
-	}
+    /**
+     * @param mixed $value
+     * @param string $replacement
+     * @return mixed
+     */
+    protected function substituteIfString($value, string $replacement) {
+        if (is_string($value)) {
+            return str_ireplace('$' . $this->sanitizedPlaceholder, $replacement, $value);
+        }
+        return $value;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
 	 */
 	protected function checkPlaceholder(): void {
 		$this->sanitizedPlaceholder = trim(strtolower($this->placeholder));
-		if (!(bool)\preg_match('/^[a-z0-9]*$/', $this->sanitizedPlaceholder)) {
+		if (!(bool) \preg_match('/^[a-z0-9]*$/', $this->sanitizedPlaceholder)) {
 			throw new \RuntimeException(sprintf(
 				'Invalid placeholder %s, only [a-z0-9] are allowed', $this->sanitizedPlaceholder
 			));
@@ -80,7 +80,7 @@ 
 	 */
 	protected function substituteIfString($value, string $replacement) {
 		if (is_string($value)) {
-			return str_ireplace('$' . $this->sanitizedPlaceholder, $replacement, $value);
+			return str_ireplace('$'.$this->sanitizedPlaceholder, $replacement, $value);
 		}
 		return $value;
 	}

apps/comments/lib/Collaboration/CommentersSorter.php

Spacing +2 added lines, -2 removed lines

@@ -60,11 +60,11 @@
 			// at least on PHP 5.6 usort turned out to be not stable. So we add
 			// the current index to the value and compare it on a draw
 			$i = 0;
-			$workArray = array_map(function ($element) use (&$i) {
+			$workArray = array_map(function($element) use (&$i) {
 				return [$i++, $element];
 			}, $byType);
 
-			usort($workArray, function ($a, $b) use ($commenters, $type) {
+			usort($workArray, function($a, $b) use ($commenters, $type) {
 				$r = $this->compare($a[1], $b[1], $commenters[$type]);
 				if ($r === 0) {
 					$r = $a[0] - $b[0];

Indentation +68 added lines, -68 removed lines

@@ -9,83 +9,83 @@
 use OCP\Comments\ICommentsManager;
 
 class CommentersSorter implements ISorter {
-	public function __construct(
-		private ICommentsManager $commentsManager,
-	) {
-	}
+    public function __construct(
+        private ICommentsManager $commentsManager,
+    ) {
+    }
 
-	public function getId(): string {
-		return 'commenters';
-	}
+    public function getId(): string {
+        return 'commenters';
+    }
 
-	/**
-	 * Sorts people who commented on the given item atop (descelating) of the
-	 * others
-	 *
-	 * @param array &$sortArray
-	 * @param array $context
-	 */
-	public function sort(array &$sortArray, array $context): void {
-		$commenters = $this->retrieveCommentsInformation($context['itemType'], $context['itemId']);
-		if (count($commenters) === 0) {
-			return;
-		}
+    /**
+     * Sorts people who commented on the given item atop (descelating) of the
+     * others
+     *
+     * @param array &$sortArray
+     * @param array $context
+     */
+    public function sort(array &$sortArray, array $context): void {
+        $commenters = $this->retrieveCommentsInformation($context['itemType'], $context['itemId']);
+        if (count($commenters) === 0) {
+            return;
+        }
 
-		foreach ($sortArray as $type => &$byType) {
-			if (!isset($commenters[$type])) {
-				continue;
-			}
+        foreach ($sortArray as $type => &$byType) {
+            if (!isset($commenters[$type])) {
+                continue;
+            }
 
-			// at least on PHP 5.6 usort turned out to be not stable. So we add
-			// the current index to the value and compare it on a draw
-			$i = 0;
-			$workArray = array_map(function ($element) use (&$i) {
-				return [$i++, $element];
-			}, $byType);
+            // at least on PHP 5.6 usort turned out to be not stable. So we add
+            // the current index to the value and compare it on a draw
+            $i = 0;
+            $workArray = array_map(function ($element) use (&$i) {
+                return [$i++, $element];
+            }, $byType);
 
-			usort($workArray, function ($a, $b) use ($commenters, $type) {
-				$r = $this->compare($a[1], $b[1], $commenters[$type]);
-				if ($r === 0) {
-					$r = $a[0] - $b[0];
-				}
-				return $r;
-			});
+            usort($workArray, function ($a, $b) use ($commenters, $type) {
+                $r = $this->compare($a[1], $b[1], $commenters[$type]);
+                if ($r === 0) {
+                    $r = $a[0] - $b[0];
+                }
+                return $r;
+            });
 
-			// and remove the index values again
-			$byType = array_column($workArray, 1);
-		}
-	}
+            // and remove the index values again
+            $byType = array_column($workArray, 1);
+        }
+    }
 
-	/**
-	 * @return array<string, array<string, int>>
-	 */
-	protected function retrieveCommentsInformation(string $type, string $id): array {
-		$comments = $this->commentsManager->getForObject($type, $id);
-		if (count($comments) === 0) {
-			return [];
-		}
+    /**
+     * @return array<string, array<string, int>>
+     */
+    protected function retrieveCommentsInformation(string $type, string $id): array {
+        $comments = $this->commentsManager->getForObject($type, $id);
+        if (count($comments) === 0) {
+            return [];
+        }
 
-		$actors = [];
-		foreach ($comments as $comment) {
-			if (!isset($actors[$comment->getActorType()])) {
-				$actors[$comment->getActorType()] = [];
-			}
-			if (!isset($actors[$comment->getActorType()][$comment->getActorId()])) {
-				$actors[$comment->getActorType()][$comment->getActorId()] = 1;
-			} else {
-				$actors[$comment->getActorType()][$comment->getActorId()]++;
-			}
-		}
-		return $actors;
-	}
+        $actors = [];
+        foreach ($comments as $comment) {
+            if (!isset($actors[$comment->getActorType()])) {
+                $actors[$comment->getActorType()] = [];
+            }
+            if (!isset($actors[$comment->getActorType()][$comment->getActorId()])) {
+                $actors[$comment->getActorType()][$comment->getActorId()] = 1;
+            } else {
+                $actors[$comment->getActorType()][$comment->getActorId()]++;
+            }
+        }
+        return $actors;
+    }
 
-	protected function compare(array $a, array $b, array $commenters): int {
-		$a = $a['value']['shareWith'];
-		$b = $b['value']['shareWith'];
+    protected function compare(array $a, array $b, array $commenters): int {
+        $a = $a['value']['shareWith'];
+        $b = $b['value']['shareWith'];
 
-		$valueA = $commenters[$a] ?? 0;
-		$valueB = $commenters[$b] ?? 0;
+        $valueA = $commenters[$a] ?? 0;
+        $valueB = $commenters[$b] ?? 0;
 
-		return $valueB - $valueA;
-	}
+        return $valueB - $valueA;
+    }
 }