CommentFileSizeAuthor
#2 3121925-2.patch2.05 KBswatichouhan012
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swatichouhan012 created an issue. See original summary.

swatichouhan012’s picture

swatichouhan012 CreditAttribution: swatichouhan012 at Valuebound for Valuebound commented
Assigned: swatichouhan012 » Unassigned
Status: Active » Needs review
FileSize
3121925-2.patch2.05 KB

Kindly review patch

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous at Agiledrop - Your Trusted Drupal Teammates commented
Status: Needs review » Reviewed & tested by the community

Looks good,

thanks for your work.

Kind regards, Denis

chr.fritsch’s picture

chr.fritsch
he/him
Primary language German
Location 🇩🇪🇪🇺🌍
CreditAttribution: chr.fritsch at Thunder commented
Status: Reviewed & tested by the community » Fixed

Looks good. Thank you

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Title: \Drupal calls should be avoided in classes, use dependency injection instead » DropzoneJsEbWidget: \Drupal calls should be avoided in classes, use dependency injection instead

Changing title slightly, as there are many other classes needing to fix this exact same problem.

Status: Fixed » Closed (fixed)

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