Fix SlimSelect dropdown scrolling regression

contao/templates/frontend/event_filter.html.twig

@@ -81,6 +81,11 @@
     #eventfilters .eventfilter-reset[hidden] {
         display: none;
     }
+
+    #eventfilters .ss-content,
+    #eventfilters .ss-list {
+        overscroll-behavior: contain;
+    }
 </style>
 
 <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/slim-select@3/dist/slimselect.css">
@@ -332,5 +337,110 @@
         applyFilter({ type: 'all', value: 'all' });
     });
 
+    const getDropdownFromTarget = (target) => {
+        if (!(target instanceof Element)) {
+            return null;
+        }
+
+        return target.closest('.ss-content, .ss-list');
+    };
+
+    const getScrollContainer = (targetDropdown) => {
+        if (!targetDropdown) {
+            return null;
+        }
+
+        if (targetDropdown.scrollHeight > targetDropdown.clientHeight) {
+            return targetDropdown;
+        }
+
+        const contentContainer = targetDropdown.closest('.ss-content') || targetDropdown;
+        const listContainer = contentContainer.querySelector('.ss-list');
+
+        if (listContainer && listContainer.scrollHeight > listContainer.clientHeight) {
+            return listContainer;
+        }
+
+        if (contentContainer.scrollHeight > contentContainer.clientHeight) {
+            return contentContainer;
+        }
+
+        return null;
+    };
+
+    const shouldBlockBoundaryScroll = (scrollContainer, delta) => {
+        if (!scrollContainer || delta === 0) {
+            return false;
+        }
+
+        const top = scrollContainer.scrollTop;
+        const height = scrollContainer.clientHeight;
+        const scrollHeight = scrollContainer.scrollHeight;
+
+        const atTop = top <= 0;
+        const atBottom = top + height >= scrollHeight - 1;
+
+        return (atTop && delta < 0) || (atBottom && delta > 0);
+    };
+
+    let lastTouchY = null;
+
+    document.addEventListener('wheel', (event) => {
+        const dropdown = getDropdownFromTarget(event.target);
+        if (!dropdown) {
+            return;
+        }
+
+        const scrollContainer = getScrollContainer(dropdown);
+
+        if (shouldBlockBoundaryScroll(scrollContainer, event.deltaY)) {
+            event.preventDefault();
+        }
+
+        event.stopPropagation();
+    }, { passive: false, capture: true });
+
+    document.addEventListener('touchstart', (event) => {
+        const dropdown = getDropdownFromTarget(event.target);
+        if (!dropdown || event.touches.length === 0) {
+            lastTouchY = null;
+            return;
+        }
+
+        lastTouchY = event.touches[0].clientY;
+    }, { passive: true, capture: true });
+
+    document.addEventListener('touchmove', (event) => {
+        const dropdown = getDropdownFromTarget(event.target);
+        if (!dropdown || event.touches.length === 0 || lastTouchY === null) {
+            return;
+        }
+
+        const scrollContainer = getScrollContainer(dropdown);
+
+        if (!scrollContainer) {
+            return;
+        }
+
+        const currentTouchY = event.touches[0].clientY;
+        const delta = lastTouchY - currentTouchY;
+
+        if (shouldBlockBoundaryScroll(scrollContainer, delta)) {
+            event.preventDefault();
+        }
+
+        event.stopPropagation();
+
+        lastTouchY = currentTouchY;
+    }, { passive: false, capture: true });
+
+    document.addEventListener('touchend', () => {
+        lastTouchY = null;
+    }, { passive: true, capture: true });
+
+    document.addEventListener('touchcancel', () => {
+        lastTouchY = null;
+    }, { passive: true, capture: true });
+
     applyFilter(currentFilter);
 </script>

src/EventListener/OrganizationListingTemplateDataListener.php

@@ -34,11 +34,6 @@ class OrganizationListingTemplateDataListener
         }
 
         $rowToOrganizationIdMap = [];
