Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment #0.0
douglasmiller CreditAttribution: douglasmiller commentedAdded reference to the original issue this was noticed in.
Comment #1
plachComment #2
BerdirThe 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.
Comment #3
swentel CreditAttribution: swentel commentedAnd it's also gone now
Comment #3.0
swentel CreditAttribution: swentel commentedUpdating reference issue to point to the correct issue instead of being self referential