Problem/Motivation

This issue is to add documentation for the following source plugins:

  • Language.php
  • d6/LanguageContentSettings.php
  • d6/LanguageContentSettingsTaxonomyVocabulary.php
  • d7/LanguageContentSettings.php
  • d7/LanguageContentSettingsTaxonomyVocabulary.php

Proposed resolution

Do it.

Remaining tasks

Review, commit.

Issue fork drupal-3199999

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Matroskeen created an issue. See original summary.

Matroskeen opened merge request !344

matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
commented
Status: Active » Needs review
quietone’s picture

quietone commented
Status: Needs review » Needs work

Looks good. I like that the comments add the name of the source property where the additional data is stored.

matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
commented
Status: Needs work » Needs review

Thanks! And moving back to NR

quietone’s picture

quietone commented
Status: Needs review » Needs work

Another wee change please.

matroskeen’s picture

matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
commented
Issue summary: View changes
Status: Needs work » Needs review

👍

quietone’s picture

quietone commented
Status: Needs review » Reviewed & tested by the community

I like the last commit message, made me smile. Thanks.

This definitely improves the documentation for the source plugins in the language module. So RTBC.

  • catch committed d8cb0aa on 9.2.x 
    Issue #3199999 by Matroskeen, quietone: Add documentation for language...

  • catch committed c6c50a0 on 9.1.x 
    Issue #3199999 by Matroskeen, quietone: Add documentation for language...
catch’s picture

catch
he/him
Primary language English
commented
Version: 9.2.x-dev » 9.1.x-dev
Status: Reviewed & tested by the community » Fixed

Found one typo in the MR but fixed that on commit.

Matroskeen closed merge request !344

Status: Fixed » Closed (fixed)

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