t() calls should be avoided in classes, use DI and $this->t() instead.

FILE: ...l/pareviewsh/pareview_temp/src/Form/LoginDestinationSettingsForm.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------
 35 | WARNING | t() calls should be avoided in classes, use dependency
    |         | injection and $this->t() instead
 36 | WARNING | t() calls should be avoided in classes, use dependency
    |         | injection and $this->t() instead
 41 | WARNING | t() calls should be avoided in classes, use dependency
    |         | injection and $this->t() instead
 42 | WARNING | t() calls should be avoided in classes, use dependency
    |         | injection and $this->t() instead
--------------------------------------------------------------------------
FILE: ...rupal/pareviewsh/pareview_temp/src/Form/LoginDestinationRuleForm.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
--------------------------------------------------------------------------
 167 | WARNING | Unused variable $destination.
 219 | WARNING | t() calls should be avoided in classes, use dependency
     |         | injection and $this->t() instead
--------------------------------------------------------------------------
CommentFileSizeAuthor
#2 3071361-2.patch3.58 KBvuil

Comments

ilchovuchkov created an issue. See original summary.

vuil’s picture

StatusFileSize
new3.58 KB
vuil’s picture

Status: Active » Needs review
vuil’s picture

Title: t() calls should be avoided in classes, use DI and $this->t() instead » Deprecated t() calls need to be updated

Please use Closed (won't fix) instead of non-activity about the issue.

vuil’s picture

Title: Deprecated t() calls need to be updated » Deprecated t() calls need to be replaced with DI & StringTranslationTrait

Update the title only. I think that it is more corrected now.

  • rsvelko committed c52d0ed on 8.x-1.x authored by vuil
    Issue #3071361 by vuil: Deprecated t() calls need to be replaced with DI...
rsvelko’s picture

Status: Needs review » Fixed
Related issues: +#3106146: Plan for next Login Destination 8.x-1.x Release
vuil’s picture

Status: Fixed » Closed (fixed)

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