- Issue created by @mstrelan
- Merge request !5563Fix strict type errors: Convert FormattableMarkup to strings (simple replacement) in core Functional tests → (Closed) created by mstrelan
- Status changed to Needs review
12 months ago 11:07pm 27 November 2023 - Status changed to RTBC
12 months ago 2:26pm 28 November 2023 - 🇦🇺Australia mstrelan
Alternative is to cast FormattableMarkup to string before passing to assert* methods, but then using FM in the first place is a little silly. Or there is sprintf but that didn't go down so well.
- Status changed to Needs work
12 months ago 5:01am 1 December 2023 The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to RTBC
12 months ago 5:28am 1 December 2023 - 🇳🇿New Zealand quietone
I'm triaging RTBC issues → . I read the IS and the comments. I didn't find any unanswered questions or other work to do.
I read the MR and all the changes are fine.
The only question I have is how can I check that there are no other "simple conversions"?
Leaving at RTBC.
- 🇦🇺Australia mstrelan
Thanks for the review. For Kernel tests we had an extra issue for any "miscellaneous" fixes. I think once all fixes in the meta are done we can apply the phpcbf rule and see what's leftover.
-
quietone →
committed 061989d4 on 11.x
Issue #3404273 by mstrelan, smustgrave: Fix strict type errors: Convert...
-
quietone →
committed 061989d4 on 11.x
-
quietone →
committed 828b4be3 on 10.2.x
Issue #3404273 by mstrelan, smustgrave: Fix strict type errors: Convert...
-
quietone →
committed 828b4be3 on 10.2.x
- Status changed to Fixed
11 months ago 2:48am 8 January 2024 - 🇳🇿New Zealand quietone
@mstrelan, thanks for the explanation.
Committed to 11.x and 10.2.x because of the testing improvement.
Automatically closed - issue fixed for 2 weeks with no activity.