CommentFileSizeAuthor
#6 3144396-6.patch1.46 KBjaperry
#2 captcha_3144396_1.patch12.06 KBkavitasonawane

Comments

kavitasonawane created an issue. See original summary.

kavitasonawane’s picture

kavitasonawane commented
Status: Active » Needs review
StatusFileSize
new captcha_3144396_1.patch12.06 KB

Status: Needs review » Needs work

The last submitted patch, 2: captcha_3144396_1.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

mrinalini9’s picture

mrinalini9 commented
Assigned: Unassigned » mrinalini9
mrinalini9’s picture

mrinalini9 commented
Assigned: mrinalini9 » Unassigned
japerry’s picture

japerry
Location KOMK
commented
Status: Needs work » Needs review
StatusFileSize
new 3144396-6.patch1.46 KB

t() is meant to be used in .module (procedural code). Also, without the StringTranslationTrait, '$this' wont' work in the service.

  • japerry committed d16214a on 8.x-1.x 
    Issue #3144396: Add StringTranslationTrait to CaptchaService.
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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