Skip to content
Snippets Groups Projects

remove t()

1 open thread

Closes #3133726

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
157 154 ],
158 155 ]);
159 156 $this->assertSame(sprintf('1: [user]: name=%s||name=%s||mail=Email field is required.', $username_constraint->illegalMessage, new FormattableMarkup($username_constraint->tooLongMessage, ['%name' => $long_username, '%max' => 60])), $this->messages[0], 'First message should have 3 validation errors.');
160 // phpcs:ignore Drupal.Semantics.FunctionT.NotLiteralString
161 $this->assertSame(sprintf('1: [user]: name=%s||name=%s||mail=Email field is required.', $username_constraint->illegalMessage, $this->t($username_constraint->tooLongMessage, ['%name' => $long_username, '%max' => 60])), $this->messages[0], 'First message should have 3 validation errors.');
  • Shouldn't this be replacing the assertion instead of removing it?

  • Author Maintainer

    It is a duplicate line of the previous line. It was introduced in [#3123067], probably as a result of the many rebases and issues that were touching this block of code for the various coding standards issues.

  • Please register or sign in to reply
  • Please register or sign in to reply
    Loading