Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add grouping of locations. #63

Merged
merged 1 commit into from
Jul 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 46 additions & 5 deletions Classes/Domain/Repository/DateRepository.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
use DateTimeImmutable;
use DateTimeZone;
use TYPO3\CMS\Core\Context\Context;
use TYPO3\CMS\Core\Database\Connection;
use TYPO3\CMS\Core\Database\ConnectionPool;
use TYPO3\CMS\Core\Utility\GeneralUtility;
use TYPO3\CMS\Extbase\Persistence\Generic\Qom\ConstraintInterface;
Expand All @@ -22,11 +23,21 @@ class DateRepository extends Repository
*/
protected $context;

/**
* @var ConnectionPool
*/
protected $connectionPool;

public function injectContext(Context $context): void
{
$this->context = $context;
}

public function injectConnectionPool(ConnectionPool $connectionPool): void
{
$this->connectionPool = $connectionPool;
}

public function findByUids(string $uids): QueryResult
{
$uids = explode(',', $uids);
Expand Down Expand Up @@ -60,7 +71,7 @@ protected function createDemandQuery(DateDemand $demand): QueryInterface
}

if ($demand->getLocations() !== []) {
$constraints['locations'] = $query->in('event.location', $demand->getLocations());
$constraints['locations'] = $this->createLocationConstraint($query, $demand);
}

