From 9059f4c29f3928a916275364f47e883e943a10ec Mon Sep 17 00:00:00 2001 From: Daniel Siepmann Date: Wed, 24 Jul 2024 10:38:01 +0200 Subject: [PATCH] Add grouping of locations. (#63) 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. Relates: #11233 --- Classes/Domain/Repository/DateRepository.php | 51 +++++++- .../TCA/tx_events_domain_model_location.php | 21 +++- Documentation/Changelog/3.9.0.rst | 9 ++ .../Language/de.locallang_csh_location.xlf | 18 ++- .../Language/locallang_csh_location.xlf | 13 +- Tests/Functional/Frontend/FilterTest.php | 22 +++- ...FilterDatesByParentLocationViaFlexform.php | 116 ++++++++++++++++++ ext_tables.sql | 1 + 8 files changed, 238 insertions(+), 13 deletions(-) create mode 100644 Tests/Functional/Frontend/Fixtures/Database/FilterDatesByParentLocationViaFlexform.php diff --git a/Classes/Domain/Repository/DateRepository.php b/Classes/Domain/Repository/DateRepository.php index b1224be..3ae906b 100644 --- a/Classes/Domain/Repository/DateRepository.php +++ b/Classes/Domain/Repository/DateRepository.php @@ -5,6 +5,7 @@ namespace Wrm\Events\Domain\Repository; 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; @@ -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); @@ -60,7 +71,7 @@ class DateRepository extends Repository } if ($demand->getLocations() !== []) { - $constraints['locations'] = $query->in('event.location', $demand->getLocations()); + $constraints['locations'] = $this->createLocationConstraint($query, $demand); } if ($demand->getOrganizers() !== []) { @@ -146,8 +157,7 @@ class DateRepository extends Repository $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) { @@ -250,10 +260,41 @@ class DateRepository extends Repository 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(); diff --git a/Configuration/TCA/tx_events_domain_model_location.php b/Configuration/TCA/tx_events_domain_model_location.php index f020bfb..1f572c9 100644 --- a/Configuration/TCA/tx_events_domain_model_location.php +++ b/Configuration/TCA/tx_events_domain_model_location.php @@ -20,6 +20,7 @@ return [ 'starttime' => 'starttime', 'endtime' => 'endtime', ], + 'default_sortby' => 'name', 'searchFields' => 'name', 'iconfile' => 'EXT:events/Resources/Public/Icons/tx_events_domain_model_location.svg', ], @@ -31,7 +32,6 @@ return [ l10n_diffsource, hidden, name, - global_id, street, district, @@ -41,6 +41,10 @@ return [ phone, latitude, longitude, + --div--;' . $l10nPath . ':tabs.grouping, + children, + --div--;' . $l10nPath . ':tabs.tech, + global_id, --div--;' . $l10nPath . ':tabs.access, starttime, endtime', @@ -146,6 +150,21 @@ return [ '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', diff --git a/Documentation/Changelog/3.9.0.rst b/Documentation/Changelog/3.9.0.rst index 9e561f1..c823eb4 100644 --- a/Documentation/Changelog/3.9.0.rst +++ b/Documentation/Changelog/3.9.0.rst @@ -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. diff --git a/Resources/Private/Language/de.locallang_csh_location.xlf b/Resources/Private/Language/de.locallang_csh_location.xlf index 1383846..74ff254 100644 --- a/Resources/Private/Language/de.locallang_csh_location.xlf +++ b/Resources/Private/Language/de.locallang_csh_location.xlf @@ -7,6 +7,18 @@ Location Veranstaltungsort + + Grouping + Gruppierung + + + Tech + Technik + + + Access + Zugriff + Global UID Globale UID @@ -15,9 +27,9 @@ Auto generated from the values. Wird automatisch aus den Werten generiert. - - Slug - URL-Segment + + Children + Kinder Name diff --git a/Resources/Private/Language/locallang_csh_location.xlf b/Resources/Private/Language/locallang_csh_location.xlf index fed190b..8abe623 100644 --- a/Resources/Private/Language/locallang_csh_location.xlf +++ b/Resources/Private/Language/locallang_csh_location.xlf @@ -6,14 +6,23 @@ Location + + Grouping + + + Tech + + + Access + Global UID Auto generated from the values. - - Slug + + Children Name diff --git a/Tests/Functional/Frontend/FilterTest.php b/Tests/Functional/Frontend/FilterTest.php index d919d58..d7e3b68 100644 --- a/Tests/Functional/Frontend/FilterTest.php +++ b/Tests/Functional/Frontend/FilterTest.php @@ -24,7 +24,7 @@ class FilterTest extends AbstractFunctionalTestCase /** * @test */ - public function canFilterByASingleLocationViaFlexform(): void + public function canFilterDatesByASingleLocationViaFlexform(): void { $this->importPHPDataSet(__DIR__ . '/Fixtures/Database/FilterByASingleLocationViaFlexform.php'); @@ -42,7 +42,7 @@ class FilterTest extends AbstractFunctionalTestCase /** * @test */ - public function canFilterByTwoLocationsViaFlexform(): void + public function canFilterDatesByTwoLocationsViaFlexform(): void { $this->importPHPDataSet(__DIR__ . '/Fixtures/Database/FilterByTwoLocationsViaFlexform.php'); @@ -56,4 +56,22 @@ class FilterTest extends AbstractFunctionalTestCase 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); + } } diff --git a/Tests/Functional/Frontend/Fixtures/Database/FilterDatesByParentLocationViaFlexform.php b/Tests/Functional/Frontend/Fixtures/Database/FilterDatesByParentLocationViaFlexform.php new file mode 100644 index 0000000..036f258 --- /dev/null +++ b/Tests/Functional/Frontend/Fixtures/Database/FilterDatesByParentLocationViaFlexform.php @@ -0,0 +1,116 @@ + [ + [ + 'pid' => '1', + 'uid' => '1', + 'CType' => 'list', + 'list_type' => 'events_datelist', + 'header' => 'Kino Events', + 'pi_flexform' => ' + + + + + + 1 + + + + + + ', + ], + ], + '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', + ], + ], +]; diff --git a/ext_tables.sql b/ext_tables.sql index fe6210d..01a8882 100644 --- a/ext_tables.sql +++ b/ext_tables.sql @@ -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) );