search

crs-tools / tracker

CRS Ticket Tracker
Apache License 2.0
18 stars 11 forks source link

Suche auf GET Umstellen zwecks Bookmarking #111

Open a-tze opened 9 years ago

a-tze commented 9 years ago

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 commented 9 years ago

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.