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

Updated: Comment #0

Problem/Motivation

Docs are missing for properties.
https://drupal.org/node/1354#var

Proposed resolution

Add the docs.

Remaining tasks

User interface changes

No.

API changes

No.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

YesCT’s picture

Assigned: YesCT » Unassigned
Status: Active » Needs review
FileSize
1.59 KB

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.

Proposal:

Flag indicating if this is the only site default language.

tte’s picture

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 https://drupal.org/documentation/git/interdiff | Microbranching workflow: http://xjm.drupalgardens.com/blog/interdiffs-how-make-them-and-why-they-...

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.