Follow up for #1641166: Undefined property: Drupal\Core\Language\Language::$locked in language_list()

Updated: Comment #0


Docs are missing for properties.

Proposed resolution

Add the docs.

Remaining tasks

User interface changes


API changes


#3 drupal8.language-system.2035007-3.patch1.59 KBspeely
PASSED: [[SimpleTest]]: [MySQL] 56,938 pass(es). View
#1 drupal8.language-system.2035007-1.patch1.59 KBYesCT
PASSED: [[SimpleTest]]: [MySQL] 56,905 pass(es). View


YesCT’s picture

Assigned: YesCT » Unassigned
Status: Active » Needs review
1.59 KB
PASSED: [[SimpleTest]]: [MySQL] 56,905 pass(es). View

A first pass.

Needs review to see if the descriptions are actually accurate for what the properties are storing and why.

penyaskito’s picture

Looks good to me, but default could be clearer.

+++ b/core/lib/Drupal/Core/Language/Language.phpundefined
@@ -17,13 +17,64 @@
+   * Site default language, there is only one site default.


Flag indicating if this is the only site default language.

speely’s picture

1.59 KB
PASSED: [[SimpleTest]]: [MySQL] 56,938 pass(es). View

Looks good to me, too - as well as penyaskito's proposal.

Attached is a patch which combines both proposals.

YesCT’s picture

@speely interdiffs are really cool.

Want to try making one?

instructions on creating an interdiff, see | Microbranching workflow:

andypost’s picture

Looks good to go

parthipanramesh’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Good work!

YesCT’s picture

still applies.

jhodgdon’s picture

Component: language system » documentation

Changing component so I notice it next time I am doing Core commits.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks all! Committed to 8.x.

Status: Fixed » Closed (fixed)

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