Replace t() calls inside of Controllers that do not use StringTranslationTrait

Replaced all the t() with $this->t()
Please review.

Also, the issue summary needs to be updated, since the issue has been rescoped.

I've been doing some more digging into the issues for fixing t() function. In this comment 📌 [META] Replace t() calls inside of classes Active in the parent issue xjm outlines the scope for these issues. That means, that this issue needs to split into two issues, one for the Controllers that have the trait and one for the Controllers that do not have the trait.

There is also a sniff for this. To find the find that need to be changed I altered the DrupalPractice.Objects.GlobalFunction rule in phpcs.xml to the following.

  <rule ref="DrupalPractice.Objects.GlobalFunction">
    <include-pattern>*/Plugin/*</include-pattern>
    <include-pattern>*Controller*</include-pattern>
    <exclude-pattern>*/tests/*</exclude-pattern>
  </rule>

The results show errors in 8 files, 5 of which extend from ControllerBase, which has the trait. Lets do those here. I have updated the IS with the files to be fixed here.

The ones that do not have the trait are

• core/modules/system/src/Controller/SystemInfoController.php
• core/modules/file/src/Controller/FileWidgetAjaxController.php
• core/modules/config/src/Controller/ConfigController.php

And are to be done in a separate issue.

Hiding patches for clarity.

Updated the summary to note that 2 of the findings aren't needed since those moved to contrib.

Reviewing the current code changes in the MR and change seems good

One comment on the MR.

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Seems straight forward to me and non disruptive

nod_ → closed merge request !8575

Committed and pushed d2cf8e5268 to 11.x and ceeb8e0277 to 11.0.x and c550b71cd1 to 10.4.x and 0ce937d2f6 to 10.3.x. Thanks!

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