minor #1491 More consistent code style (shuraknows)

This PR was squashed before being merged into the main branch.

Discussion
----------

More consistent code style

Multiple-line code blocks is being separated with new line from single-line code blocks.

Also php-cs-fixer suggested fix: functions with default null arguments should accept null value.

Commits
-------

cb82e93 More consistent code style

src/Command/AddUserCommand.php

@@ -138,6 +138,7 @@ protected function interact(InputInterface $input, OutputInterface $output): voi
 
         // Ask for the email if it's not defined
         $email = $input->getArgument('email');
+
         if (null !== $email) {
             $this->io->text(' > <info>Email</info>: '.$email);
         } else {
@@ -147,6 +148,7 @@ protected function interact(InputInterface $input, OutputInterface $output): voi
 
         // Ask for the full name if it's not defined
         $fullName = $input->getArgument('full-name');
+
         if (null !== $fullName) {
             $this->io->text(' > <info>Full Name</info>: '.$fullName);
         } else {
@@ -198,6 +200,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $this->io->success(sprintf('%s was successfully created: %s (%s)', $isAdmin ? 'Administrator user' : 'User', $user->getUsername(), $user->getEmail()));
 
         $event = $stopwatch->stop('add-user-command');
+
         if ($output->isVerbose()) {
             $this->io->comment(sprintf('New user database id: %d / Elapsed time: %.2f ms / Consumed memory: %.2f MB', $user->getId(), $event->getDuration(), $event->getMemory() / (1024 ** 2)));
         }

src/Controller/BlogController.php

@@ -51,9 +51,11 @@ final class BlogController extends AbstractController
     public function index(Request $request, int $page, string $_format, PostRepository $posts, TagRepository $tags): Response
     {
         $tag = null;
+
         if ($request->query->has('tag')) {
             $tag = $tags->findOneBy(['name' => $request->query->get('tag')]);
         }
+
         $latestPosts = $posts->findLatest($page, $tag);
 
         // Every template name also has two extensions that specify the format and

src/DataFixtures/AppFixtures.php

@@ -135,6 +135,7 @@ private function getTagData(): array
     private function getPostData(): array
     {
         $posts = [];
+
         foreach ($this->getPhrases() as $i => $title) {
             // $postData = [$title, $slug, $summary, $content, $publishedAt, $author, $tags, $comments];
 

src/Entity/Post.php

@@ -154,6 +154,7 @@ public function getComments(): Collection
     public function addComment(Comment $comment): void
     {
         $comment->setPost($this);
+
         if (!$this->comments->contains($comment)) {
             $this->comments->add($comment);
         }

src/EventSubscriber/CommentNotificationSubscriber.php

@@ -63,6 +63,7 @@ public function onCommentCreated(CommentCreatedEvent $event): void
         ], UrlGeneratorInterface::ABSOLUTE_URL);
 
         $subject = $this->translator->trans('notification.comment_created');
+
         $body = $this->translator->trans('notification.comment_created.description', [
             'title' => $post->getTitle(),
             'link' => $linkToPost,

src/EventSubscriber/RedirectToPreferredLocaleSubscriber.php

@@ -37,9 +37,10 @@ final class RedirectToPreferredLocaleSubscriber implements EventSubscriberInterf
     public function __construct(
         private readonly UrlGeneratorInterface $urlGenerator,
         string $locales,
-        string $defaultLocale = null
+        ?string $defaultLocale = null
     ) {
         $this->locales = explode('|', trim($locales));
+
         if (empty($this->locales)) {
             throw new \UnexpectedValueException('The list of supported locales must not be empty.');
         }
@@ -75,6 +76,7 @@ public function onKernelRequest(RequestEvent $event): void
         // Ignore requests from referrers with the same HTTP host in order to prevent
         // changing language for users who possibly already selected it for this application.
         $referrer = $request->headers->get('referer');
+
         if (null !== $referrer && u($referrer)->ignoreCase()->startsWith($request->getSchemeAndHttpHost())) {
             return;
         }

src/Form/DataTransformer/TagArrayToStringTransformer.php

@@ -61,7 +61,9 @@ public function reverseTransform($string): array
         $tags = $this->tags->findBy([
             'name' => $names,
         ]);
+
         $newNames = array_diff($names, $tags);
+
         foreach ($newNames as $name) {
             $tags[] = new Tag($name);
 

src/Repository/PostRepository.php

@@ -39,7 +39,7 @@ public function __construct(ManagerRegistry $registry)
         parent::__construct($registry, Post::class);
     }
 
-    public function findLatest(int $page = 1, Tag $tag = null): Paginator
+    public function findLatest(int $page = 1, ?Tag $tag = null): Paginator
     {
         $qb = $this->createQueryBuilder('p')
             ->addSelect('a', 't')

src/Twig/AppExtension.php

@@ -64,6 +64,7 @@ public function getLocales(): array
         }
 
         $this->locales = [];
+
         foreach ($this->localeCodes as $localeCode) {
             $this->locales[] = ['code' => $localeCode, 'name' => Locales::getName($localeCode, $localeCode)];
         }

src/Twig/SourceCodeExtension.php

@@ -65,6 +65,7 @@ public function getFunctions(): array
     public function linkSourceFile(Environment $twig, string $file, int $line): string
     {
         $text = str_replace('\\', '/', $file);
+
         if (str_starts_with($text, $this->projectDir)) {
             $text = mb_substr($text, mb_strlen($this->projectDir));
         }
@@ -180,6 +181,7 @@ private function unindentCode(string $code): string
         });
 
         $codeIsIndented = \count($indentedOrBlankLines) === \count($codeLines);
+
         if ($codeIsIndented) {
             $unindentedLines = array_map(static function ($lineOfCode) {
                 return u($lineOfCode)->after('    ');