Add option to disable pagination for rex_list

[AndiLeni]

Hallo,

es wäre schön, wenn die Paginierung für rex_list() deaktiviert werden könnte.
Aktuell macht rex_list() einen SELECT SQL_CALC_FOUND_ROWS Aufruf, welcher ab einer gewissen Zeilenanzahl sehr unperformant läuft.
Zudem benötigen manche Addons die Paginierung nicht, beispielsweise weil diese eine JS Bibliothek wie Datatables für die Paginierung verwenden.

Hier ein Beispiel wie viel diese zusätzliche Anfrage ausmachen kann:

REDAXO: 5.12
PHP: 7.4

[TobiasKrais]

Erlebe das schon in der Praxis. Das statistics Addon ist aufgrund dieser fehlenden Option extrem langsam, da bei mir viele Datensätze vorhanden sind.

[gharlan]

Von SQL_CALC_FOUND_ROWS müssen wir übrigens sowieso weg, da deprecated: #4296

Aber unabhängig davon können wir gerne auch einen Modus ganz ohne Paginierung einbauen.
Ich schlage vor, den $rowsPerPage-Parameter nullable zu machen, und wenn null, dann keine Paginierung: https://github.com/redaxo/redaxo/blob/main/redaxo/src/core/lib/list.php#L132

Mag das einer von euch vielleicht als PR umsetzen?

[staabm]

Soweit ich mich erinnere wurde das problem im Slack weiter eingegrenzt.

Am ende hat ein index in der db gefehlt und das SQL_CALC_FOUND_ROWS hatte mit dem eigentlichen Problem nix zu tun

[AndiLeni]

Danke für die schnelle Umsetzung!

Wenn ich das richtig verstehe wird das deaktivieren erst ab redaxo 5.13 möglich sein, oder?
Wie verhält der Code sich da in älteren Versionen, wenn die pagination Länge auf Null gesetzt wird?

[gharlan]

Wenn ich das richtig verstehe wird das deaktivieren erst ab redaxo 5.13 möglich sein, oder?

Ja richtig.

Wie verhält der Code sich da in älteren Versionen, wenn die pagination Länge auf Null gesetzt wird?

Ich vermute genauso wie 0, also wahrscheinlich werden 0 Datensätze pro Seite angezeigt.

Ungetestet, aber ich würde es daher so in der Art notieren, wenn ich kompatibel zu älteren Versionen bleiben möchte:

$list = rex_list::factory($query, defined(rex_list::class.'::DISABLE_PAGINATION') ? rex_list::DISABLE_PAGINATION : 100000);