Problem/motivation

The "multiple" special language was added to core earlier in Drupal 8 with the thinking of completeness (see #965300: Change LANGUAGE_NONE to LANGUAGE_NOT_SPECIFIED; add LANGUAGE_NOT_APPLICABLE and LANGUAGE_MULTIPLE), so you can express if something is in a certain language, no language, an unknown language or in multiple languages. The later is however confused by people who we tested with unrelated things. For example, a possible capability to have entities with fields that have different language codes or even that this language code would designate a translatable/translated entity and then individual translations would use specific languages. See http://groups.drupal.org/node/271918 for more details.

@webchick was also confused with the "multiple" language when she committed the original patch and we agreed user testing would figure out if people get it or not. They don't :)

Proposal

This special language was added only for completeness and has no special significance in the language system. It is very easy to remove and is equally easy for contrib to add it back if need be. The Drupal 8 language system allows contribs to add any number of special languages like this which would then behave like other human-created languages.

In short removing this from core does not actually remove functionality that was present in Drupal 7 (core or contrib) or that is not easily possible to add back in Drupal 8 contrib.

OneLess.png

Related issues

The usability testing also showed that showing the special languages at all in this table is confusing. The removal of them is discussed in #1869328: Restore simplicity of language list.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

miro_dietiker’s picture

We all know, multilingual systems are very complex.
I don't feel like "mul" should be a user added language. Instead it has a technical meaning and this concept is what we introduce or not.

If you want to simplify the UI, there's always the option to hide things that are actually there. Why not hide the three common records in the UI in regular mode and add a checkbox to show advanced configuration?

miro_dietiker’s picture

Issue summary: View changes

Add reference to original issue where this was added.

Gábor Hojtsy’s picture

@miro_dietiker: well, our user testing showed people did not understand what "multiple" was for. In fact they hugely misunderstood it. Contribs can add "mul" as well as any other languages as needed, if/once languages become config entities, all it would take is to ship a .yml file with a couple lines of config with your module.

As for removing the special languages from the admin UI, see #1869328: Restore simplicity of language list.

As for establishing an entities to sets of languages related to limit what shows up on the UI: #1314250: Allow filtering/configuration of which languages apply to what (UI, nodes, files, etc) (where I did not have any success to (a) make people enthusiastic about the idea (b) entice anybody to work on it - you are more than welcome :).

Bojhan’s picture

This sounds like a solid choice, if we do not feel core needs this and it causes confusion than its much better if contrib adds it as it sees fit.

sun’s picture

Status: Needs review » Reviewed & tested by the community

This looks RTBC to me. And if this is really not covered by any tests, then it's even better to remove it. ;)

So let's do this, if the bot comes back green. Otherwise, we'll have to remove corresponding test coverage, too, I guess. (or move the definition into language_test.module - as an actual proof that contrib is able to re-introduce it?)

Status: Reviewed & tested by the community » Needs work

The last submitted patch, remove-multiple.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
3.19 KB

So it did have test coverage :) Also the same test coverage already tests adding other locked languages, so we are covered there. This passes tests for me.

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Should be back to RTBC as per above.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

I agree that this is a UX improvement. Committed to 8.x. Thanks!

Gábor Hojtsy’s picture

Issue tags: -sprint

Superb, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

YesCT’s picture

YesCT’s picture

Issue tags: +budapest2012

tagging

YesCT’s picture

Issue summary: View changes

Add special language removal issue link.