clarat-org / clarat

An Open Source Charity Directory
http://www.clarat.org/
MIT License
8 stars 10 forks source link

Pagination innerhalb von Divisions (Objekten generell?) kaputt #1361

Closed Twiek closed 6 years ago

Twiek commented 6 years ago

Wenn ich mir im Backend die Division mit der ID 5 anschaue, dann sind dort 24 verknüpfte SplitBases aufgeführt. Wenn ich mir aber mit Hilfe der Pagination diese 24 anschauen, dann sind dort diverse doppelt aufgelistet. Konkret sehe ich dort die folgenden 24 IDs: 701, 716, 751, 865, 867, 882, 883, 886, 887, 1104, 1116, 1282, 1293, 1299, 1321, 1407, 1293, 865, 716, 2818, 1104, 882, 1282, 751. Wenn ich die doppelten (z.B. 751) lösche, dann bleiben nur noch 17 übrig (701, 716, 751, 865, 867, 882, 883, 886, 887, 1104, 1116, 1282, 1293, 1299, 1321, 1407, 2818). Es sind aber wirklich 24. Auf diese Zahl kommt man, wenn man über die Konsole geht, oder interessanter Weise, wenn man im Backend die SplitBases nach IDs sortiert.

Es sollten egal wie die verknüpften Objekte sortiert sind alle und keine doppelten angezeigt werden. Gerne Sortierung nach ID als default.

KonstantinKo commented 6 years ago

Soweit ich das nachvollziehen kann, hängt das mit unserem Freund eager_load zusammen, weil schainbar mit einem LEFT OUTER JOIN sehr andere Dinge passieren als mit dem INNER JOIN eines joins. Bei letzterem funktioniert es wie erwartet. Ich würde da gern nochmal mit @NilsVollmer sprechen, wo genau die Probleme bei joins lagen.

KonstantinKo commented 6 years ago

Fixed via default ordering. Wir hoffen mal, dass diese eager_load Schwäche keine weiteren Auswirkungen hat

KonstantinKo commented 6 years ago

als fix auf master entwickelt & deployed