- ๐ฏ๐ตJapan tyler36 Osaka
Patch
3270638-2-fail.patch
contains the test, which fails as expected.
Patch3270638-2.patch
applies without issue to10.0.8
and passes test. - Status changed to RTBC
over 1 year ago 3:06am 1 June 2023 - Status changed to Needs work
over 1 year ago 9:51pm 5 July 2023 - ๐ฌ๐งUnited Kingdom longwave UK
#3 doesn't apply cleanly to 11.x. Also testbot never ran on either of the patches for some reason.
- last update
over 1 year ago Custom Commands Failed - ๐ง๐ชBelgium ducktape
Since 10.1 the createUser has a deprecation warning on the parameters, so I updated these in the original.
The patch was not being tested, because phpstan blocked it. Due to the patch, drupalSetUpCurrentUser() does not exist any more. This was still used in BundleClassTest.
- last update
12 months ago 29,722 pass - Status changed to Needs review
11 months ago 12:49pm 21 March 2024 - Status changed to Needs work
11 months ago 1:05pm 21 March 2024 The Needs Review Queue Bot โ tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request โ . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ made their first commit to this issueโs fork.
- Status changed to Needs review
9 months ago 8:46am 16 May 2024 - ๐บ๐ธUnited States smustgrave
MR appears to be the patch that @larowlan uploaded here https://www.drupal.org/files/issues/2019-04-16/3039955.patch โ just fyi for reviewers.
- Status changed to Needs work
9 months ago 2:03pm 23 May 2024 - ๐บ๐ธUnited States smustgrave
Put the permission/values in the right order (I guess).
Not sure this matches what's being done.