t() calls should be avoided in classes

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

Comments & Activities

Issue created by @Shanu Chouhan
Issue was unassigned.
Status changed to Needs review almost 2 years ago9:03am 15 May 2023
Comment almost 2 years ago →
🇮🇳India Shanu Chouhan
I have Created a patch to fix this issue.
Comment almost 2 years ago →
🇸🇾Syria hodba Dubai
Thanks for stopping by. Actually, your patch will not work in the implementation as the StripeCustomerPortalStripe service does not use the StringTranslationTrait.
I will update the code soon to allow using $this->t() method.
Status changed to Needs work almost 2 years ago4:02pm 16 May 2023
Comment almost 2 years ago →
🇸🇾Syria hodba Dubai
Status changed to Needs review almost 2 years ago1:45pm 22 May 2023
Comment almost 2 years ago →
🇮🇳India ashutosh ahirwal India
Fixed the issue and providing patch
Status changed to RTBC almost 2 years ago2:09pm 13 June 2023
Comment almost 2 years ago →
🇵🇭Philippines kenyoOwen
Hi Ashutosh Ahirwal

I reviewed and applied your patch and confirmed that the phpcs issue mentioned is resolved. Please see the screenshots attached.

Thank you.

t() calls should be avoided in classes

Problem/Motivation

Steps to reproduce

Proposed resolution

Comments & Activities