All \Drupal calls should be removed from classes.

CommentFileSizeAuthor
#6 breadcrumb.PNG10.46 KBchetan 11

Issue fork breadcrumb_extra_field-3406770

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

AstonVictor created an issue. See original summary.

chetan 11 made their first commit to this issue’s fork.

AstonVictor changed the visibility of the branch 3406770-add-dependency-injection to hidden.

AstonVictor opened merge request !6

astonvictor’s picture

astonvictor commented
Assigned: astonvictor » Unassigned
Status: Active » Needs review

Created a new MR.

Added dependency injection to the form class.

chetan 11’s picture

chetan 11 commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new breadcrumb.PNG10.46 KB

Hi,
I have install the module on my local setup & test the raised MR with that.
It's working fine with updated code.
Attaching the screenshot & moving this to +RTBC.
Thanks.

astonvictor’s picture

astonvictor commented
Status: Reviewed & tested by the community » Fixed

thanks for reviewing it.

merged the MR. changes will be added to the next release

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.