FILE: \core\modules\views\src\DisplayPluginCollection.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------
83 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------------------------------------------------

FILE: core\modules\views\src\Entity\View.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------
161 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------

CommentFileSizeAuthor
#3 3116099-3.patch1.19 KBneelam_wadhwani

Comments

neelam_wadhwani created an issue. See original summary.

neelam_wadhwani’s picture

neelam_wadhwani commented
Assigned: Unassigned » neelam_wadhwani
Status: Needs review » Needs work
neelam_wadhwani’s picture

neelam_wadhwani commented
StatusFileSize
new 3116099-3.patch1.19 KB

Please review patch.

nishantghetiya’s picture

nishantghetiya commented
Status: Needs work » Reviewed & tested by the community

@neelam_wadhwani Hello,
Above provided patch looks good for me. Thanks for your contribution.

cilefen’s picture

cilefen commented
Status: Reviewed & tested by the community » Needs work
Parent issue: » #3113904: [META] Replace t() calls inside of classes

Well, tests have failed. See also #3116169: Some minor bugs for drupal core file module, we don't always fix issues like this one module-by-module.

I have attached the parent issue.

Version: 8.9.x-dev » 9.1.x-dev

Drupal 8.9.0-beta1 was released on March 20, 2020. 8.9.x is the final, long-term support (LTS) minor release of Drupal 8, which means new developments and disruptive changes should now be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs work » Closed (duplicate)

I am closing this as a duplicate of #3113904: [META] Replace t() calls inside of classes . It is not correct to scope these issues module-by-module and other work is going on in that issue to separate out the tasks in a more manageable way.