Check locale availability upon configuration save

When the locale chosen for FD is not installed on the server, setting the locale fails and FD triggers a PHP error to signal the problem.

I think it would make sense in the configuration screen to check if the locale chosen is available in the check() method, therefore forbidding the admin from selecting a locale that will cause problems.

This does not fix the case where locale is set to automatic and the user browser asks for a locale that is not set up, but it will help with other cases where a locale is chosen in configuration.

changed milestone to %FusionDirectory 1.4

By bmortier on 2019-11-21T10:11:37 (imported from GitLab)

added PJ1802-0188 label

changed due date to November 21, 2019

By bmortier on 2019-11-21T10:11:50 (imported from GitLab)

created merge request !715 to address this issue

By Côme Chilliet on 2019-11-21T10:18:51 (imported from GitLab)

mentioned in merge request !715

By Côme Chilliet on 2019-11-21T10:18:52 (imported from GitLab)

mentioned in commit e598dddb

By Côme Chilliet on 2019-11-21T10:40:46 (imported from GitLab)

added 30m of time spent at 2019-11-21

By Côme Chilliet on 2019-11-21T10:42:50 (imported from GitLab)

added To Be Tested label

assigned to @jswaelens and unassigned @MCMic

By Côme Chilliet on 2019-11-26T08:20:49 (imported from GitLab)

Just tested on demo-dev and it works well. Could it be possible to extend it so that it also works when users choice a languages or maybe displaying only languages that are working on the server ?

Cheers

By Jonathan Swaelens on 2019-11-29T10:06:29 (imported from GitLab)

added 10m of time spent at 2019-11-29

By Jonathan Swaelens on 2019-11-29T10:06:29 (imported from GitLab)

removed To Be Tested label

assigned to @MCMic and unassigned @jswaelens

By Jonathan Swaelens on 2019-11-29T10:06:29 (imported from GitLab)

added fusiondirectory-core label

@jswaelens Not sure it makes sense, the locale chosen by the user may be used in other softwares than FusionDirectory.

By Côme Chilliet on 2019-12-03T08:23:39 (imported from GitLab)

@MCMic

Is there a way to maybe "difference" the one that would change FusionDirectory language ?

I can imagine the user that want change the language to something but that it didn't work

By Jonathan Swaelens on 2019-12-03T09:47:58 (imported from GitLab)

added 5m of time spent at 2019-12-03

By Jonathan Swaelens on 2019-12-03T09:47:58 (imported from GitLab)

The problem can also happen with the language requested by the user browser, so we won’t be able to completely remove this kind of problems. The point of this ticket was to avoid the problem in the configuration screen as it was quite easy to do.

By Côme Chilliet on 2019-12-17T09:17:50 (imported from GitLab)

closed

By Côme Chilliet on 2019-12-17T09:17:50 (imported from GitLab)

added 5m of time spent at 2019-12-17

By Côme Chilliet on 2019-12-17T09:17:51 (imported from GitLab)

added Added label

reopened

created merge request !817 to address this issue

By Côme Chilliet on 2020-10-20T07:38:11 (imported from GitLab)

mentioned in merge request !817

By Côme Chilliet on 2020-10-20T07:38:12 (imported from GitLab)

mentioned in commit 1361ecdb

By Côme Chilliet on 2020-10-20T07:47:32 (imported from GitLab)

mentioned in commit 3ee4b668

By Côme Chilliet on 2020-10-20T07:47:32 (imported from GitLab)

mentioned in commit cfb5c8fe

By Côme Chilliet on 2020-10-20T07:47:32 (imported from GitLab)

closed

added 30m of time spent at 2020-10-20

By Côme Chilliet on 2020-10-20T07:53:23 (imported from GitLab)

removed due date

By bmortier on 2021-01-28T20:43:27 (imported from GitLab)