Changed translation function on $this->t()

CommentFileSizeAuthor
#5 Changed_translation-2875658-5.patch7.59 KBMunavijayalakshmi
#2 Changed_translation-2875658-2.patch7.68 KBMunavijayalakshmi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
Changed_translation-2875658-2.patch7.68 KB
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Needs work

Hi @Munavijayalakshmi

This issue will create conflicts with https://www.drupal.org/node/2875650

Please create a unified patch or make changes according to short array syntax issue.

Thank you.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Unassigned » Munavijayalakshmi
Issue tags: +Needs reroll
Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Needs work » Needs review
FileSize
Changed_translation-2875658-5.patch7.59 KB
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community

Now patch applied cleanly. Replaced all the t() functions in .php files are replaced with $this->t() functions.

Good work!

micropat’s picture

micropat CreditAttribution: micropat commented
Title: $this->t() should be used instead of t() for Drupal 8 version. » Use $this->t() instead of t() for object-oriented code
Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs reroll

Fixed in dev, thanks!

Status: Fixed » Closed (fixed)

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