Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Small follow-up to #2005166: Create simple file listing under admin/content/file related to meta #500866: [META] remove t() from assert message.
A couple of these snuck in with this recent commit, let's remove them :)
The offending assertions are in \Drupal\file\Tests\FileListingTest.
Comment | File | Size | Author |
---|---|---|---|
#4 | 2059699-remove-t-asserts.patch | 1.37 KB | tim-e |
#1 | 205699-remove-t-assertion-filelisting.patch | 1.92 KB | tim-e |
Comments
Comment #1
tim-e CreditAttribution: tim-e commentedPatch
Comment #2
tim-e CreditAttribution: tim-e commentedChanging status
Comment #3
star-szrThanks for working on this @tim-e! Good start, needs to be dialed back a bit though. The top section of #500866: [META] remove t() from assert message gives a rough guide to what we are changing.
Only assertion messages should have t() removed, so $this->assert____ calls. Both of these lines need to be rolled back.
This one changes too much - the 'Temporary' string should still be wrapped in t().
This one looks great because only the assertion message (last parameter) is changing.
Comment #4
tim-e CreditAttribution: tim-e commentedOk, I think Ive got it now. Only remove t() from assert message param.
Here's another pass at it.
Comment #5
larowlanLooking good
Comment #6
star-szrAgreed :)
Comment #7
webchickThanks for catching these!
Committed and pushed to 8.x.