FILE: ...eb/modules/contrib/views_condition/src/Plugin/Condition/ViewsCondition.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
204 | WARNING | t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and
| | $this->t() instead
223 | WARNING | t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and
| | $this->t() instead

CommentFileSizeAuthor
#2 3145551-2.patch909 bytesmo_farhaz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mo_farhaz created an issue. See original summary.

mo_farhaz’s picture

mo_farhaz CreditAttribution: mo_farhaz at Specbee commented
Assigned: mo_farhaz » Unassigned
Status: Active » Needs review
FileSize
3145551-2.patch909 bytes

please review.

drupal.ninja03’s picture

drupal.ninja03 CreditAttribution: drupal.ninja03 at TA Digital commented
Issue tags: +Coding standards, +@deprecated
drupal.ninja03’s picture

drupal.ninja03 CreditAttribution: drupal.ninja03 at TA Digital commented
Category: Bug report » Task
drupal.ninja03’s picture

drupal.ninja03 CreditAttribution: drupal.ninja03 at TA Digital commented
Status: Needs review » Reviewed & tested by the community

I have tested the patch and it looks good!

  • pookmish committed 1f8de23 on 8.x-1.x authored by mo_farhaz 
    Issue #3145551 by mo_farhaz: t() calls should be avoided in classes
pookmish’s picture

pookmish CreditAttribution: pookmish commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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