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

Issue fork feeds_ldap-3364587

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

arti_parmar created an issue. See original summary.

uttam made their first commit to this issue’s fork.

uttam opened merge request !5

atuliet05’s picture

atuliet05 commented
Assigned: arti_parmar » atuliet05
atuliet05’s picture

atuliet05 commented

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 .

atuliet05’s picture

atuliet05 commented
Assigned: atuliet05 » Unassigned
Status: Active » Reviewed & tested by the community

a_mitch made their first commit to this issue’s fork.

  • a_mitch committed fc4cd931 on 4.0.x authored by uttam 
    task: #3364587 t() calls should be avoided in classes, use \Drupal\Core\...
a_mitch’s picture

a_mitch commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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