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

Suche auf GET Umstellen zwecks Bookmarking #111

Open
a-tze opened this issue Aug 23, 2015 · 1 comment
Open

Suche auf GET Umstellen zwecks Bookmarking #111

a-tze opened this issue Aug 23, 2015 · 1 comment

Comments

@a-tze
Copy link
Collaborator

a-tze commented Aug 23, 2015

Issue migrated from trac ticket # 111

component: unknown | priority: minor

2015-08-23 13:10:28: MaZderMind created the issue


Es wäre hilfreich, die Suche als GET statt als POST-Request abzuwickeln. So könnte man sich Suchergebnisse (z.B. "alle zu Checkenden Master-Endcoding-Tickets") Bookmarken.

Auch der Back-Button im Browser könnte dann sinnvoll funktionieren.

@a-tze
Copy link
Collaborator Author

a-tze commented Sep 5, 2015

2015-09-05 02:21:24: @jjeising commented


GET alleine reicht nicht um die gewünschte Funktionalität abzubilden. Es ist zwar kein Problem das zu ändern (und das funktioniert auch), allerdings benötigen zur Zeit alle Form-Submissions einen Token, der auch nur einmal gültig ist. Die URLs wären also weiterhin nur einmal nutzbar, auch wenn sie alle nötigen Informationen theoretisch enthalten.

Um das zu unterstützen sind größere Änderungen in der Klasse Form notwendig, hier sollten zuerst einige Tests ergänzt werden. Weiterhin muss dann Controller_Tickets::evaluateSearch darauf hin geprüft werden, dass es korrekt mit nicht mehr validierten Inputs (z.B. aus <select>-Feldern) umgeht.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants