Opsrc 645/added editing controller

features/editing_name_of_wishlist.feature

@@ -8,13 +8,13 @@ Feature: Editing wishlists name
         Given the store operates on a single channel in "United States"
         Given I am on "/"
 
-    @ui
+    @ui @javascript
     Scenario: Editing wishlists name
         And the store has a wishlist named "Wishlist1"
         When I go to "/wishlists"
         Then I should see "Wishlist1"
-        And I follow edit for "Wishlist1"
-        And I fill in "edit_wishlist_name_name" with "Wishlist2"
-        When I press "edit_wishlist_name_save"
-        Then I should be on "/wishlists"
-        And I should see "Wishlist2"
+        When I press "wishlist-edit-button-Wishlist1"
+        And I fill in "edit_wishlist_name" with "Wishlist2"
+        When I press "edit_wishlist_save"
+        Then I go to "/wishlists"
+        And I should see "Wishlist2"

features/removing_wishlist.feature

@@ -8,18 +8,25 @@ Feature: Removing a wishlist
         Given the store operates on a single channel in "United States"
         Given I am on "/"
 
-    @ui
+    @ui @javascript
     Scenario: Removing a wishlist
         And the store has a wishlist named "Wishlist1"
         And the store has a wishlist named "Wishlist2"
         When I go to "/wishlists"
         Then I should have 3 wishlists
-        When I follow remove for "Wishlist1"
+        When I press "wishlist-delete-button-Wishlist1"
+        When I press "remove_wishlist_save"
+        Then I should be on "/wishlists"
+        Then I should wait for one second
         Then I should have 2 wishlists
 
-    @ui
+
+    @ui @javascript
     Scenario: Removing a wishlist with one existing
         When I go to "/wishlists"
         Then I should have 1 wishlists
-        When I follow remove for "Wishlist1"
-        Then I should have 1 wishlists
+        When I press "wishlist-delete-button-Wishlist"
+        When I press "remove_wishlist_save"
+        Then I should be on "/wishlists"
+        Then I should wait for one second
+        Then I should have 1 wishlists

src/Command/Wishlist/UpdateWishlistName.php

@@ -0,0 +1,39 @@
+<?php
+
+/*
+ * This file was created by developers working at BitBag
+ * Do you need more information about us and what we do? Visit our https://bitbag.io website!
+ * We are hiring developers from all over the world. Join us and start your new, exciting adventure and become part of us: https://bitbag.io/career
+*/
+
+declare(strict_types=1);
+
+namespace BitBag\SyliusWishlistPlugin\Command\Wishlist;
+
+use BitBag\SyliusWishlistPlugin\Entity\WishlistInterface;
+
+final class UpdateWishlistName implements UpdateWishlistNameInterface
+{
+    public string $name;
+
+    public ?string $channelCode = null;
+
+    private WishlistInterface $wishlist;
+
+    public function __construct(string $name, WishlistInterface $wishlist)
+    {
+        $this->name = $name;
+        $this->wishlist = $wishlist;
+    }
+
+    public function getName(): string
+    {
+        return $this->name;
+    }
+
+    public function getWishlist(): WishlistInterface
+    {
+        return $this->wishlist;
+    }
+}
+

src/Command/Wishlist/UpdateWishlistNameInterface.php

@@ -0,0 +1,20 @@
+<?php
+
+/*
+ * This file was created by developers working at BitBag
+ * Do you need more information about us and what we do? Visit our https://bitbag.io website!
+ * We are hiring developers from all over the world. Join us and start your new, exciting adventure and become part of us: https://bitbag.io/career
+*/
+
+declare(strict_types=1);
+
+namespace BitBag\SyliusWishlistPlugin\Command\Wishlist;
+
+use BitBag\SyliusWishlistPlugin\Entity\WishlistInterface;
+
+interface UpdateWishlistNameInterface
+{
+    public function getName(): string;
+
+    public function getWishlist(): WishlistInterface;
+}

src/CommandHandler/Wishlist/UpdateWishlistNameHandler.php

@@ -0,0 +1,43 @@
+<?php
+/*
+ * This file was created by developers working at BitBag
+ * Do you need more information about us and what we do? Visit our https://bitbag.io website!
+ * We are hiring developers from all over the world. Join us and start your new, exciting adventure and become part of us: https://bitbag.io/career
+*/
+declare(strict_types=1);
+
+namespace BitBag\SyliusWishlistPlugin\CommandHandler\Wishlist;
+
+use BitBag\SyliusWishlistPlugin\Command\Wishlist\UpdateWishlistName;
+use BitBag\SyliusWishlistPlugin\Exception\WishlistNameIsTakenException;
+use BitBag\SyliusWishlistPlugin\Repository\WishlistRepositoryInterface;
+use BitBag\SyliusWishlistPlugin\Resolver\WishlistCookieTokenResolverInterface;
+
+final class UpdateWishlistNameHandler
+{
+    private WishlistRepositoryInterface $wishlistRepository;
+
+    private WishlistCookieTokenResolverInterface $wishlistCookieTokenResolver;
+
+    public function __construct(
+        WishlistRepositoryInterface $wishlistRepository,
+        WishlistCookieTokenResolverInterface $wishlistCookieTokenResolver
+    ) {
+        $this->wishlistRepository = $wishlistRepository;
+        $this->wishlistCookieTokenResolver = $wishlistCookieTokenResolver;
+    }
+
+    public function __invoke(UpdateWishlistName $updateWishlistName): void
+    {
+        $wishlist = $updateWishlistName->getWishlist();
+
+        $wishlistCookieToken = $this->wishlistCookieTokenResolver->resolve();
+
+        if ($this->wishlistRepository->findOneByTokenAndName($wishlistCookieToken, $updateWishlistName->getName())) {
+            throw new WishlistNameIsTakenException();
+        }
+
+        $wishlist->setName($updateWishlistName->getName());
+        $this->wishlistRepository->add($wishlist);
+    }
+}

