Bug #85359

BE-Pagebrowser broken

Added by David Bruchmann over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2018-06-23
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

The pagebrowser to browse records shows only the first page.
I tested with pages as well as news-records.
TYPO3 9.3

recordbrowser-p2.gif View (3.24 KB) David Bruchmann, 2018-06-23 13:46

recordbrowser-p1.gif View (7.51 KB) David Bruchmann, 2018-06-23 13:46


Related issues

Related to TYPO3 Core - Bug #78129: Statement::rowCount not reliable for SELECT queries Closed 2016-10-02
Duplicated by TYPO3 Core - Bug #85743: An exception occurred if I click on List Closed 2016-10-03 2016-10-03

Associated revisions

Revision e5045205 (diff)
Added by Georg Ringer over 1 year ago

[BUGFIX] Calculate count of records instead of using a query

Calculate the count of records within the code instead of using a
query which is conceptionally wrong using limit and offset.

Resolves: #85359
Releases: master
Change-Id: I5b4356fb860b73d64585f28a3bd4384a7884715a
Reviewed-on: https://review.typo3.org/57347
Tested-by: TYPO3com <>
Reviewed-by: David Bruchmann <>
Tested-by: David Bruchmann <>
Reviewed-by: Jigal van Hemert <>
Tested-by: Jigal van Hemert <>

History

#2 Updated by Georg Ringer over 1 year ago

  • Status changed from New to Accepted
  • Is Regression set to Yes

reason is the change #78129

#3 Updated by Georg Ringer over 1 year ago

  • Related to Bug #78129: Statement::rowCount not reliable for SELECT queries added

#4 Updated by Georg Ringer over 1 year ago

reason the change in DatabaseRecordList line 1109

previously

$dbCount = $queryResult->rowCount();

now

$dbCount = $queryBuilder
                                        ->count('uid')
                                    ->execute()
                                    ->fetchColumn(0)

#5 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57347

#6 Updated by Georg Ringer over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#7 Updated by Stephan GroƟberndt over 1 year ago

  • Description updated (diff)

#8 Updated by bitfire it services about 1 year ago

Had this problem too (clean Typo3 9.3.3) with a custom record type and can confirm that it's fixed with the patch.

Thanks!

#9 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

#10 Updated by Susanne Moog about 1 year ago

  • Duplicated by Bug #85743: An exception occurred if I click on List added

Also available in: Atom PDF