Problem/Motivation

Replacing the entity type manager service method with dependency injections

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 diff.txt1.08 KBkeshavv
#4 3381889-4.patch1.78 KBkeshavv
#2 Dependency_Injection_issue-3381889-1.patch1.97 KBjigish.addweb

Issue fork conditional_fields-3381889

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

jigish.addweb created an issue. See original summary.

jigish.addweb’s picture

jigish.addweb commented
StatusFileSize
new Dependency_Injection_issue-3381889-1.patch1.97 KB
keshavv’s picture

keshavv commented
Status: Active » Needs review
keshavv’s picture

keshavv commented
StatusFileSize
new 3381889-4.patch1.78 KB
new diff.txt1.08 KB

@jigish.addweb Thanks for the patch. It works perfectly.
But there are some indentation issues. So I also fixed that and updated the patch.

keshavv opened merge request !28

ivnish’s picture

ivnish
he/him
commented
Assigned: jigish.addweb » Unassigned
Status: Needs review » Closed (won't fix)

Already fixed here #3428532: Automated Drupal 11 compatibility fixes for conditional_fields