Skip to content

Commit 3d56382

Browse files
author
Paweł Piórkowski
committed
fix/OP-565 [Wishlist] Issue with multiple wishlists using the same token
1 parent e1bb162 commit 3d56382

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

spec/CommandHandler/Wishlist/CreateNewWishlistHandlerSpec.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@ public function it_creates_new_wishlist_for_user(
7474
$tokenUserResolver->resolve($token)->willReturn($shopUser);
7575

7676
$wishlistCookieTokenResolver->resolve()->willReturn('token');
77+
$wishlistCookieTokenResolver->new()->willReturn('token');
7778
$wishlistFactory->createForUser($shopUser)->willReturn($wishlist);
7879

7980
$wishlist->getShopUser()->willReturn($shopUser);
@@ -147,6 +148,7 @@ public function it_doesnt_add_duplicated_wishlist_name_for_user(
147148
$tokenUserResolver->resolve($token)->willReturn($shopUser);
148149

149150
$wishlistCookieTokenResolver->resolve()->willReturn('token');
151+
$wishlistCookieTokenResolver->new()->willReturn('token');
150152
$wishlistFactory->createForUser($shopUser)->willReturn($wishlist);
151153

152154
$wishlist->getShopUser()->willReturn($shopUser);

src/CommandHandler/Wishlist/CreateNewWishlistHandler.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,9 @@ public function __invoke(CreateNewWishlist $createNewWishlist): int
6161

6262
if ('' !== $wishlistCookieToken) {
6363
if ($user instanceof ShopUserInterface) {
64-
$wishlist->setToken($wishlistCookieToken);
65-
} else {
6664
$wishlist->setToken($this->wishlistCookieTokenResolver->new());
65+
} else {
66+
$wishlist->setToken($wishlistCookieToken);
6767
}
6868
}
6969

0 commit comments

Comments
 (0)