Problem/Motivation

FILE: src/Form/EntityDeleteConfirmationForm.php
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
-------------------------------------------------------------------------------------------------------------
  46 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
  60 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
  74 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
  92 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 128 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------------------


FILE: src/Form/EntityDeleteForm.php
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
-------------------------------------------------------------------------------------------------------------
 114 | WARNING |  \Drupal calls should be avoided in classes, use dependency injection instead
 137 | WARNING |  \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------------------

Time: 147ms; Memory: 10MB

Steps to reproduce

Run phpcs. There's also other problems, but lets fix them in a separated issue.

Proposed resolution

Fix them all

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 entity_delete_standards.patch7.8 KBj-barnes

Comments

hmendes created an issue. See original summary.

tmaiochi’s picture

Assigned: Unassigned » tmaiochi

I'll work on this.

j-barnes’s picture

StatusFileSize
new7.8 KB
tmaiochi’s picture

Assigned: tmaiochi » Unassigned
Status: Active » Needs review

As @j-barnes was finished the patch I'm removing my assignment and put this to needs review.

baikho’s picture

Version: 8.x-1.x-dev » 2.0.x-dev

  • baikho committed bc30f53 on 2.0.x authored by j-barnes
    Issue #3258058 by j-barnes, hmendes, tmaiochi, baikho: Fix Dependency...
baikho’s picture

Status: Needs review » Fixed

Thanks for your contribution!

Status: Fixed » Closed (fixed)

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