Skip to content

Commit 78e1b32

Browse files
wachterjohanneswachterjohannes
committed
11 - Add a relation between events and locations
1 parent d116cb7 commit 78e1b32

File tree

9 files changed

+96
-19
lines changed

9 files changed

+96
-19
lines changed

‎config/forms/event_details.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
</meta>
1313
</property>
1414

15-
<property name="location" type="text_line" colspan="3">
15+
<property name="locationId" type="single_location_selection" mandatory="true" colspan="3">
1616
<meta>
1717
<title>app.location</title>
1818
</meta>

‎config/packages/sulu_admin.yaml

+14
Original file line numberDiff line numberDiff line change
@@ -38,3 +38,17 @@ sulu_admin:
3838
icon: fa-calendar
3939
label: 'app.events'
4040
overlay_title: 'app.events'
41+
42+
single_selection:
43+
single_location_selection:
44+
default_type: list_overlay
45+
resource_key: locations
46+
types:
47+
list_overlay:
48+
adapter: table
49+
list_key: locations
50+
display_properties:
51+
- name
52+
icon: fa-home
53+
empty_text: 'app.location.no_selections'
54+
overlay_title: 'app.locations'

‎src/Controller/Admin/EventController.php

+10-2
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
use App\Admin\DoctrineListRepresentationFactory;
88
use App\Entity\Event;
99
use App\Repository\EventRepository;
10+
use App\Repository\LocationRepository;
1011
use FOS\RestBundle\Controller\Annotations as Rest;
1112
use FOS\RestBundle\Routing\ClassResourceInterface;
1213
use Sulu\Component\Rest\RestController;
@@ -21,16 +22,23 @@ class EventController extends RestController implements ClassResourceInterface
2122
*/
2223
private $repository;
2324

25+
/**
26+
* @var LocationRepository
27+
*/
28+
private $locationRepository;
29+
2430
/**
2531
* @var DoctrineListRepresentationFactory
2632
*/
2733
private $doctrineListRepresentationFactory;
2834

2935
public function __construct(
3036
EventRepository $repository,
37+
LocationRepository $locationRepository,
3138
DoctrineListRepresentationFactory $doctrineListRepresentationFactory
3239
) {
3340
$this->repository = $repository;
41+
$this->locationRepository = $locationRepository;
3442
$this->doctrineListRepresentationFactory = $doctrineListRepresentationFactory;
3543
}
3644

@@ -135,8 +143,8 @@ protected function mapDataToEntity(array $data, Event $entity): void
135143
$entity->setEndDate(new \DateTimeImmutable($endDate));
136144
}
137145

138-
if ($location = $data['location'] ?? null) {
139-
$entity->setLocation($location);
146+
if ($locationId = $data['locationId'] ?? null) {
147+
$entity->setLocation($this->locationRepository->findById((int) $locationId));
140148
}
141149
}
142150

‎src/DataFixtures/ORM/AppFixtures.php

+18-4
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
namespace App\DataFixtures\ORM;
66

77
use App\Entity\Event;
8+
use App\Entity\Location;
89
use Doctrine\Bundle\FixturesBundle\Fixture;
910
use Doctrine\Common\Persistence\ObjectManager;
1011

@@ -14,7 +15,8 @@ class AppFixtures extends Fixture
1415