src/Controller/Action/UpdateWishlistNameAction.php

@@ -0,0 +1,73 @@
+<?php
+
+/*
+ * This file was created by developers working at BitBag
+ * Do you need more information about us and what we do? Visit our https://bitbag.io website!
+ * We are hiring developers from all over the world. Join us and start your new, exciting adventure and become part of us: https://bitbag.io/career
+*/
+
+declare(strict_types=1);
+
+namespace BitBag\SyliusWishlistPlugin\Controller\Action;
+
+use BitBag\SyliusWishlistPlugin\Command\Wishlist\UpdateWishlistName;
+use BitBag\SyliusWishlistPlugin\Repository\WishlistRepositoryInterface;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpFoundation\Session\Flash\FlashBagInterface;
+use Symfony\Component\Messenger\Exception\HandlerFailedException;
+use Symfony\Component\Messenger\MessageBusInterface;
+use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
+use Symfony\Contracts\Translation\TranslatorInterface;
+use Webmozart\Assert\Assert;
+
+final class UpdateWishlistNameAction
+{
+    private MessageBusInterface $commandBus;
+
+    private FlashBagInterface $flashBag;
+
+    private TranslatorInterface $translator;
+
+    private WishlistRepositoryInterface $wishlistRepository;
+
+    private UrlGeneratorInterface $urlGenerator;
+
+    public function __construct(
+        MessageBusInterface $commandBus,
+        FlashBagInterface $flashBag,
+        TranslatorInterface $translator,
+        WishlistRepositoryInterface $wishlistRepository,
+        UrlGeneratorInterface $urlGenerator
+    ) {
+        $this->commandBus = $commandBus;
+        $this->flashBag = $flashBag;
+        $this->translator = $translator;
+        $this->wishlistRepository = $wishlistRepository;
+        $this->urlGenerator = $urlGenerator;
+    }
+
+    public function __invoke(Request $request): Response
+    {
+        $wishlistName = $request->request->get('edit_wishlist_name')['name'];
+        Assert::string($wishlistName);
+        $wishlistId = $request->attributes->getInt('id');
+        $wishlist = $this->wishlistRepository->find($wishlistId);
+
+        try {
+            $updateWishlistName = new UpdateWishlistName($wishlistName, $wishlist);
+            $this->commandBus->dispatch($updateWishlistName);
+
+            $this->flashBag->add(
+                'success',
+                $this->translator->trans('bitbag_sylius_wishlist_plugin.ui.wishlist_name_changed')
+            );
+        } catch (HandlerFailedException $exception) {
+            $this->flashBag->add(
+                'error',
+                $this->translator->trans('bitbag_sylius_wishlist_plugin.ui.wishlist_name_already_exists')
+            );
+        }
+        return new Response($this->urlGenerator->generate('bitbag_sylius_wishlist_plugin_shop_wishlist_list_products'));
+    }
+}

src/Repository/WishlistRepository.php

@@ -48,7 +48,7 @@ public function findAllByToken(string $token): ?array
             ->getResult()
             ;
     }
-    
+
     public function findAllByShopUser(int $shopUser): ?array
     {
         return $this->createQueryBuilder('o')
@@ -114,4 +114,18 @@ public function findAllByAnonymousAndChannel(?string $token, ChannelInterface $c
 
         return $qb->getQuery()->getResult();
     }
+
+    public function findOneByTokenAndName(string $token, string $name): ?WishlistInterface
+    {
+        return $this->createQueryBuilder('o')
+            ->where('o.token = :token')
+            ->andWhere('o.name =:name')
+            ->setParameter('token', $token)
+            ->setParameter('name', $name)
+            ->getQuery()
+            ->setMaxResults(1)
+            ->getOneOrNullResult()
+            ;
+    }
+
 }

src/Repository/WishlistRepositoryInterface.php

@@ -35,4 +35,6 @@ public function findOneByShopUserAndChannel(
     ): ?WishlistInterface;
 
     public function findAllByAnonymousAndChannel(?string $token, ChannelInterface $channel): ?array;
+
+    public function findOneByTokenAndName(string $token, string $name): ?WishlistInterface;
 }