From 0b8ebb15d9c7f97c3d192fc4f88fd910b6c968c3 Mon Sep 17 00:00:00 2001
From: Johannes Wachter <johannes@sulu.io>
Date: Thu, 3 Oct 2019 08:20:54 +0200
Subject: [PATCH] 09 - Add a list representation for locations in the admin
 interface

---
 config/lists/locations.xml                    | 41 +++++++++++++++
 config/packages/sulu_admin.yaml               |  4 ++
 config/routes_admin.yaml                      |  8 +++
 src/Admin/LocationAdmin.php                   | 51 +++++++++++++++++++
 src/Controller/Admin/LocationController.php   | 40 +++++++++++++++
 src/Entity/Location.php                       |  2 +
 .../Admin/LocationControllerTest.php          | 49 ++++++++++++++++++
 tests/Functional/Traits/LocationTrait.php     | 35 +++++++++++++
 translations/admin.de.json                    |  3 +-
 translations/admin.en.json                    |  3 +-
 10 files changed, 234 insertions(+), 2 deletions(-)
 create mode 100644 config/lists/locations.xml
 create mode 100644 src/Admin/LocationAdmin.php
 create mode 100644 src/Controller/Admin/LocationController.php
 create mode 100644 tests/Functional/Controller/Admin/LocationControllerTest.php
 create mode 100644 tests/Functional/Traits/LocationTrait.php

diff --git a/config/lists/locations.xml b/config/lists/locations.xml
new file mode 100644
index 0000000..b72bc73
--- /dev/null
+++ b/config/lists/locations.xml
@@ -0,0 +1,41 @@
+<?xml version="1.0" ?>
+<list xmlns="http://schemas.sulu.io/list-builder/list">
+    <key>locations</key>
+
+    <properties>
+        <property name="id" visibility="no" translation="sulu_admin.id">
+            <field-name>id</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+
+        <property name="name" visibility="always" searchability="yes" translation="sulu_admin.name">
+            <field-name>name</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+
+        <property name="street" visibility="yes" searchability="yes" translation="sulu_contact.street">
+            <field-name>street</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+
+        <property name="number" visibility="yes" searchability="yes" translation="sulu_contact.number">
+            <field-name>number</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+
+        <property name="postalCode" visibility="yes" searchability="yes" translation="sulu_contact.zip">
+            <field-name>postalCode</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+
+        <property name="city" visibility="yes" searchability="yes" translation="sulu_contact.city">
+            <field-name>city</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+
+        <property name="countryCode" visibility="yes" searchability="yes" translation="sulu_contact.country">
+            <field-name>countryCode</field-name>
+            <entity-name>App\Entity\Location</entity-name>
+        </property>
+    </properties>
+</list>
diff --git a/config/packages/sulu_admin.yaml b/config/packages/sulu_admin.yaml
index 30c2925..f8caeb1 100644
--- a/config/packages/sulu_admin.yaml
+++ b/config/packages/sulu_admin.yaml
@@ -18,6 +18,10 @@ sulu_admin:
             routes:
                 list: app.get_event_registrations
 
+        locations:
+            routes:
+                list: app.get_locations
+
     # Registering Selection Field Types in this section
     field_type_options:
         selection:
diff --git a/config/routes_admin.yaml b/config/routes_admin.yaml
index 5923813..f4c319f 100644
--- a/config/routes_admin.yaml
+++ b/config/routes_admin.yaml
@@ -15,3 +15,11 @@ app_event_registrations:
     name_prefix: app.
     options:
         expose: true
