Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paginierung mit GET in Verbindung mit Filter #1493

Closed
4 tasks done
Shania567 opened this issue Feb 15, 2023 · 2 comments
Closed
4 tasks done

Paginierung mit GET in Verbindung mit Filter #1493

Shania567 opened this issue Feb 15, 2023 · 2 comments
Assignees
Labels
bug A bug! A bug! Fast, squish it!
Milestone

Comments

@Shania567
Copy link

Shania567 commented Feb 15, 2023

Checklist before I submit this issue report

I confirm that:

Issue description

Mein MetaModel ist für die Paginierung der Listenansicht auf GET eingestellt.

paginierung

Ich habe eine Liste mit vielen Seiten und über der Liste einen Filter. Nun gehe ich in der Liste z.B. auf die Seite 4.

Der Link ändert sich zu
https://www.***.net/seitenname.html?page=4

Nun klicke ich oben in der Auswahl und möchte die Liste einschränken.

Der Link ändert sich zu
https://www.***.net/seitenname/page/4/filtername/ausprägungsname.html

Jetzt gibt es aber gar keine 4 Seiten mehr, sondern nur noch zwei. Angezeigt wird die erste Seite. Klicke ich dann auf Seite 2 sieht der Links so aus

https://www.***.net/seitenname/page/4/filtername/ausprägungsname.html?page=2

Diese Seite gibt es natürlich nicht und das wird auch angezeigt.

Dieser Fehler führt auch in 2.3 mit dazu, dass ich den Suchindex nicht einfach aktualisieren kann. Es werden viel zu viele mögliche Seiten erzeugt, die alle gecrawlt werden sollen. Das läuft sich zu Tode. Ich muss dann alle Filter ausblenden, damit ich den Seitenindex aufbauen kann. Da ich auf der Seite aber zahlreiche MMs nutze, ist das nicht wirklich praktikabel und geht maximal bei einem Upgrade. Das ist nun seit MM 2.2 so bzw. Contao 4.9. Da hätte ich es wohl besser schon hier gemeldet.

Link zum Thema im Forum:
https://community.contao.org/de/showthread.php?84597-MM-2-3-Paginierung
zonky kam im Forum darauf, dass es an der GET Methode liegen könnte, die sich mit slug der Filter beißt. Das habe ich getestet und ist erst einmal korrekt. Stelle ich die Paginierung auf slug um, gibt es keine nicht gefundenen Seiten.

Gehe ich mit slug genauso vor wie oben beschrieben bleibt zwar die Seite 4 nach der Auswahl des Filters erst einmal stehen, auch, wenn es gar keine Seite 4 mit dem Filter mehr gibt (so gesehen ist das wohl nicht ganz richtig, aber immerhin funktioniert es erst einmal), aber wenn man die Seite wechselt, auf z.B. Seite 2, wird der Link korrekt angepasst und kommt auch die Seite 2.

Würde man den Link mit der Seite 4 posten, klappt es aber sicherlich auch an dieser Stelle nicht. Das habe ich aber jetzt nichts ausprobiert.

@zonky2
Copy link
Contributor

zonky2 commented Feb 15, 2023

Workaround: Paginierung als Slug-Parameter verwenden

@zonky2 zonky2 self-assigned this Feb 15, 2023
@zonky2 zonky2 added the bug A bug! A bug! Fast, squish it! label Feb 15, 2023
@zonky2 zonky2 added this to the 2.2.8 milestone Feb 15, 2023
@zonky2
Copy link
Contributor

zonky2 commented Mar 1, 2023

Fixed in MM 2.3

@zonky2 zonky2 closed this as completed Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug! A bug! Fast, squish it!
Projects
None yet
Development

No branches or pull requests

2 participants