dt() is used in Drush in place of t() and does not support context.

CommentFileSizeAuthor
#5 2713373-5.patch1.55 KBherom
#5 interdiff-2713373-2-5.txt1.01 KBherom
#2 2713373.patch1.62 KBGábor Hojtsy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy created an issue. See original summary.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Issue summary: View changes
FileSize
2713373.patch1.62 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2713373.patch, failed testing.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

Hm, that should have theoretically worked. Not sure why it fails :/ @herom, do you have any idea? :)

herom’s picture

herom CreditAttribution: herom as a volunteer commented
Status: Needs work » Needs review
FileSize
interdiff-2713373-2-5.txt1.01 KB
2713373-5.patch1.55 KB

The test was parsing for install strings only, a few lines before the new assert:

$this->parseFile($filename, POTX_API_7, POTX_STRING_INSTALLER);
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Needs review » Reviewed & tested by the community

Eh, huh, duh. Silly me :D

  • herom committed bc3481b on 7.x-3.x 
    Issue #2713373 by Gábor Hojtsy, herom: Add support for drush dt
herom’s picture

herom CreditAttribution: herom as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks Gábor.

Status: Fixed » Closed (fixed)

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