Updated changelog

CHANGELOG.md

@@ -24,7 +24,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com), and this
 
 #### Fixed
 
-* *Nothing*
+* [#837](https://github.com/shlinkio/shlink/issues/837) Drastically improved performance when creating a new shortUrl and providing `findIfExists = true`.
 
 ## 2.3.0 - 2020-08-09
 

module/Core/src/Repository/ShortUrlRepository.php

@@ -241,7 +241,7 @@ DQL;
 
         foreach ($tags as $index => $tag) {
             $alias = 't_' . $index;
-            $qb->join('s.tags', $alias, Join::WITH, $qb->expr()->eq($alias . '.name', ':tag' . $index))
+            $qb->join('s.tags', $alias, Join::WITH, $alias . '.name = :tag' . $index)
                ->setParameter('tag' . $index, $tag);
         }
 

module/Core/test/Entity/ShortUrlTest.php

@@ -4,7 +4,6 @@ declare(strict_types=1);
 
 namespace ShlinkioTest\Shlink\Core\Entity;
 
-use Cake\Chronos\Chronos;
 use PHPUnit\Framework\TestCase;
 use Shlinkio\Shlink\Core\Entity\ShortUrl;
 use Shlinkio\Shlink\Core\Exception\ShortCodeCannotBeRegeneratedException;

module/Core/test/Service/UrlShortenerTest.php

@@ -21,8 +21,6 @@ use Shlinkio\Shlink\Core\Repository\ShortUrlRepository;
 use Shlinkio\Shlink\Core\Service\UrlShortener;
 use Shlinkio\Shlink\Core\Util\UrlValidatorInterface;
 
-use function array_map;
-
 class UrlShortenerTest extends TestCase
 {
     private UrlShortener $urlShortener;