fix phpcs

Signed-off-by: Luka Trovic <luka@nextcloud.com>

lib/Activity/ActivityManager.php

@@ -100,7 +100,7 @@ class ActivityManager {
 		StackMapper $stackMapper,
 		AclMapper $aclMapper,
 		IFactory $l10nFactory,
-		?string $userId
+		?string $userId,
 	) {
 		$this->manager = $manager;
 		$this->permissionService = $permissionsService;
@@ -484,7 +484,7 @@ class ActivityManager {
 					$objectId = $entity->getObjectId();
 					break;
 				default:
-					throw new InvalidArgumentException('No entity relation present for '. $className . ' to ' . $objectType);
+					throw new InvalidArgumentException('No entity relation present for ' . $className . ' to ' . $objectType);
 			}
 			return $this->cardMapper->find($objectId);
 		}
@@ -499,11 +499,11 @@ class ActivityManager {
 					$objectId = $entity->getBoardId();
 					break;
 				default:
-					throw new InvalidArgumentException('No entity relation present for '. $className . ' to ' . $objectType);
+					throw new InvalidArgumentException('No entity relation present for ' . $className . ' to ' . $objectType);
 			}
 			return $this->boardMapper->find($objectId);
 		}
-		throw new InvalidArgumentException('No entity relation present for '. $className . ' to ' . $objectType);
+		throw new InvalidArgumentException('No entity relation present for ' . $className . ' to ' . $objectType);
 	}
 
 	private function findDetailsForStack($stackId) {

lib/Activity/Filter.php

@@ -15,7 +15,7 @@ class Filter implements \OCP\Activity\IFilter {
 
 	public function __construct(
 		IL10N $l10n,
-		IURLGenerator $urlGenerator
+		IURLGenerator $urlGenerator,
 	) {
 		$this->l10n = $l10n;
 		$this->urlGenerator = $urlGenerator;

lib/Command/BoardImport.php

@@ -15,7 +15,7 @@ use Symfony\Component\Console\Output\OutputInterface;
 
 class BoardImport extends Command {
 	public function __construct(
-		private BoardImportCommandService $boardImportCommandService
+		private BoardImportCommandService $boardImportCommandService,
 	) {
 		parent::__construct();
 	}

lib/Command/TransferOwnership.php

@@ -82,7 +82,7 @@ final class TransferOwnership extends Command {
 		}
 
 		if ($boardId) {
-			$output->writeln('Transfer board ' . $board->getTitle() . ' from '. $board->getOwner() ." to $newOwner");
+			$output->writeln('Transfer board ' . $board->getTitle() . ' from ' . $board->getOwner() . " to $newOwner");
 		} else {
 			$output->writeln("Transfer all boards from $owner to $newOwner");
 		}
@@ -94,7 +94,7 @@ final class TransferOwnership extends Command {
 
 		if ($boardId) {
 			$this->boardService->transferBoardOwnership($boardId, $newOwner, $remapAssignment);
-			$output->writeln('<info>Board ' . $board->getTitle() . ' from '. $board->getOwner() ." transferred to $newOwner completed</info>");
+			$output->writeln('<info>Board ' . $board->getTitle() . ' from ' . $board->getOwner() . " transferred to $newOwner completed</info>");
 			return 0;
 		}
 

lib/Controller/SessionController.php

@@ -21,7 +21,8 @@ use OCP\AppFramework\OCSController;
 use OCP\IRequest;
 
 class SessionController extends OCSController {
-	public function __construct($appName,
+	public function __construct(
+		$appName,
 		IRequest $request,
 		private SessionService $sessionService,
 		private PermissionService $permissionService,

lib/Cron/ScheduledNotifications.php

@@ -20,7 +20,7 @@ class ScheduledNotifications extends Job {
 		ITimeFactory $time,
 		protected CardMapper $cardMapper,
 		protected NotificationHelper $notificationHelper,
-		protected LoggerInterface $logger
+		protected LoggerInterface $logger,
 	) {
 		parent::__construct($time);
 	}

lib/DAV/DeckCalendarBackend.php

@@ -32,7 +32,7 @@ class DeckCalendarBackend {
 
 	public function __construct(
 		BoardService $boardService, StackService $stackService, CardService $cardService, PermissionService $permissionService,
-		BoardMapper $boardMapper
+		BoardMapper $boardMapper,
 	) {
 		$this->boardService = $boardService;
 		$this->stackService = $stackService;

lib/Db/BoardMapper.php

@@ -31,7 +31,7 @@ class BoardMapper extends QBMapper implements IPermissionMapper {
 		private IUserManager $userManager,
 		private IGroupManager $groupManager,
 		private CirclesService $circlesService,
-		private LoggerInterface $logger
+		private LoggerInterface $logger,
 	) {
 		parent::__construct($db, 'deck_boards', Board::class);
 

lib/Db/CardMapper.php

@@ -45,7 +45,7 @@ class CardMapper extends QBMapper implements IPermissionMapper {
 		IManager $notificationManager,
 		ICacheFactory $cacheFactory,
 		$databaseType = 'sqlite3',
-		$database4ByteSupport = true
+		$database4ByteSupport = true,
 	) {
 		parent::__construct($db, 'deck_cards', Card::class);
 		$this->labelMapper = $labelMapper;
@@ -402,7 +402,7 @@ class CardMapper extends QBMapper implements IPermissionMapper {
 		if (count($query->getTag())) {
 			foreach ($query->getTag() as $index => $tag) {
 				$qb->innerJoin('c', 'deck_assigned_labels', 'al' . $index, $qb->expr()->eq('c.id', 'al' . $index . '.card_id'));
-				$qb->innerJoin('al'. $index, 'deck_labels', 'l' . $index, $qb->expr()->eq('al' . $index . '.label_id', 'l' . $index . '.id'));
+				$qb->innerJoin('al' . $index, 'deck_labels', 'l' . $index, $qb->expr()->eq('al' . $index . '.label_id', 'l' . $index . '.id'));
 				$qb->andWhere($qb->expr()->iLike('l' . $index . '.title', $qb->createNamedParameter('%' . $this->db->escapeLikeParameter($tag->getValue()) . '%', IQueryBuilder::PARAM_STR)));
 			}
 		}

lib/Db/ChangeHelper.php

@@ -24,7 +24,7 @@ class ChangeHelper {
 		IDBConnection $db,
 		ICacheFactory $cacheFactory,
 		IRequest $request,
-		?string $userId
+		?string $userId,
 	) {
 		$this->db = $db;
 		$this->cache = $cacheFactory->createDistributed('deck_changes');
@@ -43,7 +43,7 @@ class ChangeHelper {
 	public function cardChanged($cardId, $updateCard = true) {
 		$time = time();
 		$etag = md5($time . microtime());
-		$this->cache->set(self::TYPE_CARD . '-' .$cardId, $etag);
+		$this->cache->set(self::TYPE_CARD . '-' . $cardId, $etag);
 		if ($updateCard) {
 			$sql = 'UPDATE `*PREFIX*deck_cards` SET `last_modified` = ?, `last_editor` = ? WHERE `id` = ?';
 			$this->db->executeUpdate($sql, [time(), $this->userId, $cardId]);
@@ -60,7 +60,7 @@ class ChangeHelper {
 	public function stackChanged($stackId, $updateBoard = true) {
 		$time = time();
 		$etag = md5($time . microtime());
-		$this->cache->set(self::TYPE_CARD . '-' .$stackId, $etag);
+		$this->cache->set(self::TYPE_CARD . '-' . $stackId, $etag);
 		if ($updateBoard) {
 			$sql = 'UPDATE `*PREFIX*deck_stacks` SET `last_modified` = ? WHERE `id` = ?';
 			$this->db->executeUpdate($sql, [time(), $stackId]);
@@ -76,7 +76,7 @@ class ChangeHelper {
 	}
 
 	public function getEtag($type, $id) {
-		$entry = $this->cache->get($type . '-' .$id);
+		$entry = $this->cache->get($type . '-' . $id);
 		if ($entry === 'null') {
 			return '';
 		}

lib/Db/StackMapper.php

@@ -24,7 +24,7 @@ class StackMapper extends DeckMapper implements IPermissionMapper {
 	public function __construct(
 		IDBConnection $db,
 		CardMapper $cardMapper,
-		ICacheFactory $cacheFactory
+		ICacheFactory $cacheFactory,
 	) {
 		parent::__construct($db, 'deck_stacks', Stack::class);
 		$this->cardMapper = $cardMapper;

lib/Listeners/LiveUpdateListener.php

@@ -38,7 +38,7 @@ class LiveUpdateListener implements IEventListener {
 		IRequest $request,
 		LoggerInterface $logger,
 		SessionService $sessionService,
-		StackMapper $stackMapper
+		StackMapper $stackMapper,
 	) {
 		try {
 			$this->queue = $container->get(IQueue::class);

lib/Migration/Version11000Date20240222115515.php

@@ -20,13 +20,13 @@ class Version11000Date20240222115515 extends SimpleMigrationStep {
 		$returnValue = null;
 
 		$assignedUsersTable = $schema->getTable('deck_assigned_users');
-		if($assignedUsersTable->hasIndex('deck_assigned_users_idx_c')) {
+		if ($assignedUsersTable->hasIndex('deck_assigned_users_idx_c')) {
 			$assignedUsersTable->dropIndex('deck_assigned_users_idx_c');
 			$returnValue = $schema;
 		}
 
 		$boardAclTable = $schema->getTable('deck_board_acl');
-		if($boardAclTable->hasIndex('deck_board_acl_idx_i')) {
+		if ($boardAclTable->hasIndex('deck_board_acl_idx_i')) {
 			$boardAclTable->dropIndex('deck_board_acl_idx_i');
 			$returnValue = $schema;
 		}

lib/Notification/NotificationHelper.php

@@ -58,7 +58,7 @@ class NotificationHelper {
 		IConfig $config,
 		IManager $notificationManager,
 		IGroupManager $groupManager,
-		$userId
+		$userId,
 	) {
 		$this->cardMapper = $cardMapper;
 		$this->boardMapper = $boardMapper;

lib/Notification/Notifier.php

@@ -36,7 +36,7 @@ class Notifier implements INotifier {
 		IUserManager $userManager,
 		CardMapper $cardMapper,
 		StackMapper $stackMapper,
-		BoardMapper $boardMapper
+		BoardMapper $boardMapper,
 	) {
 		$this->l10nFactory = $l10nFactory;
 		$this->url = $url;

lib/Provider/DeckProvider.php

@@ -47,7 +47,7 @@ class DeckProvider implements IFullTextSearchProvider {
 	 * @param FullTextSearchService $fullTextSearchService
 	 */
 	public function __construct(
-		IL10N $l10n, IUrlGenerator $urlGenerator, FullTextSearchService $fullTextSearchService
+		IL10N $l10n, IUrlGenerator $urlGenerator, FullTextSearchService $fullTextSearchService,
 	) {
 		$this->l10n = $l10n;
 		$this->urlGenerator = $urlGenerator;

lib/Reference/CreateCardReferenceProvider.php

@@ -17,7 +17,8 @@ class CreateCardReferenceProvider extends ADiscoverableReferenceProvider {
 	public function __construct(
 		private IL10N $l10n,
 		private IURLGenerator $urlGenerator,
-		private ?string $userId) {
+		private ?string $userId,
+	) {
 	}
 
 	/**

lib/Search/CardCommentProvider.php

@@ -25,7 +25,7 @@ class CardCommentProvider implements IProvider {
 
 	public function __construct(
 		SearchService $searchService,
-		IL10N $l10n
+		IL10N $l10n,
 	) {
 		$this->searchService = $searchService;
 		$this->l10n = $l10n;

lib/Search/DeckProvider.php

@@ -27,7 +27,7 @@ class DeckProvider implements IProvider {
 	public function __construct(
 		SearchService $searchService,
 		IURLGenerator $urlGenerator,
-		IL10N $l10n
+		IL10N $l10n,
 	) {
 		$this->l10n = $l10n;
 		$this->searchService = $searchService;

lib/Service/AssignmentService.php

@@ -76,7 +76,7 @@ class AssignmentService {
 		ChangeHelper $changeHelper,
 		IEventDispatcher $eventDispatcher,
 		AssignmentServiceValidator $assignmentServiceValidator,
-		$userId
+		$userId,
 	) {
 		$this->assignmentServiceValidator = $assignmentServiceValidator;
 		$this->permissionService = $permissionService;

lib/Service/AttachmentService.php

@@ -58,7 +58,7 @@ class AttachmentService {
 		$userId,
 		IL10N $l10n,
 		ActivityManager $activityManager,
-		AttachmentServiceValidator $attachmentServiceValidator
+		AttachmentServiceValidator $attachmentServiceValidator,
 	) {
 		$this->attachmentMapper = $attachmentMapper;
 		$this->cardMapper = $cardMapper;

lib/Service/BoardService.php

@@ -89,7 +89,7 @@ class BoardService {
 		IDBConnection $connection,
 		BoardServiceValidator $boardServiceValidator,
 		SessionMapper $sessionMapper,
-		?string $userId
+		?string $userId,
 	) {
 		$this->boardMapper = $boardMapper;
 		$this->stackMapper = $stackMapper;

lib/Service/CardService.php

@@ -77,7 +77,7 @@ class CardService {
 		LoggerInterface $logger,
 		IRequest $request,
 		CardServiceValidator $cardServiceValidator,
-		?string $userId
+		?string $userId,
 	) {
 		$this->cardMapper = $cardMapper;
 		$this->stackMapper = $stackMapper;
@@ -356,11 +356,11 @@ class CardService {
 		$card = $this->cardMapper->update($card);
 		$oldBoardId = $this->stackMapper->findBoardId($changes->getBefore()->getStackId());
 		$boardId = $this->cardMapper->findBoardId($card->getId());
-		if($boardId !== $oldBoardId) {
+		if ($boardId !== $oldBoardId) {
 			$stack = $this->stackMapper->find($card->getStackId());
 			$board = $this->boardService->find($this->cardMapper->findBoardId($card->getId()));
 			$boardLabels = $board->getLabels() ?? [];
-			foreach($card->getLabels() as $cardLabel) {
+			foreach ($card->getLabels() as $cardLabel) {
 				$this->removeLabel($card->getId(), $cardLabel->getId());
 				$label = $this->labelMapper->find($cardLabel->getId());
 				$filteredLabels = array_values(array_filter($boardLabels, fn ($item) => $item->getTitle() === $label->getTitle()));

lib/Service/ConfigService.php

@@ -29,7 +29,7 @@ class ConfigService {
 
 	public function __construct(
 		IConfig $config,
-		IGroupManager $groupManager
+		IGroupManager $groupManager,
 	) {
 		$this->groupManager = $groupManager;
 		$this->config = $config;

lib/Service/DefaultBoardService.php

@@ -27,7 +27,7 @@ class DefaultBoardService {
 		BoardService $boardService,
 		StackService $stackService,
 		CardService $cardService,
-		IConfig $config
+		IConfig $config,
 	) {
 		$this->boardService = $boardService;
 		$this->stackService = $stackService;

lib/Service/FileService.php

@@ -33,7 +33,7 @@ class FileService implements IAttachmentService {
 		private IRootFolder $rootFolder,
 		private IConfig $config,
 		private AttachmentMapper $attachmentMapper,
-		private IMimeTypeDetector $mimeTypeDetector
+		private IMimeTypeDetector $mimeTypeDetector,
 	) {
 	}
 

lib/Service/FilesAppService.php

@@ -55,7 +55,7 @@ class FilesAppService implements IAttachmentService, ICustomAttachmentService {
 		CardMapper  $cardMapper,
 		LoggerInterface $logger,
 		IDBConnection $connection,
-		?string $userId
+		?string $userId,
 	) {
 		$this->request = $request;
 		$this->l10n = $l10n;

lib/Service/FullTextSearchService.php

@@ -39,7 +39,7 @@ class FullTextSearchService {
 	private $cardMapper;
 
 	public function __construct(
-		BoardMapper $boardMapper, StackMapper $stackMapper, CardMapper $cardMapper
+		BoardMapper $boardMapper, StackMapper $stackMapper, CardMapper $cardMapper,
 	) {
 		$this->boardMapper = $boardMapper;
 		$this->stackMapper = $stackMapper;

lib/Service/Importer/BoardImportService.php

@@ -70,7 +70,7 @@ class BoardImportService {
 		private CardMapper $cardMapper,
 		private ICommentsManager $commentsManager,
 		private IEventDispatcher $eventDispatcher,
-		private LoggerInterface $logger
+		private LoggerInterface $logger,
 	) {
 		$this->board = new Board();
 		$this->disableCommentsEvents();
@@ -364,7 +364,7 @@ class BoardImportService {
 					$this->getImportSystem()->updateCardAssignment($cardId, (string)$assignment->getId(), $assignment);
 					$this->addOutput('Assignment ' . $assignment->getParticipant() . ' added');
 				} catch (NotFoundException $e) {
-					$this->addError('No origin or mapping found for card "' . $cardId . '" and ' . $assignment->getTypeString() .' assignment "' . $assignment->getParticipant(), $e);
+					$this->addError('No origin or mapping found for card "' . $cardId . '" and ' . $assignment->getTypeString() . ' assignment "' . $assignment->getParticipant(), $e);
 				}
 			}
 		}

lib/Service/Importer/Systems/TrelloApiService.php

@@ -31,7 +31,7 @@ class TrelloApiService extends TrelloJsonService {
 		IURLGenerator $urlGenerator,
 		IL10N $l10n,
 		LoggerInterface $logger,
-		IClientService $httpClientService
+		IClientService $httpClientService,
 	) {
 		parent::__construct($userManager, $urlGenerator, $l10n);
 		$this->logger = $logger;

lib/Service/Importer/Systems/TrelloJsonService.php

@@ -37,7 +37,7 @@ class TrelloJsonService extends ABoardImportService {
 	public function __construct(
 		IUserManager $userManager,
 		IURLGenerator $urlGenerator,
-		IL10N $l10n
+		IL10N $l10n,
 	) {
 		$this->userManager = $userManager;
 		$this->urlGenerator = $urlGenerator;

lib/Service/LabelService.php

@@ -32,7 +32,7 @@ class LabelService {
 		PermissionService $permissionService,
 		BoardService $boardService,
 		ChangeHelper $changeHelper,
-		LabelServiceValidator $labelServiceValidator
+		LabelServiceValidator $labelServiceValidator,
 	) {
 		$this->labelMapper = $labelMapper;
 		$this->permissionService = $permissionService;

lib/Service/OverviewService.php

@@ -36,7 +36,7 @@ class OverviewService {
 		AssignmentMapper $assignedUsersMapper,
 		IUserManager $userManager,
 		ICommentsManager $commentsManager,
-		AttachmentService $attachmentService
+		AttachmentService $attachmentService,
 	) {
 		$this->cardService = $cardService;
 		$this->boardMapper = $boardMapper;

lib/Service/PermissionService.php

@@ -40,7 +40,7 @@ class PermissionService {
 		private IGroupManager $groupManager,
 		private IManager $shareManager,
 		private IConfig $config,
-		private ?string $userId
+		private ?string $userId,
 	) {
 		$this->boardCache = new CappedMemoryCache();
 		$this->permissionCache = new CappedMemoryCache();

lib/Service/SearchService.php

@@ -46,7 +46,7 @@ class SearchService {
 		FilterStringParser $filterStringParser,
 		IUserManager $userManager,
 		IL10N $l10n,
-		IURLGenerator $urlGenerator
+		IURLGenerator $urlGenerator,
 	) {
 		$this->boardService = $boardService;
 		$this->cardMapper = $cardMapper;

lib/Service/SessionService.php

@@ -33,7 +33,7 @@ class SessionService {
 		ISecureRandom $secureRandom,
 		ITimeFactory $timeFactory,
 		$userId,
-		IEventDispatcher $eventDispatcher
+		IEventDispatcher $eventDispatcher,
 	) {
 		$this->sessionMapper = $sessionMapper;
 		$this->secureRandom = $secureRandom;

lib/Service/StackService.php

@@ -56,7 +56,7 @@ class StackService {
 		ChangeHelper $changeHelper,
 		LoggerInterface $logger,
 		IEventDispatcher $eventDispatcher,
-		StackServiceValidator $stackServiceValidator
+		StackServiceValidator $stackServiceValidator,
 	) {
 		$this->stackMapper = $stackMapper;
 		$this->boardMapper = $boardMapper;

lib/Sharing/DeckShareProvider.php

@@ -69,7 +69,7 @@ class DeckShareProvider implements \OCP\Share\IShareProvider {
 		IL10N $l,
 		ITimeFactory $timeFactory,
 		IMimeTypeLoader $mimeTypeLoader,
-		?string $userId
+		?string $userId,
 	) {
 		$this->dbConnection = $connection;
 		$this->shareManager = $shareManager;
@@ -174,7 +174,7 @@ class DeckShareProvider implements \OCP\Share\IShareProvider {
 		string $target,
 		int $permissions,
 		string $token,
-		?\DateTime $expirationDate
+		?\DateTime $expirationDate,
 	): int {
 		$qb = $this->dbConnection->getQueryBuilder();
 		$qb->insert('share')

lib/Teams/DeckTeamResourceProvider.php

@@ -59,6 +59,6 @@ class DeckTeamResourceProvider implements \OCP\Teams\ITeamResourceProvider {
 	}
 
 	public function getBoardBulletIcon(Board $board): string {
-		return '<svg xmlns="http://www.w3.org/2000/svg" height="16" width="16" version="1.1" viewBox="0 0 16 16"><g fill="#' . $board->getColor(). '"><rect ry="15" height="14" width="14" y="1" x="1"/></g></svg>';
+		return '<svg xmlns="http://www.w3.org/2000/svg" height="16" width="16" version="1.1" viewBox="0 0 16 16"><g fill="#' . $board->getColor() . '"><rect ry="15" height="14" width="14" y="1" x="1"/></g></svg>';
 	}
 }

lib/Validators/BaseValidator.php

@@ -45,7 +45,7 @@ abstract class BaseValidator {
 					} else {
 						if (!$this->{$rule}($value)) {
 							throw new BadRequestException(
-								$field . ' must be provided and must be '. str_replace('_', ' ', $rule));
+								$field . ' must be provided and must be ' . str_replace('_', ' ', $rule));
 						}
 					}
 				}
@@ -154,10 +154,10 @@ abstract class BaseValidator {
 	protected function getErrorMessage($rule, $field, $parameter = null): string {
 		if (in_array($rule, ['max', 'min'])) {
 			return $rule === 'max'
-			? $field . ' cannot be longer than '. $parameter . ' characters '
-			: $field . ' must be at least '. $parameter . ' characters long ';
+			? $field . ' cannot be longer than ' . $parameter . ' characters '
+			: $field . ' must be at least ' . $parameter . ' characters long ';
 		}
 
-		return $field . ' must be provided and must be '. str_replace('_', ' ', $rule);
+		return $field . ' must be provided and must be ' . str_replace('_', ' ', $rule);
 	}
 }

tests/integration/features/bootstrap/BoardContext.php

@@ -71,7 +71,7 @@ class BoardContext implements Context {
 					break;
 				}
 			}
-			Assert::assertNotNull($id, 'Could not find board named '.$boardName);
+			Assert::assertNotNull($id, 'Could not find board named ' . $boardName);
 		} else {
 			$id = $this->board['id'];
 		}
@@ -259,7 +259,7 @@ class BoardContext implements Context {
 	}
 
 	private function assignToCard($participant, $type) {
-		$this->requestContext->sendJSONrequest('POST', '/index.php/apps/deck/cards/' . $this->card['id'] .'/assign', [
+		$this->requestContext->sendJSONrequest('POST', '/index.php/apps/deck/cards/' . $this->card['id'] . '/assign', [
 			'userId' => $participant,
 			'type' => $type
 		]);
@@ -274,7 +274,7 @@ class BoardContext implements Context {
 			return $label['title'] === $tag;
 		});
 		$label = array_shift($filteredLabels);
-		$this->requestContext->sendJSONrequest('POST', '/index.php/apps/deck/cards/' . $this->card['id'] .'/label/' . $label['id']);
+		$this->requestContext->sendJSONrequest('POST', '/index.php/apps/deck/cards/' . $this->card['id'] . '/label/' . $label['id']);
 		$this->requestContext->getResponse()->getBody()->seek(0);
 	}
 

tests/integration/features/bootstrap/CommentContext.php

@@ -45,7 +45,7 @@ class CommentContext implements Context {
 	 */
 	public function updateACommentWithContentOnTheCard($content) {
 		$card = $this->boardContext->getLastUsedCard();
-		$this->requestContext->sendOCSRequest('PUT', '/apps/deck/api/v1.0/cards/' . $card['id'] . '/comments/'. $this->lastComment['id'], [
+		$this->requestContext->sendOCSRequest('PUT', '/apps/deck/api/v1.0/cards/' . $card['id'] . '/comments/' . $this->lastComment['id'], [
 			'message' => $content,
 			'parentId' => null
 		]);
@@ -56,7 +56,7 @@ class CommentContext implements Context {
 	 */
 	public function deleteTheCommentOnTheCard() {
 		$card = $this->boardContext->getLastUsedCard();
-		$this->requestContext->sendOCSRequest('DELETE', '/apps/deck/api/v1.0/cards/' . $card['id'] . '/comments/'. $this->lastComment['id']);
+		$this->requestContext->sendOCSRequest('DELETE', '/apps/deck/api/v1.0/cards/' . $card['id'] . '/comments/' . $this->lastComment['id']);
 	}
 
 }

tests/unit/Activity/ActivityManagerTest.php

@@ -103,13 +103,13 @@ class ActivityManagerTest extends TestCase {
 				if ($format !== '') {
 					$this->assertStringContainsString('{user}', $format);
 				} else {
-					self::addWarning('No activity string found for '. $constant);
+					self::addWarning('No activity string found for ' . $constant);
 				}
 				$format = $this->activityManager->getActivityFormat('cz', $value, [], true);
 				if ($format !== '') {
 					$this->assertStringStartsWith('You', $format);
 				} else {
-					self::addWarning('No own activity string found for '. $constant);
+					self::addWarning('No own activity string found for ' . $constant);
 				}
 			}
 		}

tests/unit/Notification/NotifierTest.php

@@ -220,7 +220,7 @@ class NotifierTest extends \Test\TestCase {
 			->with('otheruser')
 			->willReturn($user);
 
-		$expectedMessage = 'The card "Card title" on "Board title" has been assigned to you by '.$dn.'.';
+		$expectedMessage = 'The card "Card title" on "Board title" has been assigned to you by ' . $dn . '.';
 		$notification->expects($this->once())
 			->method('setParsedSubject')
 			->with($expectedMessage);
@@ -304,7 +304,7 @@ class NotifierTest extends \Test\TestCase {
 			->with('otheruser')
 			->willReturn($user);
 
-		$expectedMessage = 'The board "Board title" has been shared with you by '.$dn.'.';
+		$expectedMessage = 'The board "Board title" has been shared with you by ' . $dn . '.';
 		$notification->expects($this->once())
 			->method('setParsedSubject')
 			->with($expectedMessage);