t() calls should be avoided in classes, use dependency injection and $this->t() instead.

CommentFileSizeAuthor
#3 $this->t() erros.png235.38 KBakash.addweb
#2 elevate-image-zoom-2996328-2.patch5.26 KBishani.addweb

Comments

vandna b created an issue. See original summary.

ishani.addweb’s picture

ishani.addweb commented
Status: Active » Needs review
StatusFileSize
new elevate-image-zoom-2996328-2.patch5.26 KB

@vandna b, Please review my attached patch in which I tried to resolve all the possible errors that shown in Pareview report.
Hope my patch helps you.

thanks!..

akash.addweb’s picture

akash.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new $this->t() erros.png235.38 KB

@Ishani.addweb, I checked your patch with Pareview standard issues & your patch clears all the related error of above-mentioned issues.
PFA Which shows that I checked all the mentioned error.

keshavv’s picture

keshavv commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch.

  • keshav.k committed 1018647 on 8.x-1.x authored by Ishani.addweb 
    Issue #2996328 by Ishani.addweb: t() calls should be avoided in classes...
keshavv’s picture

keshavv commented
Status: Fixed » Closed (fixed)