Closed (fixed)
Project:
Drupal core
Version:
10.0.x-dev
Component:
other
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
8 Nov 2022 at 11:04 UTC
Updated:
5 Dec 2022 at 21:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
alexpottComment #3
mondrakeGood for 10.1.x, will need porting to 10.0
Comment #4
mondrakeComment #5
alexpottWidening the scope slightly to incorporate the other trait that causes issues like this. Crediting @mondrake since this is similar to the work in #3319582: Fix calls to methods with too many parameters passed in
And have patches for both 10.0.x and 10.1.x
Comment #6
alexpottNow that we have PHPStan level 1 any calls to assertMailString that have 4 arguments will produce an error - so we don't need to trigger a deprecation here.
As above.
Comment #7
alexpottThis patch shrinks core/phpstan-baseline.neon from 199K to 150K... not bad.
Comment #8
mallezieBoth patches look good to me. (Awaiting tests offcourse).
Both code and comments updated, Baseline removals are all in scope, nothing unexpected here. So RTBC.
Changes are small (and same on both patches). And they clean up a nice chunk in the baseline.
Comment #9
mallezieDon't we need the deprecation for contrib / custom testing using this trait?
Comment #10
mallezieSince PHP allows you to pass in as many extra params as you like this will not break any other contrib / custom code using this parameter. (Thanks @alexpott for explaining).
Solves my remark.
Comment #12
catchCommitted/pushed the respective patches to 10.1.x and 10.0.x, thanks!
Comment #13
mondrakeOpened #3322986: Missed removing a $group parameter from AssertMailTrait to fix a straggler.