Last modified: 2014-03-07 11:44:09 UTC
The parameter setlang should use [[Post/Redirect/Get]].
The "parameter" cannot use anything, since the user can type it in himself as well. Redirect introduces latency, and the usual reasons (avoiding double forms submission) don't seem to apply here.
Created attachment 11745 [details] POST instead of GET for setlang setlang changes a setting so it should use POST instead of GET. A simple POST without redirect would also be possible. See patch. But with a simple POST the problems described in [[Post/Redirect/Get]] still exist.
Fomafix: Thanks! Could you put the patch into Gerrit, or is that not possible?
I don't understand what is the advantage of doing this. Can anybody explain? If there is no clear advantage, I'd go for WONTFIX.
Change 110360 had a related patch set uploaded by Gerrit Patch Uploader: (Bug 44649) Use POST instead of GET with setlang to change user settings https://gerrit.wikimedia.org/r/110360
With setlang the user setting will set again when you reload the page.
Change 110360 had a related patch set uploaded by Gerrit Patch Uploader: Use POST instead of GET with setlang to change user settings https://gerrit.wikimedia.org/r/110360