+
+app_locations:
+    type: rest
+    resource: App\Controller\Admin\LocationController
+    prefix: /admin/api
+    name_prefix: app.
+    options:
+        expose: true
diff --git a/src/Admin/LocationAdmin.php b/src/Admin/LocationAdmin.php
new file mode 100644
index 0000000..f713a57
--- /dev/null
+++ b/src/Admin/LocationAdmin.php
@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Admin;
+
+use App\Entity\Location;
+use Sulu\Bundle\AdminBundle\Admin\Admin;
+use Sulu\Bundle\AdminBundle\Admin\Navigation\NavigationItem;
+use Sulu\Bundle\AdminBundle\Admin\Navigation\NavigationItemCollection;
+use Sulu\Bundle\AdminBundle\Admin\View\ViewBuilderFactoryInterface;
+use Sulu\Bundle\AdminBundle\Admin\View\ViewCollection;
+
+class LocationAdmin extends Admin
+{
+    const LOCATION_LIST_KEY = 'locations';
+
+    const LOCATION_LIST_VIEW = 'app.locations_list';
+
+    /**
+     * @var ViewBuilderFactoryInterface
+     */
+    private $viewBuilderFactory;
+
+    public function __construct(ViewBuilderFactoryInterface $viewBuilderFactory)
+    {
+        $this->viewBuilderFactory = $viewBuilderFactory;
+    }
+
+    public function configureNavigationItems(NavigationItemCollection $navigationItemCollection): void
+    {
+        $module = $navigationItemCollection->get('app.events');
+
+        $locations = new NavigationItem('app.locations');
+        $locations->setPosition(10);
+        $locations->setView(static::LOCATION_LIST_VIEW);
+
+        $module->addChild($locations);
+    }
+
+    public function configureViews(ViewCollection $viewCollection): void
+    {
+        $listView = $this->viewBuilderFactory->createListViewBuilder(self::LOCATION_LIST_VIEW, '/locations')
+            ->setResourceKey(Location::RESOURCE_KEY)
+            ->setListKey(self::LOCATION_LIST_KEY)
+            ->setTitle('app.locations')
+            ->addListAdapters(['table'])
+            ->addToolbarActions([]);
+        $viewCollection->add($listView);
+    }
+}
diff --git a/src/Controller/Admin/LocationController.php b/src/Controller/Admin/LocationController.php
new file mode 100644
index 0000000..e363610
--- /dev/null
+++ b/src/Controller/Admin/LocationController.php
@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Admin;
+
+use App\Common\DoctrineListRepresentationFactory;
+use App\Entity\Location;
+use FOS\RestBundle\Routing\ClassResourceInterface;
+use FOS\RestBundle\View\ViewHandlerInterface;
+use Sulu\Component\Rest\AbstractRestController;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
+
+class LocationController extends AbstractRestController implements ClassResourceInterface
+{
+    /**
+     * @var DoctrineListRepresentationFactory
+     */
+    private $doctrineListRepresentationFactory;
+
+    public function __construct(
+        DoctrineListRepresentationFactory $doctrineListRepresentationFactory,
+        ViewHandlerInterface $viewHandler,
+        ?TokenStorageInterface $tokenStorage = null
+    ) {
+        $this->doctrineListRepresentationFactory = $doctrineListRepresentationFactory;
+
+        parent::__construct($viewHandler, $tokenStorage);
+    }
+
+    public function cgetAction(): Response
+    {
+        $listRepresentation = $this->doctrineListRepresentationFactory->createDoctrineListRepresentation(
+            Location::RESOURCE_KEY
+        );
+
+        return $this->handleView($this->view($listRepresentation));
+    }
+}
diff --git a/src/Entity/Location.php b/src/Entity/Location.php
index 892f081..5c2f6d0 100644
--- a/src/Entity/Location.php
+++ b/src/Entity/Location.php
@@ -11,6 +11,8 @@
  */
 class Location
 {
+    const RESOURCE_KEY = 'locations';
+
     /**
      * @var int|null
      *
diff --git a/tests/Functional/Controller/Admin/LocationControllerTest.php b/tests/Functional/Controller/Admin/LocationControllerTest.php
new file mode 100644
index 0000000..24d494d
--- /dev/null
+++ b/tests/Functional/Controller/Admin/LocationControllerTest.php
@@ -0,0 +1,49 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Tests\Functional\Controller\Admin;
+
+use App\Tests\Functional\Traits\LocationTrait;
+use Sulu\Bundle\TestBundle\Testing\SuluTestCase;
+use Symfony\Bundle\FrameworkBundle\KernelBrowser;
+use Symfony\Component\HttpFoundation\Response;
+
+class LocationControllerTest extends SuluTestCase
+{
+    use LocationTrait;
+
+    /**
+     * @var KernelBrowser
+     */
+    private $client;
+
+    public function setUp(): void
+    {
+        $this->client = $this->createAuthenticatedClient();
+        $this->purgeDatabase();
+    }
+
+    public function testCGet(): void
+    {
+        $location1 = $this->createLocation('Sulu');
+        $location2 = $this->createLocation('Symfony');
+
+        $this->client->request('GET', '/admin/api/locations');
+
+        $response = $this->client->getResponse();
+        $this->assertInstanceOf(Response::class, $response);
+        $result = json_decode($response->getContent() ?: '', true);
+        $this->assertHttpStatusCode(200, $response);
+
+        $this->assertSame(2, $result['total']);
+        $this->assertCount(2, $result['_embedded']['locations']);
+        $items = $result['_embedded']['locations'];
+
+        $this->assertSame($location1->getId(), $items[0]['id']);
+        $this->assertSame($location2->getId(), $items[1]['id']);
+
+        $this->assertSame($location1->getName(), $items[0]['name']);
+        $this->assertSame($location2->getName(), $items[1]['name']);
+    }
+}
diff --git a/tests/Functional/Traits/LocationTrait.php b/tests/Functional/Traits/LocationTrait.php
new file mode 100644
index 0000000..bc8304c
--- /dev/null
+++ b/tests/Functional/Traits/LocationTrait.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Tests\Functional\Traits;
+
+use App\Entity\Location;
+use App\Repository\LocationRepository;
+use Doctrine\ORM\EntityManagerInterface;
+
+trait LocationTrait
+{
+    public function createLocation(string $name): Location
+    {
+        $location = $this->getLocationRepository()->create();
+        $location->setName($name);
+        $location->setStreet('');
+        $location->setNumber('');
+        $location->setPostalCode('');
+        $location->setCity('');
+        $location->setCountryCode('');
+
+        $this->getEntityManager()->persist($location);
+        $this->getEntityManager()->flush();
+
+        return $location;
+    }
+
+    protected function getLocationRepository(): LocationRepository
+    {
+        return $this->getEntityManager()->getRepository(Location::class);
+    }
+
+    abstract protected function getEntityManager(): EntityManagerInterface;
+}
diff --git a/translations/admin.de.json b/translations/admin.de.json
index 54fd06f..ac72ae7 100644
--- a/translations/admin.de.json
+++ b/translations/admin.de.json
@@ -7,5 +7,6 @@
     "app.end_date": "Ende",
     "app.enable_event": "Veranstaltungen aktivieren",
     "app.enabled": "Aktiviert",
-    "app.location": "Standort"
+    "app.location": "Standort",
+    "app.locations": "Standorte"
 }
diff --git a/translations/admin.en.json b/translations/admin.en.json
index 293fa2d..b073ae1 100644
--- a/translations/admin.en.json
+++ b/translations/admin.en.json
@@ -7,5 +7,6 @@
     "app.end_date": "Ende",
     "app.enable_event": "Enable event",
     "app.enabled": "Enabled",
-    "app.location": "Location"
+    "app.location": "Location",
+    "app.locations": "Locations"
 }