1516
public function load(ObjectManager $manager)
1617
{
17-
$repository = $manager->getRepository(Event::class);
18+
$eventRepository = $manager->getRepository(Event::class);
19+
$locationRepository = $manager->getRepository(Location::class);
1820

1921
$data = [
2022
[
@@ -83,17 +85,29 @@ public function load(ObjectManager $manager)
8385
];
8486

8587
foreach ($data as $item) {
86-
$event = $repository->create(self::LOCALE);
88+
$location = null;
89+
if ($item['location']) {
90+
$location = $locationRepository->create();
91+
$location->setName($item['location']);
92+
$location->setStreet('');
93+
$location->setNumber('');
94+
$location->setCity('');
95+
$location->setCountryCode('');
96+
$location->setPostalCode('');
97+
$locationRepository->save($location);
98+
}
99+
100+
$event = $eventRepository->create(self::LOCALE);
87101

88102
$event->setTitle($item['title']);
89-
$event->setLocation($item['location']);
103+
$event->setLocation($location);
90104
$event->setTeaser($item['teaser']);
91105
$event->setDescription('<p>' . $item['description'] . '</p>');
92106
$event->setStartDate(new \DateTimeImmutable($item['startDate']));
93107
$event->setEndDate(new \DateTimeImmutable($item['endDate']));
94108
$event->setEnabled($item['enabled']);
95109

96-
$repository->save($event);
110+
$eventRepository->save($event);
97111
}
98112

99113
$manager->flush();

‎src/Entity/Event.php

+17-4
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,10 @@ class Event
4747
private $endDate;
4848

4949
/**
50-
* @var string|null
50+
* @var Location|null
5151
*
52-
* @ORM\Column(type="string", nullable=true)
52+
* @ORM\ManyToOne(targetEntity="App\Entity\Location")
53+
* @ORM\JoinColumn(onDelete="SET NULL")
5354
*/
5455
private $location;
5556

@@ -114,18 +115,30 @@ public function setEndDate(?\DateTimeImmutable $endDate): self
114115
return $this;
115116
}
116117

117-
public function getLocation(): ?string
118+
public function getLocation(): ?Location
118119
{
119120
return $this->location;
120121
}
121122

122-
public function setLocation(?string $location): self
123+
public function setLocation(?Location $location): self
123124
{
124125
$this->location = $location;
125126

126127
return $this;
127128
}
128129

130+
/**
131+
* @Serializer\VirtualProperty
132+
*/
133+
public function getLocationId(): ?int
134+
{
135+
if (!$this->location) {
136+
return null;
137+
}
138+
139+
return $this->location->getId();
140+
}
141+
129142
/**
130143
* @Serializer\VirtualProperty(name="title")
131144
*/

‎tests/Functional/Controller/Admin/EventControllerTest.php

+11-6
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,14 @@
55
namespace App\Tests\Functional\Controller\Admin;
66

77
use App\Tests\Functional\Traits\EventTrait;
8+
use App\Tests\Functional\Traits\LocationTrait;
89
use Sulu\Bundle\TestBundle\Testing\SuluTestCase;
910
use Symfony\Component\HttpFoundation\Response;
1011

1112
class EventControllerTest extends SuluTestCase
1213
{
1314
use EventTrait;
15+
use LocationTrait;
1416

1517
public function setUp(): void
1618
{
@@ -65,6 +67,8 @@ public function testPost(): void
6567
{
6668
$client = $this->createAuthenticatedClient();
6769

70+
$location = $this->createLocation('Sulu HQ');
71+
6872
$client->request(
6973
'POST',
7074
'/admin/api/events?locale=de',
@@ -74,7 +78,7 @@ public function testPost(): void
7478
'startDate' => '2019-01-01 12:00',
7579
'endDate' => '2019-01-02 12:00',
7680
'description' => 'Sulu is really awesome',
77-
'location' => 'Dornbirn',
81+
'locationId' => $location->getId(),
7882
]
7983
);
8084

@@ -91,7 +95,7 @@ public function testPost(): void
9195
$this->assertSame('2019-01-01T12:00:00', $result['startDate']);
9296
$this->assertSame('2019-01-02T12:00:00', $result['endDate']);
9397
$this->assertSame('Sulu is really awesome', $result['description']);
94-
$this->assertSame('Dornbirn', $result['location']);
98+
$this->assertSame($location->getId(), $result['locationId']);
9599

96100
$result = $this->findEventById($result['id'], 'de');
97101

@@ -104,7 +108,7 @@ public function testPost(): void
104108
$this->assertNotNull($result->getEndDate());
105109
$this->assertSame('2019-01-02T12:00:00', $result->getEndDate()->format('Y-m-d\TH:i:s'));
106110
$this->assertSame('Sulu is really awesome', $result->getDescription());
107-
$this->assertSame('Dornbirn', $result->getLocation());
111+
$this->assertSame($location->getId(), $result->getLocationId());
108112
}
109113

110114
public function testPostNullValues(): void
@@ -151,6 +155,7 @@ public function testPut(): void
151155
$client = $this->createAuthenticatedClient();
152156

153157
$event = $this->createEvent('Symfony', 'de');
158+
$location = $this->createLocation('Sulu HQ');
154159

155160
$client->request(
156161
'PUT',
@@ -161,7 +166,7 @@ public function testPut(): void
161166
'startDate' => '2019-01-01 12:00',
162167
'endDate' => '2019-01-02 12:00',
163168
'description' => 'Symfony Live is really awesome',
164-
'location' => 'Dornbirn',
169+
'locationId' => $location->getId(),
165170
]
166171
);
167172

@@ -178,7 +183,7 @@ public function testPut(): void
178183
$this->assertSame('2019-01-01T12:00:00', $result['startDate']);
179184
$this->assertSame('2019-01-02T12:00:00', $result['endDate']);
180185
$this->assertSame('Symfony Live is really awesome', $result['description']);
181-
$this->assertSame('Dornbirn', $result['location']);
186+
$this->assertSame($location->getId(), $result['locationId']);
182187

183188
$result = $this->findEventById($result['id'], 'de');
184189

@@ -191,7 +196,7 @@ public function testPut(): void
191196
$this->assertNotNull($result->getEndDate());
192197
$this->assertSame('2019-01-02T12:00:00', $result->getEndDate()->format('Y-m-d\TH:i:s'));
193198
$this->assertSame('Symfony Live is really awesome', $result->getDescription());
194-
$this->assertSame('Dornbirn', $result->getLocation());
199+
$this->assertSame($location->getId(), $result->getLocationId());
195200
}
196201

