fix: backport fix for #3579

Author: thorsten

phpmyfaq/search.php

@@ -148,8 +148,14 @@
             }
         }
 
-        $searchResults = $faq->renderFaqsByFaqIds($recordIds);
         $numOfResults = count($recordIds);
+
+        // Apply pagination to record IDs for tag search
+        $confPerPage = $faqConfig->get('records.numberOfRecordsPerPage');
+        $first = ($page - 1) * $confPerPage;
+        $paginatedRecordIds = array_slice($recordIds, $first, $confPerPage);
+
+        $searchResults = $faq->renderFaqsByFaqIds($paginatedRecordIds, 'fd.id', 'ASC', false);
     }
 } else {
     $searchResults = [];

phpmyfaq/src/phpMyFAQ/Faq.php

@@ -458,10 +458,15 @@ public function renderFaqsByCategoryId(int $categoryId, string $orderBy = 'id',
      * @param int[]  $faqIds Array of record ids
      * @param string $orderBy Order by
      * @param string $sortBy Sort by
+     * @param bool   $usePagination Whether to use internal pagination
      * @throws CommonMarkException
      */
-    public function renderFaqsByFaqIds(array $faqIds, string $orderBy = 'fd.id', string $sortBy = 'ASC'): array
-    {
+    public function renderFaqsByFaqIds(
+        array $faqIds,
+        string $orderBy = 'fd.id',
+        string $sortBy = 'ASC',
+        bool $usePagination = true
+    ): array {
         $records = implode(', ', $faqIds);
         $page = Filter::filterInput(INPUT_GET, 'seite', FILTER_VALIDATE_INT, 1);
 
@@ -530,21 +535,20 @@ public function renderFaqsByFaqIds(array $faqIds, string $orderBy = 'fd.id', str
         $num = $this->configuration->getDb()->numRows($result);
         $numberPerPage = $this->configuration->get('records.numberOfRecordsPerPage');
 
-        if ($page == 1) {
-            $first = 0;
-        } else {
-            $first = ($page * $numberPerPage) - $numberPerPage;
-        }
+        $first = $usePagination && $page > 1 ? ($page * $numberPerPage) - $numberPerPage : 0;
 
         $searchResults = [];
         if ($num > 0) {
             $counter = 0;
             $displayedCounter = 0;
             $lastFaqId = 0;
             $faqHelper = new FaqHelper($this->configuration);
-            while (($row = $this->configuration->getDb()->fetchObject($result)) && $displayedCounter < $numberPerPage) {
+            while (
+                ($row = $this->configuration->getDb()->fetchObject($result)) &&
+                (!$usePagination || $displayedCounter < $numberPerPage)
+            ) {
                 ++$counter;
-                if ($counter <= $first) {
+                if ($usePagination && $counter <= $first) {
                     continue;
                 }