Posted by xjm

Part of meta issue #500866: [META] remove t() from assert message.

Files: 
CommentFileSizeAuthor
#4 shortcut-1797510-4.patch2.15 KBdcam
PASSED: [[SimpleTest]]: [MySQL] 40,451 pass(es). View
#1 1797510-1-t-shortcut.patch3.07 KBLars Toomre
PASSED: [[SimpleTest]]: [MySQL] 41,908 pass(es). View

Comments

Lars Toomre’s picture

Status: Active » Needs review
FileSize
3.07 KB
PASSED: [[SimpleTest]]: [MySQL] 41,908 pass(es). View

Here is an initial untested patch. It has a format_string() and addresses 4 changes across two test files.

xjm’s picture

Assigned: xjm » jhodgdon
Status: Needs review » Reviewed & tested by the community

Another nice simple one.

jhodgdon’s picture

Version: 8.x-dev » 7.x-dev
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks! Committed to 8.x, ready for 7.x port.

dcam’s picture

Status: Patch (to be ported) » Needs review
FileSize
2.15 KB
PASSED: [[SimpleTest]]: [MySQL] 40,451 pass(es). View

Backported #1 to D7.

dcam’s picture

#4: shortcut-1797510-4.patch queued for re-testing.

dcam’s picture

Issue tags: +Novice

Tagging as Novice.

jhodgdon’s picture

#4: shortcut-1797510-4.patch queued for re-testing.

lazysoundsystem’s picture

Status: Needs review » Reviewed & tested by the community

This too is good. RTBC.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 7.x.

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

Anonymous’s picture

Issue summary: View changes

Removing myself from the author field so I can unfollow. --xjm