content_translation_entity_info_alter() docs need rewrite

Created on 2 May 2013, over 11 years ago
Updated 17 July 2024, 5 months ago

Problem/Motivation

Quoting @jhodgdon from #1968970-13: Standardize module-provided entity info documentation and clean-up the @EntityType annotation [wrt to the docblock of EntityTranslationControllerInterface]:

- How about a @defgroup so it shows up as a Topic?

- This @file doc starts out "The entity translation UI relies on the entity info to provide its features." and then doesn't explain what "entity info" is or how to define it (a hook? plugin annotation?).

- Then the next paragraph starts out telling me that some keys are required but not all, and then gets really really confusing so that I can't even follow it. I think it would be better to just make a list of the keys (using our standard list formatting to indicate (optional) for optional keys and explaining for each key why/when it is really optional).

- The rest of the documentation in the @file just seems like a ramble... it really needs to be organized.

So... Can the documentation be rewritten so that it
a) Is in a @defgroup not @file, with relevant classes/hooks/whatever using @ingroup so they reference each other?
b) Actually tells a developer what they need to do to define a translatable entity class.
c) Is in a logical order.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

🐛 Bug report
Status

Needs work

Version

11.0 🔥

Component
Documentation 

Last updated 1 day ago

No maintainer
Created by

🇮🇹Italy plach Venezia

Live updates comments and jobs are added and updated live.
  • D8MI

    (Drupal 8 Multilingual Initiative) is the tag used by the multilingual initiative to mark core issues (and some contributed module issues). For versions other than Drupal 8, use the i18n (Internationalization) tag on issues which involve or affect multilingual / multinational support. That is preferred over Translation.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • Merge request !8760improve hook doc → (Open) created by quietone
  • Status changed to Needs review 5 months ago
  • 🇳🇿New Zealand quietone

    Lets see if we can complete this.

  • Status changed to Needs work 5 months ago
  • 🇺🇸United States smustgrave

    So looking at the quote of the summary talks about adding @defgroup but that's not the case. So is the goal just to rewrite it now?

Production build 0.71.5 2024