[Admin][Order] Add order notes

Sylius · Jun 22, 2016 · 6318c0e · 6318c0e
1 parent 0311ffc
commit 6318c0e
Show file tree

Hide file tree

Showing 31 changed files with 347 additions and 50 deletions.
diff --git a/app/migrations/Version20160622114217.php b/app/migrations/Version20160622114217.php
@@ -0,0 +1,38 @@
+<?php
+
+namespace Sylius\Migrations;
+
+use Doctrine\DBAL\Migrations\AbstractMigration;
+use Doctrine\DBAL\Schema\Schema;
+
+/**
+ * Auto-generated Migration: Please modify to your needs!
+ */
+class Version20160622114217 extends AbstractMigration
+{
+    /**
+     * @param Schema $schema
+     */
+    public function up(Schema $schema)
+    {
+        // this up() migration is auto-generated, please modify it to your needs
+        $this->abortIf($this->connection->getDatabasePlatform()->getName() != 'mysql', 'Migration can only be executed safely on \'mysql\'.');
+
+        $this->addSql('DROP INDEX fulltext_search_idx ON sylius_search_index');
+        $this->addSql('CREATE INDEX fulltext_search_idx ON sylius_search_index (item_id)');
+        $this->addSql('ALTER TABLE sylius_order DROP additional_information, CHANGE notes notes VARCHAR(1000) DEFAULT NULL');
+    }
+
+    /**
+     * @param Schema $schema
+     */
+    public function down(Schema $schema)
+    {
+        // this down() migration is auto-generated, please modify it to your needs
+        $this->abortIf($this->connection->getDatabasePlatform()->getName() != 'mysql', 'Migration can only be executed safely on \'mysql\'.');
+
+        $this->addSql('ALTER TABLE sylius_order ADD additional_information VARCHAR(1000) DEFAULT NULL COLLATE utf8_unicode_ci, CHANGE notes notes VARCHAR(255) DEFAULT NULL COLLATE utf8_unicode_ci');
+        $this->addSql('DROP INDEX fulltext_search_idx ON sylius_search_index');
+        $this->addSql('CREATE FULLTEXT INDEX fulltext_search_idx ON sylius_search_index (value)');
+    }
+}
diff --git a/features/order/placing_an_order/adding_notes_for_customer_service.feature b/features/order/placing_an_order/adding_notes_for_customer_service.feature
@@ -0,0 +1,23 @@
+@placing_orders
+Feature: Adding a note to order
+    In order to provide some extra information to order
+    As a Customer
+    I want to be able to provide a note for customer service
+
+    Background:
+        Given the store operates on a single channel in "France"
+        And the store has a product "PHP T-Shirt" priced at "$19.99"
+        And the store ships everywhere for free
+        And the store allows paying offline
+        And the store has customer service account identified by "sylius@example.com"
+        And there is a customer account "customer@example.com" identified by "sylius"
+        And I am logged in as "customer@example.com"
+
+    @ui
+    Scenario: Adding note on the checkout summary step
+        Given I have product "PHP T-Shirt" in the cart
+        And I specified the shipping address as "Ankh Morpork", "Frost Alley", "90210", "France" for "Jon Snow"
+        And I proceed order with "Free" shipping method and "Offline" payment
+        When I provide additional note like "Code to the front gateway is #44*"
+        And I confirm my order
+        Then the customer service should know about this additional note for this order made by "customer@example.com"
diff --git a/src/Sylius/Behat/Context/Setup/UserContext.php b/src/Sylius/Behat/Context/Setup/UserContext.php
@@ -132,6 +132,28 @@ public function accountWasDeleted($email)
         $this->userRepository->remove($user);
     }
 
+    /**
+     * @Given his account was deleted
+     */
+    public function hisAccountWasDeleted()
+    {
+        $user = $this->sharedStorage->get('user');
+
+        $this->userRepository->remove($user);
+    }
+
+    /**
+     * @Given the store has customer service account identified by :email
+     */
+    public function theStoreHasCustomerServiceAccountIdentifiedBy($email)
+    {
+        $administrator = $this->userFactory->createDefaultAdmin();
+        $administrator->setEmail($email);
+
+        $this->sharedStorage->set('customer_service', $administrator);
+        $this->userRepository->add($administrator);
+    }
+
     /**
      * @param string $firstName
      * @param string $lastName
@@ -160,14 +182,4 @@ private function createAddress(
 
         return $address;
     }
-
-    /**
-     * @Given his account was deleted
-     */
-    public function hisAccountWasDeleted()
-    {
-        $user = $this->sharedStorage->get('user');
-
-        $this->userRepository->remove($user);
-    }
 }
