On line 159 (shown below), the t() call wants a replacement for key = '%old_vid' and this key is not present in the $args array at that point in the code.

$this->assertNoRaw($link, t('Bogus term (tid 0) is not displayed on node 1 vid %old_vid.', $args));

See http://drupal.org/node/500866#comment-4229084.

CommentFileSizeAuthor
#1 simpletest-upgrade_taxonomy_test-old_vid_arg-1097686-1.patch1.73 KBpillarsdotnet

Comments

pillarsdotnet’s picture

pillarsdotnet commented
Status: Active » Needs review
StatusFileSize
new simpletest-upgrade_taxonomy_test-old_vid_arg-1097686-1.patch1.73 KB

This is a d7-only bug. The offending line does not exist in d8.

pillarsdotnet’s picture

pillarsdotnet commented
Status: Needs review » Reviewed & tested by the community

Note that this may block a 7.x backport of #500866: [META] remove t() from assert message

solotandem’s picture

solotandem commented
Status: Reviewed & tested by the community » Needs review

Patch is to be reviewed and tested by someone else. It's against best practices to RTBC your own patch.

pillarsdotnet’s picture

pillarsdotnet commented

This won't ever get reviewed, tested, and committed while it's at the end of a 350-issue queue.

pillarsdotnet’s picture

pillarsdotnet commented

#1: simpletest-upgrade_taxonomy_test-old_vid_arg-1097686-1.patch queued for re-testing.

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.