fix: Use strict comparaison

Signed-off-by: Carl Schwan <carlschwan@kde.org>

Author: CarlSchwan

.php-cs-fixer.cache

@@ -166,7 +166,7 @@ public function import(): DataResponse {
 		if (!empty($file) && array_key_exists('error', $file) && $file['error'] !== UPLOAD_ERR_OK) {
 			$error = $phpFileUploadErrors[$file['error']];
 		}
-		if (!empty($file) && $file['error'] === UPLOAD_ERR_OK && !in_array($file['type'], ['application/json', 'text/plain'])) {
+		if (!empty($file) && $file['error'] === UPLOAD_ERR_OK && !in_array($file['type'], ['application/json', 'text/plain'], true)) {
 			$error = $this->l10n->t('Invalid file type. Only JSON files are allowed.');
 		}
 		if ($error !== null) {

lib/Controller/BoardController.php

@@ -90,7 +90,7 @@ public function findAssignedLabelsForBoard(int $boardId, ?int $limit = null, int
 	}
 
 	public function insert(Entity $entity): Entity {
-		if (!in_array('lastModified', $entity->getUpdatedFields())) {
+		if (!in_array('lastModified', $entity->getUpdatedFields(), true)) {
 			$entity->setLastModified(time());
 		}
 		return parent::insert($entity);

lib/Db/LabelMapper.php

@@ -49,7 +49,7 @@ public function handle(Event $event): void {
 		$applicableEvents = [
 			CommentsEvent::EVENT_UPDATE
 		];
-		if (in_array($eventType, $applicableEvents)) {
+		if (in_array($eventType, $applicableEvents, true)) {
 			$this->notificationHandler($event);
 			return;
 		}

lib/Listeners/CommentEventListener.php

@@ -541,7 +541,7 @@ public function clone(
 		foreach ($stacks as $stack) {
 			$newStack = new Stack();
 			$newStack->setTitle($stack->getTitle());
-			if ($stack->getOrder() == null) {
+			if ($stack->getOrder() === null) {
 				$newStack->setOrder(999);
 			} else {
 				$newStack->setOrder($stack->getOrder());

lib/Service/BoardService.php

@@ -279,7 +279,7 @@ public function update(int $id, string $title, int $stackId, string $type, strin
 		$resetDuedateNotification = false;
 		if (
 			$card->getDuedate() === null
-			|| ($card->getDuedate()) != ($changes->getBefore()->getDuedate())
+			|| ($card->getDuedate()) !== ($changes->getBefore()->getDuedate())
 		) {
 			$card->setNotified(false);
 			$resetDuedateNotification = true;
@@ -357,7 +357,7 @@ public function cloneCard(int $id, ?int $targetStackId = null):Card {
 		$newCard = $this->create($originCard->getTitle(), $targetStackId, $originCard->getType(), $originCard->getOrder(), $originCard->getOwner());
 		$boardId = $this->stackMapper->findBoardId($targetStackId);
 		foreach ($this->labelMapper->findAssignedLabelsForCard($id) as $label) {
-			if ($boardId != $this->stackMapper->findBoardId($originCard->getStackId())) {
+			if ($boardId !== $this->stackMapper->findBoardId($originCard->getStackId())) {
 				try {
 					$label = $this->labelService->cloneLabelIfNotExists($label->getId(), $boardId);
 				} catch (NoPermissionException $e) {

lib/Service/CardService.php

@@ -136,7 +136,7 @@ public function createDefaultBoard(string $title, string $userId, string $color)
 				);
 
 				foreach ($defaultLabels as $defaultLabel) {
-					if ($defaultLabel && in_array($defaultLabel->getTitle(), $cardData['labels'])) {
+					if ($defaultLabel && in_array($defaultLabel->getTitle(), $cardData['labels'], true)) {
 						$this->cardService->assignLabel($card->getId(), $defaultLabel->getId());
 					}
 				}

lib/Service/DefaultBoardService.php

@@ -38,6 +38,7 @@
 class BoardImportService {
 	private string $system = '';
 	private ?ABoardImportService $systemInstance = null;
+	/** @var list<array{name: string, class: class-string, internalName: string}> */
 	private array $allowedSystems = [];
 	/**
 	 * Data object created from config JSON
@@ -137,16 +138,12 @@ public function import(): void {
 	public function validateSystem(): void {
 		$allowedSystems = $this->getAllowedImportSystems();
 		$allowedSystems = array_column($allowedSystems, 'internalName');
-		if (!in_array($this->getSystem(), $allowedSystems)) {
+		if (!in_array($this->getSystem(), $allowedSystems, true)) {
 			throw new NotFoundException('Invalid system: ' . $this->getSystem());
 		}
 	}
 
-	/**
-	 * @param ?string $system
-	 * @return self
-	 */
-	public function setSystem($system): self {
+	public function setSystem(?string $system): self {
 		if ($system) {
 			$this->system = $system;
 		}
@@ -157,11 +154,17 @@ public function getSystem(): string {
 		return $this->system;
 	}
 
-	public function addAllowedImportSystem($system): self {
+	/**
+	 * @param array{name: string, class: class-string, internalName: string} $system
+	 */
+	public function addAllowedImportSystem(array $system): self {
 		$this->allowedSystems[] = $system;
 		return $this;
 	}
 
+	/**
+	 * @return list<array{name: string, class: class-string, internalName: string}>
+	 */
 	public function getAllowedImportSystems(): array {
 		if (!$this->allowedSystems) {
 			$this->addAllowedImportSystem([

lib/Service/Importer/BoardImportService.php

@@ -165,7 +165,7 @@ function (\stdClass $a) use ($trelloCard) {
 
 	private function sortComments(array $comments): array {
 		$comparison = function (\stdClass $a, \stdClass $b): int {
-			if ($a->date == $b->date) {
+			if ($a->date === $b->date) {
 				return 0;
 			}
 			return ($a->date < $b->date) ? -1 : 1;
@@ -352,7 +352,7 @@ private function replaceUsernames(string $text): string {
 	}
 
 	private function checklistItem(\stdClass $item): string {
-		if (($item->state == 'incomplete')) {
+		if (($item->state === 'incomplete')) {
 			$string_start = '- [ ]';
 		} else {
 			$string_start = '- [x]';

lib/Service/Importer/Systems/TrelloJsonService.php

@@ -152,13 +152,12 @@ protected function getSize($value): int {
 	}
 
 	/**
-	 * @param $rule
 	 * @param $field
 	 * @param $parameter
 	 * @return string
 	 */
-	protected function getErrorMessage($rule, $field, $parameter = null): string {
-		if (in_array($rule, ['max', 'min'])) {
+	protected function getErrorMessage(string $rule, $field, $parameter = null): string {
+		if (in_array($rule, ['max', 'min'], true)) {
 			return $rule === 'max'
 			? $field . ' cannot be longer than ' . $parameter . ' characters '
 			: $field . ' must be at least ' . $parameter . ' characters long ';