197202
public function testPutNullValues(): void

‎tests/Unit/Entity/EventTest.php

+21
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,26 @@
66

77
use App\Entity\Event;
88
use App\Entity\EventTranslation;
9+
use App\Entity\Location;
910
use PHPUnit\Framework\TestCase;
11+
use Prophecy\Prophecy\ObjectProphecy;
1012

1113
class EventTest extends TestCase
1214
{
15+
/**
16+
* @var Location|ObjectProphecy
17+
*/
18+
private $location;
19+
1320
/**
1421
* @var Event
1522
*/
1623
private $event;
1724

1825
public function setUp(): void
1926
{
27+
$this->location = $this->prophesize(Location::class);
28+
2029
$this->event = new Event();
2130
$this->event->setLocale('de');
2231
}
@@ -87,4 +96,16 @@ public function testDescription(): void
8796
$this->assertSame('de', $this->event->getTranslations()['de']->getLocale());
8897
$this->assertSame('Sulu is awesome', $this->event->getTranslations()['de']->getDescription());
8998
}
99+
100+
public function testLocation(): void
101+
{
102+
$this->location->getId()->willReturn(42);
103+
104+
$this->assertNull($this->event->getLocation());
105+
$this->assertNull($this->event->getLocationId());
106+
$this->assertSame($this->event, $this->event->setLocation($this->location->reveal()));
107+
$this->assertNotNull($this->event->getLocation());
108+
$this->assertSame($this->location->reveal(), $this->event->getLocation());
109+
$this->assertSame(42, $this->event->getLocationId());
110+
}
90111
}

‎translations/admin.de.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,6 @@
77
"app.enable_event": "Veranstaltungen aktivieren",
88
"app.enabled": "Aktiviert",
99
"app.location": "Standort",
10-
"app.locations": "Standorte"
10+
"app.locations": "Standorte",
11+
"app.location.no_selections": "Kein Standord ausgewählt"
1112
}

‎translations/admin.en.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,6 @@
77
"app.enable_event": "Enable event",
88
"app.enabled": "Enabled",
99
"app.location": "Location",
10-
"app.locations": "Locations"
10+
"app.locations": "Locations",
11+
"app.location.no_selections": "No location selected"
1112
}

0 commit comments

Comments
 (0)