Problem/Motivation

In the module settings form is used the t() function when should be used like this $this->t().
Read more about this at: Best practice: $this->t() versus t().

Proposed resolution

Change the t() calls for $this->t() as per new coding standard for translation.
Read more at Translation API overview.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 2872550-2.patch2.07 KBPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
2872550-2.patch2.07 KB

Applying the patch, please review.

  • adriancid committed 7a84a4b on 8.x-1.x authored by Pavan B S 
    Issue #2872550 by Pavan B S: $this->t() should be used instead of t()
adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
Title: $this->t() should be used instead of t() for Drupal 8 version » $this->t() should be used instead of t()
Issue summary: View changes
adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
Title: $this->t() should be used instead of t() » $this->t() should be used instead of t()
Status: Needs review » Fixed

Thanks @Pavan B S, nice catch!

Status: Fixed » Closed (fixed)

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