diff --git a/src/Sylius/Behat/Context/Transform/CustomerContext.php b/src/Sylius/Behat/Context/Transform/CustomerContext.php
@@ -43,6 +43,7 @@ public function __construct(CustomerRepositoryInterface $customerRepository, Fac
 
     /**
      * @Transform :customer
+     * @Transform /^made by "[^"]+"$/
      */
     public function getOrCreateCustomerByEmail($email)
     {

diff --git a/src/Sylius/Behat/Context/Transform/OrderContext.php b/src/Sylius/Behat/Context/Transform/OrderContext.php
@@ -12,24 +12,35 @@
 namespace Sylius\Behat\Context\Transform;
 
 use Behat\Behat\Context\Context;
+use Sylius\Component\Core\Model\OrderInterface;
 use Sylius\Component\Core\Repository\OrderRepositoryInterface;
+use Sylius\Component\User\Repository\CustomerRepositoryInterface;
 use Webmozart\Assert\Assert;
 
 /**
  * @author Grzegorz Sadowski <grzegorz.sadowski@lakion.com>
  */
 final class OrderContext implements Context
 {
+    /**
+     * @var CustomerRepositoryInterface
+     */
+    private $customerRepository;
+
     /**
      * @var OrderRepositoryInterface
      */
     private $orderRepository;
 
     /**
+     * @param CustomerRepositoryInterface $customerRepository
      * @param OrderRepositoryInterface $orderRepository
      */
-    public function __construct(OrderRepositoryInterface $orderRepository)
-    {
+    public function __construct(
+        CustomerRepositoryInterface $customerRepository,
+        OrderRepositoryInterface $orderRepository
+    ) {
+        $this->customerRepository = $customerRepository;
         $this->orderRepository = $orderRepository;
     }
 
@@ -46,6 +57,21 @@ public function getOrderByNumber($orderNumber)
         return $order;
     }
 
+    /**
+     * @Transform /^this order made by "([^"]+)"$/
+     */
+    public function getOrderByCustomer($email)
+    {
+        $customer = $this->customerRepository->findOneBy(['email' => $email]);
+        Assert::notNull($customer, sprintf('Cannot find customer with email %s.', $email));
+
+        $order = $this->orderRepository->findByCustomer($customer);
+        $order = end($order);
+        Assert::isInstanceOf($order, OrderInterface::class, sprintf('Cannot find order for %s.', $customer->getFullName()));
+
+        return $order;
+    }
+
     /**
      * @Transform :orderNumber
      * @Transform /^an order "([^"]+)"$/

diff --git a/src/Sylius/Behat/Context/Ui/Admin/ManagingOrdersContext.php b/src/Sylius/Behat/Context/Ui/Admin/ManagingOrdersContext.php
@@ -16,8 +16,10 @@
 use Sylius\Behat\Page\Admin\Crud\IndexPageInterface;
 use Sylius\Behat\Page\Admin\Order\ShowPageInterface;
 use Sylius\Behat\Service\NotificationCheckerInterface;
+use Sylius\Behat\Service\SecurityServiceInterface;
 use Sylius\Component\Core\Model\CustomerInterface;
 use Sylius\Component\Core\Model\OrderInterface;
+use Sylius\Component\Core\Model\UserInterface;
 use Sylius\Component\Core\Test\Services\SharedStorageInterface;
 use Webmozart\Assert\Assert;
 
@@ -47,22 +49,30 @@ final class ManagingOrdersContext implements Context
      */
     private $notificationChecker;
 
+    /**
+     * @var SecurityServiceInterface
+     */
+    private $securityService;
+
     /**
      * @param SharedStorageInterface $sharedStorage
      * @param IndexPageInterface $indexPage
      * @param ShowPageInterface $showPage
      * @param NotificationCheckerInterface $notificationChecker
+     * @param SecurityServiceInterface $securityService
      */
     public function __construct(
         SharedStorageInterface $sharedStorage,
         IndexPageInterface $indexPage,
         ShowPageInterface $showPage,
-        NotificationCheckerInterface $notificationChecker
+        NotificationCheckerInterface $notificationChecker,
+        SecurityServiceInterface $securityService
     ) {
         $this->sharedStorage = $sharedStorage;
         $this->indexPage = $indexPage;
         $this->showPage = $showPage;
         $this->notificationChecker = $notificationChecker;
+        $this->securityService = $securityService;
     }
 
     /**
@@ -539,4 +549,15 @@ public function itShouldHaveState($state)
     {
         $this->indexPage->isSingleResourceOnPage(['state' => $state]);
     }
+
+    /**
+     * @Then /^(the customer service) should know about (this additional note) for (this order made by "[^"]+")$/
+     */
+    public function theCustomerServiceShouldKnowAboutThisAdditionalNotes(UserInterface $user, $note, OrderInterface $order)
+    {
+        $this->securityService->performActionAs($user, function () use ($note, $order) {
+            $this->showPage->open(['id' => $order->getId()]);
+            Assert::true($this->showPage->hasNote($note), sprintf('I should see %s note, but I do not see', $note));
+        });
+    }
 }
diff --git a/src/Sylius/Behat/Context/Ui/CheckoutContext.php b/src/Sylius/Behat/Context/Ui/CheckoutContext.php
@@ -360,6 +360,15 @@ public function iChangeShippingMethod($shippingMethodName)
         $this->checkoutShippingStep->continueCheckout();
     }
 
+    /**
+     * @When /^I provide additional note like "([^"]+)"$/
+     */
+    public function iProvideAdditionalNotesLike($notes)
+    {
+        $this->sharedStorage->set('additional_note', $notes);
+        $this->summaryPage->addNotes($notes);
+    }
+
     /**
      * @Given /^I proceed logging as "([^"]*)" with "([^"]*)" password$/
      */

diff --git a/src/Sylius/Behat/Page/Admin/Order/ShowPage.php b/src/Sylius/Behat/Page/Admin/Order/ShowPage.php
@@ -17,6 +17,7 @@
 use Sylius\Behat\Service\Accessor\TableAccessorInterface;
 use Sylius\Component\Core\Model\OrderInterface;
 use Symfony\Component\Routing\RouterInterface;
+use Webmozart\Assert\Assert;
 
 /**
  * @author Łukasz Chruściel <lukasz.chrusciel@lakion.com>
@@ -337,6 +338,17 @@ public function getRouteName()
         return 'sylius_admin_order_show';
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function hasNote($note)
+    {
+        $orderNotesElement = $this->getElement('order_notes');
+        Assert::notNull($orderNotesElement, 'Cannot find order notes');
+
+        return $orderNotesElement->getText() === $note;
+    }
+
     /**
      * {@inheritdoc}
      */
@@ -358,6 +370,7 @@ protected function getDefinedElements()
             'taxes' => '#taxes',
             'total' => '#total',
             'order_state' => 'div.sub.header > span.ui.label',
+            'order_notes' => '#sylius-order-notes',
         ]);
     }
 

