Problem/Motivation

In #2942907: Entity system does not provide an API for retrieving an entity variant that is safe for editing, \Drupal\Core\ParamConverter\EntityConverter got a new constructor parameter: \Drupal\Core\Entity\EntityRepositoryInterface $entity_repository.

Since in core alone half a dozen subclasses depended on it, so quite likely many contrib/custom modules would be affected, we put a BC layer in place. The latter behavior was marked as deprecated.

Proposed resolution

In D9, we can make it required.

Remaining tasks

Wait for D9.

User interface changes

None.

API changes

TBD

Data model changes

None.

CommentFileSizeAuthor
#2 2938929-2.patch1.1 KBwim leers

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
StatusFileSize
new 2938929-2.patch1.1 KB
plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Issue summary: View changes

Updated IS

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Title: Make LanguageManager required for EntityConverter » Make the entity repository required for EntityConverter
Issue summary: View changes

Version: 9.x-dev » 9.0.x-dev

The 9.0.x branch will open for development soon, and the placeholder 9.x branch should no longer be used. Only issues that require a new major version should be filed against 9.0.x (for example, removing deprecated code or updating dependency major versions). New developments and disruptive changes that are allowed in a minor version should be filed against 8.9.x, and significant new features will be moved to 9.1.x at committer discretion. For more information see the Allowed changes during the Drupal 8 and 9 release cycles and the Drupal 9.0.0 release plan.

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Postponed » Closed (duplicate)

Looks like this already got fixed in some other issue.