[BUGFIX] Render correct paging label 79/58779/4
authorAndreas Fernandez <a.fernandez@scripting-base.de>
Mon, 29 Oct 2018 13:58:40 +0000 (14:58 +0100)
committerBenni Mack <benni@typo3.org>
Mon, 29 Oct 2018 15:15:53 +0000 (16:15 +0100)
In case of paging in any record list, the page browser is rendered in top
and on bottom of ther record list.

The upper browser now renders the paging correctly by calculating the
offset of items of the previous page instead of hard-coding a "1".

Resolves: #86787
Releases: master, 8.7
Change-Id: Ib7c8856f248107ea9d17aeabe83a564189f69abf
Reviewed-on: https://review.typo3.org/58779
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
typo3/sysext/backend/Classes/RecordList/AbstractRecordList.php
typo3/sysext/backend/Classes/View/PageLayoutView.php
typo3/sysext/filelist/Classes/FileList.php
typo3/sysext/recordlist/Classes/RecordList/DatabaseRecordList.php

index 29e63f9..76d2d10 100644 (file)
@@ -377,7 +377,7 @@ abstract class AbstractRecordList
         switch ($type) {
             case 'fwd':
                 $href = $this->listURL() . '&pointer=' . ($pointer - $this->iLimit) . $tParam;
-                $content = '<a href="' . htmlspecialchars($href) . '">' . $this->iconFactory->getIcon('actions-move-up', Icon::SIZE_SMALL)->render() . '</a> <i>[1 - ' . $pointer . ']</i>';
+                $content = '<a href="' . htmlspecialchars($href) . '">' . $this->iconFactory->getIcon('actions-move-up', Icon::SIZE_SMALL)->render() . '</a> <i>[' . (max(0, $pointer - $this->iLimit) + 1) . ' - ' . $pointer . ']</i>';
                 break;
             case 'rwd':
                 $href = $this->listURL() . '&pointer=' . $pointer . $tParam;
index 2972577..c60254a 100644 (file)
@@ -4253,7 +4253,7 @@ class PageLayoutView implements LoggerAwareInterface
                 $content = '<a href="' . htmlspecialchars($href) . '">' . $this->iconFactory->getIcon(
                         'actions-move-up',
                         Icon::SIZE_SMALL
-                    )->render() . '</a> <i>[1 - ' . $pointer . ']</i>';
+                    )->render() . '</a> <i>[' . (max(0, $pointer - $this->iLimit) + 1) . ' - ' . $pointer . ']</i>';
                 break;
             case 'rwd':
                 $href = $this->listURL() . '&pointer=' . $pointer . $tParam;
index 4c920bf..e099432 100644 (file)
@@ -696,7 +696,7 @@ class FileList
                 $content = '<a href="' . htmlspecialchars($href) . '">' . $this->iconFactory->getIcon(
                         'actions-move-up',
                         Icon::SIZE_SMALL
-                    )->render() . '</a> <i>[1 - ' . $pointer . ']</i>';
+                    )->render() . '</a> <i>[' . (max(0, $pointer - $this->iLimit) + 1) . ' - ' . $pointer . ']</i>';
                 break;
             case 'rwd':
                 $href = $this->listURL() . '&pointer=' . $pointer . $tParam;
index 264d6a8..22562a5 100644 (file)
@@ -4247,7 +4247,7 @@ class DatabaseRecordList
                 $content = '<a href="' . htmlspecialchars($href) . '">' . $this->iconFactory->getIcon(
                         'actions-move-up',
                         Icon::SIZE_SMALL
-                    )->render() . '</a> <i>[1 - ' . $pointer . ']</i>';
+                    )->render() . '</a> <i>[' . (max(0, $pointer - $this->iLimit) + 1) . ' - ' . $pointer . ']</i>';
                 break;
             case 'rwd':
                 $href = $this->listURL() . '&pointer=' . $pointer . $tParam;