-        $rowToTitleMap = [];
-
-        $resolvedByRowIdCount = 0;
-        $resolvedByTitleToIdCount = 0;
-        $enrichedByTitleTagFallbackCount = 0;
 
         foreach ($tbody as $rowIndex => $row) {
             if (!\is_array($row)) {
@@ -49,31 +44,6 @@ class OrganizationListingTemplateDataListener
 
             if ($organizationId > 0) {
                 $rowToOrganizationIdMap[(int) $rowIndex] = $organizationId;
-                ++$resolvedByRowIdCount;
-                continue;
-            }
-
-            $title = $this->extractRowFieldContent($row, 'title');
-
-            if ('' !== $title) {
-                $rowToTitleMap[(int) $rowIndex] = $this->normalizeTitle($title);
-            }
-        }
-
-        if ([] !== $rowToTitleMap) {
-            $titleToOrganizationIdMap = $this->resolveOrganizationIdsByTitle(array_values(array_unique(array_filter($rowToTitleMap))));
-
-            foreach ($rowToTitleMap as $rowIndex => $title) {
-                if (isset($rowToOrganizationIdMap[$rowIndex])) {
-                    continue;
-                }
-
-                $organizationId = $titleToOrganizationIdMap[$title] ?? 0;
-
-                if ($organizationId > 0) {
-                    $rowToOrganizationIdMap[$rowIndex] = $organizationId;
-                    ++$resolvedByTitleToIdCount;
-                }
             }
         }
 
@@ -81,21 +51,36 @@ class OrganizationListingTemplateDataListener
             return;
         }
 
