Problem/Motivation

Reference Issue: #1810330: Remove EntityTranslationControllerInterface::removeTranslation() comment #15
In \Drupal\Core\Entity\EntityBCDecorator, 30+ functions all have the same block comment:

  /**
   * Forwards the call to the decorated entity.
   */

Proposed resolution

Update the comments to at least include a reference to \Drupal\Core\Entity\EntityNG which is the "decorated entity" that all of the functions are referencing.

Remaining tasks

  • Determine what the block comments should include
  • Create patch that updates the block comments

User interface changes

None

API changes

None

Comments

douglasmiller’s picture

douglasmiller commented
Issue summary: View changes

Added reference to the original issue this was noticed in.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Component: translation_entity.module » entity system
berdir’s picture

berdir
Primary language German
Location Switzerland
commented

The class is a temporary hack until we converted everything to NG and then it will be removed again. I really don't think that we need to worry about this?

This will just unnecessarly conflict with real patches that make changes to this class like the Node NG issue.

swentel’s picture

swentel commented
Status: Active » Closed (won't fix)

And it's also gone now

swentel’s picture

swentel commented
Issue summary: View changes

Updating reference issue to point to the correct issue instead of being self referential