Browse Source

Merge branch 'develop' of https://forge.laclic.fr/Laclic/CaracoleBundle into develop

packProduct
Fabien Normand 3 years ago
parent
commit
144345af77
2 changed files with 9 additions and 4 deletions
  1. +6
    -2
      Factory/Order/OrderShopFactory.php
  2. +3
    -2
      Factory/Ticket/TicketFactory.php

+ 6
- 2
Factory/Order/OrderShopFactory.php View File

@@ -23,7 +23,6 @@ class OrderShopFactory extends AbstractFactory
VisitorInterface $visitor = null
): OrderShopInterface
{

$orderShop = $this->createBase($section);
$orderShopBelongTo = false;

@@ -38,7 +37,12 @@ class OrderShopFactory extends AbstractFactory
}

if (!$orderShopBelongTo) {
throw new \ErrorException('La commande doit être liée à un utilisateur ou à un visiteur.');
/*
* @TODO : lors de la première visite, le cookie visitor n'étant pas encore sur le navigateur du visiteur,
* visitor et user sont à null. Du coup, on ne peut pas déclencher cette exception. Voir s'il n'est pas
* possible d'avoir directement un visitor lors de la première visite.
*/
// throw new \ErrorException('La commande doit être liée à un utilisateur ou à un visiteur.');
}

return $orderShop;

+ 3
- 2
Factory/Ticket/TicketFactory.php View File

@@ -16,13 +16,14 @@ class TicketFactory extends SovTicketFactory
{
$ticket = parent::create();

if(!is_null($this->section)) {
if(isset($this->section) && !is_null($this->section)) {
$ticket->setSection($this->section);
}

if(!is_null($this->merchant)) {
if(isset($this->merchant) && !is_null($this->merchant)) {
$ticket->setMerchant($this->merchant);
}

return $ticket;
}
}

Loading…
Cancel
Save