-        $organizationTagMap = $this->fetchOrganizationTagMap(array_values(array_unique(array_values($rowToOrganizationIdMap))));
+        $organizationIds = array_values(array_unique(array_values($rowToOrganizationIdMap)));
-        $organizationLogoUuidMap = $this->fetchOrganizationLogoUuidMap(array_values(array_unique(array_values($rowToOrganizationIdMap))));
+        $organizationTagMap = $this->fetchOrganizationTagMap($organizationIds);
-        $organizationTagMapByTitle = $this->fetchOrganizationTagMapByTitle(array_values(array_unique(array_filter($rowToTitleMap))));
+        $organizationLogoUuidMap = $this->fetchOrganizationLogoUuidMap($organizationIds);
+
+        $organizationTagLabelMap = [];
+        $rowTagIdsList = [];
+
+        foreach ($organizationTagMap as $tagData) {
+            foreach (($tagData['ids'] ?? []) as $index => $tagId) {
+                $label = trim((string) (($tagData['labels'][$index] ?? '') ?: ''));
+
+                if ('' !== $tagId && '' !== $label && !isset($organizationTagLabelMap[(string) $tagId])) {
+                    $organizationTagLabelMap[(string) $tagId] = $label;
+                }
+            }
+        }
 
         foreach ($rowToOrganizationIdMap as $rowIndex => $organizationId) {
-            $tagData = $organizationTagMap[$organizationId] ?? ['labels' => [], 'slugs' => []];
-            $logoUuid = $organizationLogoUuidMap[$organizationId] ?? '';
-
             if (!isset($tbody[$rowIndex]) || !\is_array($tbody[$rowIndex])) {
                 continue;
             }
 
-            $tbody[$rowIndex]['tag_labels']['content'] = implode(', ', $tagData['labels']);
+            $tagData = $organizationTagMap[$organizationId] ?? ['ids' => [], 'labels' => []];
-            $tbody[$rowIndex]['tag_slugs']['content'] = implode(',', $tagData['slugs']);
+            $logoUuid = $organizationLogoUuidMap[$organizationId] ?? '';
-            $tbody[$rowIndex]['tags']['content'] = implode(', ', $tagData['labels']);
+            $tagIdsCsv = implode(',', $tagData['ids']);
+            $tagLabelsCsv = implode(', ', $tagData['labels']);
+
+            $tbody[$rowIndex]['tag_ids']['content'] = $tagIdsCsv;
+            $tbody[$rowIndex]['tag_labels']['content'] = $tagLabelsCsv;
+            $tbody[$rowIndex]['tags']['content'] = $tagLabelsCsv;
 
             if ('' !== $logoUuid) {
                 $tbody[$rowIndex]['logo_uuid']['content'] = $logoUuid;
@@ -103,60 +88,38 @@ class OrganizationListingTemplateDataListener
         }
 
         foreach ($tbody as $rowIndex => $row) {
-            if (!\is_array($row) || isset($rowToOrganizationIdMap[$rowIndex])) {
-                continue;
-            }
-
-            $title = $this->normalizeTitle($this->extractRowFieldContent($row, 'title'));
-
-            if ('' === $title || !isset($organizationTagMapByTitle[$title])) {
-                continue;
-            }
-
-            $tagData = $organizationTagMapByTitle[$title];
-
-            $tbody[$rowIndex]['tag_labels']['content'] = implode(', ', $tagData['labels']);
-            $tbody[$rowIndex]['tag_slugs']['content'] = implode(',', $tagData['slugs']);
-            $tbody[$rowIndex]['tags']['content'] = implode(', ', $tagData['labels']);
-            ++$enrichedByTitleTagFallbackCount;
-        }
-
-        $rowsWithoutTagSlugs = 0;
-        $sampleUnresolvedTitles = [];
-
-        foreach ($tbody as $row) {
             if (!\is_array($row)) {
+                $rowTagIdsList[] = '';
                 continue;
             }
 
-            $tagSlugs = $this->extractRowFieldContent($row, 'tag_slugs');
+            $organizationId = $rowToOrganizationIdMap[(int) $rowIndex] ?? 0;
-
+            $rowTagIdsList[] = ($organizationId > 0 && isset($organizationTagMap[$organizationId]))
-            if ('' !== $tagSlugs) {
+                ? implode(',', $organizationTagMap[$organizationId]['ids'] ?? [])
-                continue;
+                : '';
         }
 
-            ++$rowsWithoutTagSlugs;
+        if (null !== $this->logger) {
+            $rowsWithoutTagIds = 0;
 
-            if (\count($sampleUnresolvedTitles) < 10) {
+            foreach ($rowToOrganizationIdMap as $organizationId) {
-                $title = $this->normalizeTitle($this->extractRowFieldContent($row, 'title'));
+                if (!isset($organizationTagMap[$organizationId]) || [] === ($organizationTagMap[$organizationId]['ids'] ?? [])) {
-
+                    ++$rowsWithoutTagIds;
-                if ('' !== $title) {
-                    $sampleUnresolvedTitles[] = $title;
-                }
                 }
             }
 
-        if (null !== $this->logger && $rowsWithoutTagSlugs > 0) {
+            if ($rowsWithoutTagIds > 0) {
-            $this->logger->warning('Organization listing tag enrichment left rows without tag slugs.', [
+                $this->logger->warning('Organization listing enrichment found rows without tag IDs.', [
                     'template' => (string) $template->getName(),
                     'totalRows' => \count($tbody),
-                'resolvedByRowId' => $resolvedByRowIdCount,
+                    'mappedRows' => \count($rowToOrganizationIdMap),
-                'resolvedByTitleToId' => $resolvedByTitleToIdCount,
+                    'rowsWithoutTagIds' => $rowsWithoutTagIds,
-                'enrichedByTitleTagFallback' => $enrichedByTitleTagFallbackCount,
-                'rowsWithoutTagSlugs' => $rowsWithoutTagSlugs,
-                'sampleUnresolvedTitles' => array_values(array_unique($sampleUnresolvedTitles)),
                 ]);
             }
+        }
+
+        $template->organization_tag_label_map = $organizationTagLabelMap;
+        $template->organization_row_tag_ids_list = $rowTagIdsList;
 
         $template->tbody = $tbody;
     }
@@ -165,6 +128,10 @@ class OrganizationListingTemplateDataListener
     private function extractOrganizationId(array $row): int
     {
         foreach (['id', 'organization_id', 'org_id'] as $fieldName) {
+            if (isset($row[$fieldName]) && \is_scalar($row[$fieldName]) && ctype_digit((string) $row[$fieldName])) {
+                return (int) $row[$fieldName];
+            }
+
             $value = $this->extractRowFieldContent($row, $fieldName);
 
             if ('' !== $value && ctype_digit($value)) {
@@ -172,29 +139,23 @@ class OrganizationListingTemplateDataListener
             }
         }
 
-        $urlCandidates = [];
-
         foreach ($row as $column) {
             if (!\is_array($column)) {
                 continue;
             }
 
-            if (isset($column['url']) && \is_scalar($column['url'])) {
+            foreach (['url', 'href'] as $urlField) {
-                $urlCandidates[] = (string) $column['url'];
+                if (!isset($column[$urlField]) || !\is_scalar($column[$urlField])) {
+                    continue;
                 }
 
-            if (isset($column['href']) && \is_scalar($column['href'])) {
+                $organizationId = $this->extractIdFromUrl((string) $column[$urlField]);
-                $urlCandidates[] = (string) $column['href'];
-            }
-        }
-
-        foreach ($urlCandidates as $url) {
-            $organizationId = $this->extractIdFromUrl($url);
 
                 if ($organizationId > 0) {
                     return $organizationId;
                 }
             }
+        }
 
         return 0;
     }
@@ -202,7 +163,15 @@ class OrganizationListingTemplateDataListener
     /** @param array<string, mixed> $row */
     private function extractRowFieldContent(array $row, string $fieldName): string
     {
-        if (!isset($row[$fieldName]) || !\is_array($row[$fieldName])) {
+        if (!isset($row[$fieldName])) {
+            return '';
+        }
+
+        if (\is_scalar($row[$fieldName])) {
+            return trim((string) $row[$fieldName]);
+        }
+
+        if (!\is_array($row[$fieldName])) {
             return '';
         }
 
@@ -240,57 +209,8 @@ class OrganizationListingTemplateDataListener
         return 0;
     }
 
-    /** @param list<string> $titles
-     *  @return array<string, int>
-     */
-    private function resolveOrganizationIdsByTitle(array $titles): array
-    {
-        if ([] === $titles) {
-            return [];
-        }
-
-        $rows = $this->connection->executeQuery(
-            'SELECT o.id, o.title FROM tl_organization o WHERE o.title IN (?)',
-            [$titles],
-            [ArrayParameterType::STRING],
-        )->fetchAllAssociative();
-
-        $titleToIdsMap = [];
-
-        foreach ($rows as $row) {
-            $organizationId = (int) ($row['id'] ?? 0);
-            $title = $this->normalizeTitle((string) ($row['title'] ?? ''));
-
-            if ($organizationId <= 0 || '' === $title) {
-                continue;
-            }
-
-            $titleToIdsMap[$title][] = $organizationId;
-        }
-
-        $titleToOrganizationIdMap = [];
-
-        foreach ($titleToIdsMap as $title => $organizationIds) {
-            $organizationIds = array_values(array_unique(array_map('intval', $organizationIds)));
-
-            if (1 === \count($organizationIds)) {
-                $titleToOrganizationIdMap[$title] = $organizationIds[0];
-            }
-        }
-
-        return $titleToOrganizationIdMap;
-    }
-
-    private function normalizeTitle(string $title): string
-    {
-        $title = html_entity_decode($title, ENT_QUOTES | ENT_HTML5, 'UTF-8');
-        $title = trim(strip_tags($title));
-
-        return preg_replace('/\s+/u', ' ', $title) ?? '';
-    }
-
     /** @param list<int> $organizationIds
-     *  @return array<int, array{labels: list<string>, slugs: list<string>}>
+     *  @return array<int, array{ids: list<string>, labels: list<string>}>
      */
     private function fetchOrganizationTagMap(array $organizationIds): array
     {
@@ -299,35 +219,27 @@ class OrganizationListingTemplateDataListener
         }
 
         $rows = $this->connection->executeQuery(
-            'SELECT r.pid AS organization_id, r.tag_id, t.tag AS label FROM tl_tags_rel r INNER JOIN tl_tags t ON t.id = r.tag_id WHERE r.ptable = ? AND r.field = ? AND r.pid IN (?) ORDER BY r.pid ASC, r.tag_id ASC',
+            'SELECT r.pid AS organization_id, r.tag_id, t.tag AS label
+             FROM tl_tags_rel r
+             INNER JOIN tl_tags t ON t.id = r.tag_id
+             WHERE r.ptable = ? AND r.field = ? AND r.pid IN (?)
+             ORDER BY r.pid ASC, r.tag_id ASC',
             ['tl_organization', 'tags', $organizationIds],
             [ParameterType::STRING, ParameterType::STRING, ArrayParameterType::INTEGER],
         )->fetchAllAssociative();
 
         if ([] === $rows) {
             $rows = $this->connection->executeQuery(
-                'SELECT r.pid AS organization_id, r.tag_id, t.tag AS label FROM tl_tags_rel r INNER JOIN tl_tags t ON t.id = r.tag_id WHERE r.ptable = ? AND r.pid IN (?) ORDER BY r.pid ASC, r.tag_id ASC',
+                'SELECT r.pid AS organization_id, r.tag_id, t.tag AS label
+                 FROM tl_tags_rel r
+                 INNER JOIN tl_tags t ON t.id = r.tag_id
+                 WHERE r.ptable = ? AND r.pid IN (?)
+                 ORDER BY r.pid ASC, r.tag_id ASC',
                 ['tl_organization', $organizationIds],
                 [ParameterType::STRING, ArrayParameterType::INTEGER],
             )->fetchAllAssociative();
         }
 
-        if ([] === $rows) {
-            $rows = $this->connection->executeQuery(
-                'SELECT r.pid AS organization_id, r.tag_id, t.tag AS label FROM tl_tags_rel r INNER JOIN tl_tags t ON t.id = r.tag_id WHERE r.field = ? AND r.pid IN (?) ORDER BY r.pid ASC, r.tag_id ASC',
-                ['tags', $organizationIds],
-                [ParameterType::STRING, ArrayParameterType::INTEGER],
-            )->fetchAllAssociative();
-        }
-
-        if ([] === $rows) {
-            $rows = $this->connection->executeQuery(
-                'SELECT r.pid AS organization_id, r.tag_id, t.tag AS label FROM tl_tags_rel r INNER JOIN tl_tags t ON t.id = r.tag_id WHERE r.pid IN (?) ORDER BY r.pid ASC, r.tag_id ASC',
-                [$organizationIds],
-                [ArrayParameterType::INTEGER],
-            )->fetchAllAssociative();
-        }
-
         $map = [];
         $seen = [];
 
@@ -336,92 +248,18 @@ class OrganizationListingTemplateDataListener
             $tagId = (int) ($row['tag_id'] ?? 0);
             $label = trim((string) ($row['label'] ?? ''));
 
-            if ($organizationId <= 0 || $tagId <= 0 || '' === $label) {
+            if ($organizationId <= 0 || $tagId <= 0 || '' === $label || isset($seen[$organizationId][$tagId])) {
-                continue;
-            }
-
-            if (isset($seen[$organizationId][$tagId])) {
                 continue;
             }
 
             $seen[$organizationId][$tagId] = true;
+            $map[$organizationId]['ids'][] = (string) $tagId;
             $map[$organizationId]['labels'][] = $label;
-
-            $slug = $this->slugify($label);
-
-            if ('' !== $slug) {
-                $map[$organizationId]['slugs'][] = $slug;
-            }
         }
 
         foreach ($map as $organizationId => $tagData) {
+            $map[$organizationId]['ids'] = array_values(array_unique($tagData['ids'] ?? []));
             $map[$organizationId]['labels'] = array_values(array_unique($tagData['labels'] ?? []));
-            $map[$organizationId]['slugs'] = array_values(array_unique($tagData['slugs'] ?? []));
-        }
-
-        return $map;
-    }
-
-    /** @param list<string> $titles
-     *  @return array<string, array{labels: list<string>, slugs: list<string>}>
-     */
-    private function fetchOrganizationTagMapByTitle(array $titles): array
-    {
-        if ([] === $titles) {
-            return [];
-        }
-
-        $rows = $this->connection->executeQuery(
-            'SELECT o.title, r.tag_id, t.tag AS label FROM tl_organization o INNER JOIN tl_tags_rel r ON r.pid = o.id AND r.ptable = ? INNER JOIN tl_tags t ON t.id = r.tag_id WHERE o.title IN (?) ORDER BY o.title ASC, r.tag_id ASC',
-            ['tl_organization', $titles],
-            [ParameterType::STRING, ArrayParameterType::STRING],
-        )->fetchAllAssociative();
-
-        if ([] === $rows) {
-            $rows = $this->connection->executeQuery(
-                'SELECT o.title, r.tag_id, t.tag AS label FROM tl_organization o INNER JOIN tl_tags_rel r ON r.pid = o.id AND r.field = ? INNER JOIN tl_tags t ON t.id = r.tag_id WHERE o.title IN (?) ORDER BY o.title ASC, r.tag_id ASC',
-                ['tags', $titles],
-                [ParameterType::STRING, ArrayParameterType::STRING],
-            )->fetchAllAssociative();
-        }
-
-        if ([] === $rows) {
-            $rows = $this->connection->executeQuery(
-                'SELECT o.title, r.tag_id, t.tag AS label FROM tl_organization o INNER JOIN tl_tags_rel r ON r.pid = o.id INNER JOIN tl_tags t ON t.id = r.tag_id WHERE o.title IN (?) ORDER BY o.title ASC, r.tag_id ASC',
-                [$titles],
-                [ArrayParameterType::STRING],
-            )->fetchAllAssociative();
-        }
-
-        $map = [];
-        $seen = [];
-
-        foreach ($rows as $row) {
-            $title = $this->normalizeTitle((string) ($row['title'] ?? ''));
-            $tagId = (int) ($row['tag_id'] ?? 0);
-            $label = trim((string) ($row['label'] ?? ''));
-
-            if ('' === $title || $tagId <= 0 || '' === $label) {
-                continue;
-            }
-
-            if (isset($seen[$title][$tagId])) {
-                continue;
-            }
-
-            $seen[$title][$tagId] = true;
-            $map[$title]['labels'][] = $label;
-
-            $slug = $this->slugify($label);
-
-            if ('' !== $slug) {
-                $map[$title]['slugs'][] = $slug;
-            }
-        }
-
-        foreach ($map as $title => $tagData) {
-            $map[$title]['labels'] = array_values(array_unique($tagData['labels'] ?? []));
-            $map[$title]['slugs'] = array_values(array_unique($tagData['slugs'] ?? []));
         }
 
         return $map;
@@ -485,24 +323,4 @@ class OrganizationListingTemplateDataListener
         return '';
     }
 
-    private function slugify(string $value): string
-    {
-        $value = trim(mb_strtolower($value));
-
-        if ('' === $value) {
-            return '';
-        }
-
-        $value = strtr($value, [
-            'ä' => 'ae',
-            'ö' => 'oe',
-            'ü' => 'ue',
-            'ß' => 'ss',
-        ]);
-
-        $value = preg_replace('/[^a-z0-9]+/u', '-', $value) ?? '';
-        $value = trim($value, '-');
-
-        return $value;
-    }
 }