\Drupal calls should be avoided in classes, use dependency injection instead

Hi @urvashi_vora

Thank you very much for your issue and the patch.

I've reviewed it and adjusted it. In my eyes, we cannot replace the \Drupal call in the MosparoWidget because it uses the constructor method from WidgetBase which is not able to accept Dependency Injection (https://git.drupalcode.org/project/mosparo_integration/-/blob/1.0.x/modu...).

I'm open to suggestions if you know how we could solve that. We have the same problem in the other module (https://git.drupalcode.org/project/mosparo_integration/-/blob/1.0.x/modu...) which the CodeSniffer didn't found.

Thank you very much, and have a nice day!

Kind regards,

zepich

I fixed one \Drupal usage. The other uses are not fixable in my view.

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