? ?
avatar brianteeman
brianteeman
30 May 2017

When a site is only in one language there really is no need to display any of the language options below (there are probably other areas too).

We have an api we can check and we should do if (Multilanguage::isEnabled()).

It really makes no sense to display the language stuff in a monolingual site and just adds unneeded complexity to the ui
articles joomla 4 administration 1
articles joomla 4 administration
articles edit joomla 4 administration 1
articles edit joomla 4 administration

avatar brianteeman brianteeman - open - 30 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 30 May 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 30 May 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 30 May 2017
Category Administration com_languages Feature Request Templates (admin)
avatar Bakual
Bakual - comment - 30 May 2017

The associations tab is already hidden if the site isn't multilingual.
As far as I know it's only the language select that is always shown.

avatar brianteeman
brianteeman - comment - 30 May 2017

all the more reason to remove this useless column and field

avatar brianteeman
brianteeman - comment - 30 May 2017

Actually the associations tab is hidden if associations is not enabled - it uses JLanguageAssociations::isEnabled();
which is a different and more specific check than the one i proposed


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16370.

avatar infograf768
infograf768 - comment - 31 May 2017

JLanguageAssociations::isEnabled() implies JLanguageMultilang::isEnabled()
https://github.com/joomla/joomla-cms/blob/staging/libraries/cms/language/associations.php#L140

Now, about the RFC.
We have to take into account the fact that some 3rd party extensions may be using the language field without using our core language filter plugin. Therefore JLanguageMultilang::isEnabled() can't be the condition.

One solution I have been considering for a while is to create a new Global Configuration parameter. Obviously, when using our core mutilingual functionality, that parameter should be set (for example, enabling the core language filter plugin would automatically set that param if not done yet) and 3rd party multilingual extensions would also have to use it. (not B/C)

Now, practically, we have a lot of places where the field, list, filters, js, display/use Content Languages.
Which means a huge refactoring of the whole CMS in function of that new parameter afaik.

avatar brianteeman
brianteeman - comment - 31 May 2017

We have to take into account the fact that some 3rd party extensions may be using the language field without using our core language filter plugin. Therefore JLanguageMultilang::isEnabled() can't be the condition.

This is for J4 so we can break backwards compatibility and make it the conditions

Now, practically, we have a lot of places where the field, list, filters, js, display/use Content Languages.
Which means a huge refactoring of the whole CMS in function of that new parameter afaik.

so no reason not to do it then

avatar infograf768
infograf768 - comment - 31 May 2017

Looks like you did not read carefully what I wrote;

We have to take into account the fact that some 3rd party extensions may be using the language field without using our core language filter plugin. Therefore JLanguageMultilang::isEnabled() can't be the condition.

We can't force any 3rd party to use our system plugin if they devise another way to implement a multilingual solution for Joomla.

JLanguageMultilang::isEnabled() means that the language filter system plugin is enabled, which would break any 3rd party wanting to devise their own solution without using it.

This is why I propose to use instead a global parameter which we also would use in our core implementation, i.e. for core JLanguageMultilang::isEnabled() would include that new param and 3rd party would ONLY use that new param to display all language-related fields, etc.

avatar brianteeman
brianteeman - comment - 31 May 2017

We can't force any 3rd party to use our system plugin if they devise another way to implement a multilingual solution for Joomla.

If they chose not to use the joomla system then they have to accept any consequences that go with that

JLanguageMultilang::isEnabled() means that the language filter system plugin is enabled, which would break any 3rd party wanting to devise their own solution without using it.

the exact same argument could have been used against using JLanguageAssociations::isEnabled() but it wasnt

This is why I propose to use instead a global parameter which we also would use in our core implementation, i.e. for core JLanguageMultilang::isEnabled() would include that new param and 3rd party would ONLY use that new param to display all language-related fields, etc.

All your arguments above can be used against this as well. you are just proposing moving your issue to a different part of joomla.

To my mind we have a multiligual plugin that we require to be enabled if your site is multilingual and we have an api point to identify that. we should move on and start to using it instead of finding meaningless arguments that say we must reinvent the wheel.

avatar infograf768
infograf768 - comment - 31 May 2017

the exact same argument could have been used against using JLanguageAssociations::isEnabled() but it wasnt

You definitely do not (want to?) understand.
JLanguageAssociations::isEnabled() is totally linked to our core multingual system and JLanguageMultilang::isEnabled().

It may not be used by some existing or to be proposed 3rd party solutions which would prefer not using our core solution.

I am not making an argument. I am not against this RFC, but it should be done imho with a new API which is a Global Config parameter.

Your reply is not necessary.

avatar mbabker mbabker - change - 1 Jun 2017
Labels Added: ?
avatar mbabker mbabker - labeled - 1 Jun 2017
avatar mbabker mbabker - change - 1 Jun 2017
Status New Discussion
avatar brianteeman brianteeman - change - 14 Jun 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-06-14 19:53:54
Closed_By brianteeman
avatar brianteeman brianteeman - close - 14 Jun 2017
avatar brianteeman brianteeman - change - 19 Jun 2017
Status Closed New
Closed_Date 2017-06-14 19:53:54
Closed_By brianteeman
avatar brianteeman brianteeman - reopen - 19 Jun 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Jun 2017
Status New Discussion
avatar brianteeman brianteeman - change - 27 Jul 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-07-27 20:56:25
Closed_By brianteeman
avatar brianteeman brianteeman - close - 27 Jul 2017

Add a Comment

Login with GitHub to post a comment