t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead

Open on Drupal.org →

Created on 3 June 2023, over 1 year ago

Updated 4 April 2024, 8 months ago

phpcs --standard=DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml feeds_ldap/

FILE: C:\xampp\htdocs\drupal_9\web\modules\contrib\feeds_ldap\src\Feeds\Fetcher\QueryFetcher.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
50 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
62 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------

Time: 122ms; Memory: 6MB

📌 Task

Status

Version

4.0

Component

Code

Created by

🇮🇳India arti_parmar

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!5 t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
Open
uttam
updated 8 months ago

Comments & Activities

Issue created by @arti_parmar
First commit to issue fork.
Merge request !53364587: Make use of $this->t(). → (Open) created by uttam
Assigned to atuliet05
Comment 8 months ago →
🇮🇳India atuliet05
Comment 8 months ago →
🇮🇳India atuliet05
Steps to solve the issue.

1. We install the module in my local machine using git clone.

2. We setup phpcs in my local machine.

3. After that we test all the file inside module(feeds_ladap) using phpcs. lando phpcs --standard=DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml ./web/modules/contrib/feeds_ldap/ and we got 2 warnings related to t() function.

4. Then we download and apply patch #3 ( https://git.drupalcode.org/project/feeds_ldap/-/merge_requests/5.patch )

5. Again we run phpcs command as above.

6. Now we not got those warnings .
Issue was unassigned.
Status changed to RTBC 8 months ago7:37am 4 April 2024
Comment 8 months ago →
🇮🇳India atuliet05

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024