If you enable another translation, and disable english, then if you dont have a cookie set, or your browser is not set to that language, you will get english. So you can not disable english in either the staff client or the OPAC.
Created attachment 19509 [details] [review] Bug 10560 - fix the language selection to do the right thing
Created attachment 19582 [details] [review] [SIGNED-OFF] Bug 10560 - No way to disable english If you enable another translation, and disable english, then if you dont have a cookie set, or your browser is not set to that language, you will get english. So you can not disable english in either the staff client or the OPAC. This patch fixes the language selection to do the right thing. To test you must have at least one other language installed besides English. Apply the patch and disable the en translation. Koha should fall back to one of the enabled translations. Signed-off-by: Owen Leonard <oleonard@myacpl.org> I added a patch description and test plan, missing from the original patch.
Created attachment 19584 [details] [review] [PASSED QA] Bug 10560 - No way to disable english If you enable another translation, and disable english, then if you dont have a cookie set, or your browser is not set to that language, you will get english. So you can not disable english in either the staff client or the OPAC. This patch fixes the language selection to do the right thing. To test you must have at least one other language installed besides English. Apply the patch and disable the en translation. Koha should fall back to one of the enabled translations. Signed-off-by: Owen Leonard <oleonard@myacpl.org> I added a patch description and test plan, missing from the original patch. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> I have tested with various combinations of activated languages and have found no regression. If the cookie is set, the right language is shown accordingly. Else the first language in the list seems to be picked. It did never fall back to English in my tests, when English was explicitly deactivated. Passes all tests and QA script.
Pushed to master, along with a follow-up I wrote that adds a regression test. Thanks, Robin!
This patch has been pushed to 3.12.x, will be in 3.12.2. Thanks Robin! Note: i think the webinstaller didn't get fixed. After I push the latest patches to 3.12.x i'll try to re-test.
Pushed to 3.10.x, will be in 3.10.10
Pushed to 3.8.x, will be in 3.8.17