diff --git a/src/Sylius/Behat/Page/Admin/Order/ShowPageInterface.php b/src/Sylius/Behat/Page/Admin/Order/ShowPageInterface.php
@@ -210,4 +210,11 @@ public function getOrderState();
     public function cancelOrder();
 
     public function deleteOrder();
+
+    /**
+     * @param string $note
+     * 
+     * @return bool
+     */
+    public function hasNote($note);
 }
diff --git a/src/Sylius/Behat/Page/Shop/Checkout/FinalizeStep.php b/src/Sylius/Behat/Page/Shop/Checkout/FinalizeStep.php
@@ -11,6 +11,7 @@
 
 namespace Sylius\Behat\Page\Shop\Checkout;
 
+use Behat\Mink\Exception\ElementNotFoundException;
 use Sylius\Behat\Page\SymfonyPage;
 
 /**
@@ -23,7 +24,11 @@ class FinalizeStep extends SymfonyPage implements FinalizeStepInterface
      */
     public function confirmOrder()
     {
-        $this->getDocument()->clickLink('Place order');
+        try {
+            $this->getDocument()->clickLink('Place order');
+        } catch (ElementNotFoundException $exception) {
+            $this->getDocument()->pressButton('Place order');
+        }
     }
 
     /**

diff --git a/src/Sylius/Behat/Page/Shop/Checkout/SummaryPage.php b/src/Sylius/Behat/Page/Shop/Checkout/SummaryPage.php
@@ -90,6 +90,14 @@ public function hasBillingAddress(AddressInterface $address)
         return $this->isAddressValid($billingAddress, $address);
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function addNotes($notes)
+    {
+        $this->getElement('extra_notes')->setValue($notes);
+    }
+
     /**
      * {@inheritdoc}
      */