if ($demand->getOrganizers() !== []) {
Expand Down Expand Up @@ -146,8 +157,7 @@ private function getSearchwordConstraint(
$wordsToSearch[] = $demand->getSearchword();
$constraints = [];

$queryBuilder = $this->objectManager->get(ConnectionPool::class)
->getQueryBuilderForTable('tx_events_domain_model_date');
$queryBuilder = $this->connectionPool->getQueryBuilderForTable('tx_events_domain_model_date');

foreach ($wordsToSearch as $word) {
foreach ($fieldsToSearch as $field) {
Expand Down Expand Up @@ -250,10 +260,41 @@ private function createFeaturesConstraint(
return $query->logicalAnd($constraints);
}

private function createLocationConstraint(
QueryInterface $query,
DateDemand $demand
): ConstraintInterface {
$locations = $demand->getLocations();
$uidsToResolve = $locations;

$queryBuilder = $this->connectionPool->getQueryBuilderForTable('tx_events_domain_model_location');
$queryBuilder->select('children');
$queryBuilder->from('tx_events_domain_model_location');

// Loop as resolved uids might have further children which need to be resolved as well.
do {
$concreteQueryBuilder = clone $queryBuilder;
$concreteQueryBuilder->where($concreteQueryBuilder->expr()->in(
'uid',
$concreteQueryBuilder->createNamedParameter($uidsToResolve, Connection::PARAM_INT_ARRAY)
));

foreach ($concreteQueryBuilder->execute()->fetchFirstColumn() as $newUids) {
if (is_string($newUids) === false) {
$newUids = '';
}
$newUids = GeneralUtility::intExplode(',', $newUids, true);
$uidsToResolve = array_diff($newUids, $locations);
$locations = array_merge($locations, $uidsToResolve);
}
} while ($uidsToResolve !== []);

return $query->in('event.location', $locations);
}

public function findSearchWord(string $search): array
{
$connection = GeneralUtility::makeInstance(ConnectionPool::class)
->getConnectionForTable('tx_events_domain_model_date');
$connection = $this->connectionPool->getConnectionForTable('tx_events_domain_model_date');

$queryBuilder = $connection->createQueryBuilder();

Expand Down
21 changes: 20 additions & 1 deletion Configuration/TCA/tx_events_domain_model_location.php
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
'starttime' => 'starttime',
'endtime' => 'endtime',
],
'default_sortby' => 'name',
'searchFields' => 'name',
'iconfile' => 'EXT:events/Resources/Public/Icons/tx_events_domain_model_location.svg',
],
Expand All @@ -31,7 +32,6 @@
l10n_diffsource,
hidden,
name,
global_id,

street,
district,
Expand All @@ -41,6 +41,10 @@
phone,
latitude,
longitude,
--div--;' . $l10nPath . ':tabs.grouping,
children,
--div--;' . $l10nPath . ':tabs.tech,
global_id,
--div--;' . $l10nPath . ':tabs.access,
starttime,
endtime',
Expand Down Expand Up @@ -146,6 +150,21 @@
'eval' => 'trim',
],
],
'children' => [
'exclude' => true,
'label' => $l10nPath . ':tx_events_domain_model_location.children',
'config' => [
'type' => 'select',
'renderType' => 'selectMultipleSideBySide',
'foreign_table' => 'tx_events_domain_model_location',
'foreign_table_where' => 'AND {#tx_events_domain_model_location}.{#uid} != ###THIS_UID###',
'fieldControl' => [
'editPopup' => [
'disabled' => false,
],
],
],
],
'name' => [
'exclude' => true,
'label' => $l10nPath . ':tx_events_domain_model_location.name',
Expand Down
9 changes: 9 additions & 0 deletions Documentation/Changelog/3.9.0.rst
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,15 @@ Nothing
Features
--------

* Add grouping of locations.
It is now possible to group locations.
Each location can have arbitrary children.

That can be used for editorial structuring.
Filtering for a location will always find all dates where the location or one of the child locations is assigned.

One use case can be to group imported locations and provide a grouped location for filtering in frontend.

Backport of 4.0.0 features:

* Add meta tags.
Expand Down
18 changes: 15 additions & 3 deletions Resources/Private/Language/de.locallang_csh_location.xlf
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,18 @@
<source>Location</source>
<target>Veranstaltungsort</target>
</trans-unit>
<trans-unit id="tabs.grouping" xml:space="preserve">
<source>Grouping</source>
<target>Gruppierung</target>
</trans-unit>
<trans-unit id="tabs.tech" xml:space="preserve">
<source>Tech</source>
<target>Technik</target>
</trans-unit>
<trans-unit id="tabs.access" xml:space="preserve">
<source>Access</source>
<target>Zugriff</target>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.global_id" xml:space="preserve">
<source>Global UID</source>
<target>Globale UID</target>
Expand All @@ -15,9 +27,9 @@
<source>Auto generated from the values.</source>
<target>Wird automatisch aus den Werten generiert.</target>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.slug" xml:space="preserve">
<source>Slug</source>
<target>URL-Segment</target>
<trans-unit id="tx_events_domain_model_location.children" xml:space="preserve">
<source>Children</source>
<target>Kinder</target>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.name" xml:space="preserve">
<source>Name</source>
Expand Down
13 changes: 11 additions & 2 deletions Resources/Private/Language/locallang_csh_location.xlf
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,23 @@
<trans-unit id="tx_events_domain_model_location" xml:space="preserve">
<source>Location</source>
</trans-unit>
<trans-unit id="tabs.grouping" xml:space="preserve">
<source>Grouping</source>
</trans-unit>
<trans-unit id="tabs.tech" xml:space="preserve">
<source>Tech</source>
</trans-unit>
<trans-unit id="tabs.access" xml:space="preserve">
<source>Access</source>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.global_id" xml:space="preserve">
<source>Global UID</source>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.global_id.description" xml:space="preserve">
<source>Auto generated from the values.</source>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.slug" xml:space="preserve">
<source>Slug</source>
<trans-unit id="tx_events_domain_model_location.children" xml:space="preserve">
<source>Children</source>
</trans-unit>
<trans-unit id="tx_events_domain_model_location.name" xml:space="preserve">
<source>Name</source>
Expand Down
22 changes: 20 additions & 2 deletions Tests/Functional/Frontend/FilterTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ protected function setUp(): void
/**
* @test
*/
public function canFilterByASingleLocationViaFlexform(): void
public function canFilterDatesByASingleLocationViaFlexform(): void
{
$this->importPHPDataSet(__DIR__ . '/Fixtures/Database/FilterByASingleLocationViaFlexform.php');

Expand All @@ -42,7 +42,7 @@ public function canFilterByASingleLocationViaFlexform(): void
/**
* @test
*/
public function canFilterByTwoLocationsViaFlexform(): void
public function canFilterDatesByTwoLocationsViaFlexform(): void
{
$this->importPHPDataSet(__DIR__ . '/Fixtures/Database/FilterByTwoLocationsViaFlexform.php');

Expand All @@ -56,4 +56,22 @@ public function canFilterByTwoLocationsViaFlexform(): void
self::assertStringContainsString('Lotte in Weimar', $html);
self::assertStringContainsString('Was hat das Universum mit mir zu tun?', $html);
}

/**
* @test
*/
public function canFilterDatesByParentLocationViaFlexform(): void
{
$this->importPHPDataSet(__DIR__ . '/Fixtures/Database/FilterDatesByParentLocationViaFlexform.php');

$request = new InternalRequest();
$request = $request->withPageId(1);
$response = $this->executeFrontendRequest($request);

self::assertSame(200, $response->getStatusCode());
$html = (string)$response->getBody();

self::assertStringContainsString('Lotte in Weimar', $html);
self::assertStringContainsString('Was hat das Universum mit mir zu tun?', $html);
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,116 @@
<?php

return [
'tt_content' => [
[
'pid' => '1',
'uid' => '1',
'CType' => 'list',
'list_type' => 'events_datelist',
'header' => 'Kino Events',
'pi_flexform' => '<?xml version="1.0" encoding="utf-8" standalone="yes" ?>
<T3FlexForms>
<data>
<sheet index="sDEF">
<language index="lDEF">
<field index="settings.locations">
<value index="vDEF">1</value>
</field>
</language>
</sheet>
</data>
</T3FlexForms>
',
],
],
'tx_events_domain_model_location' => [
[
'uid' => '1',
'pid' => '2',
'name' => 'Parent',
'street' => '',
'city' => '',
'zip' => '',
'country' => '',
'longitude' => '',
'latitude' => '',
'children' => '2,3',
],
[
'uid' => '2',
'pid' => '2',
'name' => 'Child',
'street' => 'Theaterplatz 4',
'city' => 'Weimar',
'zip' => '99423',
'country' => 'Deutschland',
'longitude' => '11.3262489',
'latitude' => '50.9800023',
'district' => 'Zentrum',
// Validate we don't end in endless recursion
'children' => '1',
],
[
'uid' => '3',
'pid' => '2',
'name' => 'Child 2',
'street' => 'Cranach-Haus Markt 11/12',
'city' => 'Weimar',
'zip' => '99423',
'country' => 'Deutschland',
'longitude' => '11.330248',
'latitude' => '50.979349',
'children' => '',
],
],
'tx_events_domain_model_event' => [
[
'uid' => '1',
'pid' => '2',
'title' => 'Was hat das Universum mit mir zu tun?',
'global_id' => 'e_100478529',
'teaser' => '„WAS HAT DAS UNIVERSUM MIT MIR ZU TUN?“
Ein Abend mit Prof. Dr. Harald Lesch',
'details' => '„WAS HAT DAS UNIVERSUM MIT MIR ZU TUN?“
Ein Abend mit Prof. Dr. Harald Lesch
Auf den Spuren von Goethes Naturphilosophie ist der Astrophysiker und Wissenschaftsjournalist Prof. Dr. Harald Lesch in Weimar schon mehrmals präsent gewesen. Jetzt hält er einen Vortrag zu keiner geringeren Frage als „Was hat das Universum mit mir zu tun?“ Ob Goethe darauf eine pointierte Antwort eingefallen wäre? Sein Faust wollte die Spur seiner Erdentage nicht in Äonen untergehen sehen. Harald Lesch behauptet: Wir sind und bleiben stets Teil der Äonen - denn „wir sind alle Sternenstaub. Vor einer halben Ewigkeit ist ein Stern explodiert und hat alle Stoffe aus denen wir bestehen hervorgebracht. Und wenn das bei uns geklappt hat, könnte es auch noch woanders passiert sein.“ Erleben Sie einen faszinierenden Mix aus Rednerkunst und virtuoser musikalischer Begleitung. Neben Prof. Dr. Harald Lesch begibt sich der Musiker Hans Raths (Bayon) mit auf die Reise ins theatralische und philosophische Universum. Eine Veranstaltung nicht nur für Science-Fiction-Freaks, sondern für alle Kosmopoliten!',
'price_info' => 'Preis inklusive Platzierung mit Namensschild und einem Pausengetränk Ihrer Wahl',
'location' => '3',
],
[
'uid' => '2',
'pid' => '2',
'title' => 'Lotte in Weimar',
'global_id' => 'e_100453137',
'teaser' => 'Ein „Goethe-Götter-Lustspiel“ nach dem gleichnamigen Roman von Thomas Mann',
'details' => 'LOTTE IN WEIMAR
Ein „Goethe-Götter-Lustspiel“ nach dem gleichnamigen Roman von Thomas Mann
„Welch buchenswertes Ereignis!“, ruft der Kellner Mager aus, als er erfährt, wer da in seinem Gasthaus „Zum Elephanten“ abgestiegen ist: Die berühmte Heldin aus Goethes „Die Leiden des jungen Werthers“, Charlotte Kestner, geborene Buff aus Wetzlar, – das „Urbild“ der Lotte sozusagen! Eine heiter-ironische Abrechnung mit dem Starkult anno 1816 fast am Originalschauplatz. Mit Regine Heintze, Heike Meyer und Detlef Heintze. Inszenierung: Michael Kliefert/ Detlef Heintze.',
'price_info' => 'Preise inklusive Platzierung mit Namensschild und einem Pausengetränk Ihrer Wahl (ermäßigt alkoholfrei)',
'location' => '2',
],
],
'tx_events_domain_model_date' => [
[
'uid' => '1',
'pid' => '2',
'event' => '1',
'start' => '1661626800',
'end' => '1661632200',
],
[
'uid' => '2',
'pid' => '2',
'event' => '1',
'start' => '1660158000',
'end' => '1660163400',
],
[
'uid' => '3',
'pid' => '2',
'event' => '2',
'start' => '1661194800',
'end' => '1661200200',
],
],
];
1 change: 1 addition & 0 deletions ext_tables.sql
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ CREATE TABLE tx_events_domain_model_location (
phone varchar(255) DEFAULT '' NOT NULL,
latitude varchar(255) DEFAULT '' NOT NULL,
longitude varchar(255) DEFAULT '' NOT NULL,
children text,

KEY global_id (global_id)
);
Loading