@@ -99,6 +107,7 @@ protected function getDefinedElements()
             'billing_address' => '#addresses div:contains("Billing address") address',
             'shipping_address' => '#addresses div:contains("Shipping address") address',
             'items_table' => '#items table',
+            'extra_notes' =>'#sylius_checkout_summary_note_notes',
         ]);
     }
 

diff --git a/src/Sylius/Behat/Page/Shop/Checkout/SummaryPageInterface.php b/src/Sylius/Behat/Page/Shop/Checkout/SummaryPageInterface.php
@@ -40,4 +40,9 @@ public function hasShippingAddress(AddressInterface $address);
      * @return bool
      */
     public function hasBillingAddress(AddressInterface $address);
+
+    /**
+     * @param string $notes
+     */
+    public function addNotes($notes);
 }
diff --git a/src/Sylius/Behat/Resources/config/services/contexts/transform.xml b/src/Sylius/Behat/Resources/config/services/contexts/transform.xml
@@ -89,6 +89,7 @@
         </service>
 
         <service id="sylius.behat.context.transform.order" class="%sylius.behat.context.transform.order.class%" scope="scenario">
+            <argument type="service" id="sylius.repository.customer" container="symfony" />
             <argument type="service" id="sylius.repository.order" container="symfony" />
             <tag name="sylius.behat.context" />
         </service>

diff --git a/src/Sylius/Behat/Resources/config/services/contexts/ui.xml b/src/Sylius/Behat/Resources/config/services/contexts/ui.xml
@@ -99,6 +99,7 @@
             <argument type="service" id="sylius.behat.page.admin.order.index" />
             <argument type="service" id="sylius.behat.page.admin.order.show" />
             <argument type="service" id="sylius.behat.notification_checker" />
+            <argument type="service" id="sylius.behat.security" />
             <tag name="sylius.behat.context" />
         </service>
 

diff --git a/src/Sylius/Behat/Resources/config/suites/ui/order/placing_orders.yml b/src/Sylius/Behat/Resources/config/suites/ui/order/placing_orders.yml
@@ -8,20 +8,22 @@ default:
                 - sylius.behat.context.hook.doctrine_orm
 
                 - sylius.behat.context.setup.channel
+                - sylius.behat.context.setup.customer
                 - sylius.behat.context.setup.payment
                 - sylius.behat.context.setup.product
                 - sylius.behat.context.setup.security
                 - sylius.behat.context.setup.shipping
                 - sylius.behat.context.setup.user
 
+                - sylius.behat.context.transform.addressing
                 - sylius.behat.context.transform.customer
                 - sylius.behat.context.transform.lexical
+                - sylius.behat.context.transform.order
                 - sylius.behat.context.transform.product
                 - sylius.behat.context.transform.shared_storage
 
                 - sylius.behat.context.ui.admin.managing_orders
                 - sylius.behat.context.ui.checkout
                 - sylius.behat.context.ui.shop.cart
-
             filters:
                 tags: "@placing_orders && @ui"
diff --git a/src/Sylius/Bundle/AdminBundle/Resources/views/Order/Show/_notes.html.twig b/src/Sylius/Bundle/AdminBundle/Resources/views/Order/Show/_notes.html.twig
@@ -0,0 +1,8 @@
+{% if order.notes is defined and order.notes is not null %}
+    <h4 class="ui top attached styled header">
+        {{ 'sylius.ui.notes'|trans }}
+    </h4>
+    <div class="ui attached segment" id="sylius-order-notes">
+        {{ order.notes }}
+    </div>
+{% endif %}