- πΊπΈUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request β as a guide.
If this is a bug it will require a test case.
- Status changed to Needs review
almost 2 years ago 3:15pm 14 February 2023 - πΊπΈUnited States smustgrave
Think it should be able to be. SHould be able to mockup sending PHP_SAPI
- Status changed to RTBC
almost 2 years ago 7:09pm 2 March 2023 - πΊπΈUnited States smustgrave
Running the command without the fix I get
The user does not have permission to view that workspace.
With the patch it runs fine.
Still wonder if tests could be added but will mark it.
The last submitted patch, 2: 3323380.patch, failed testing. View results β
- Status changed to Fixed
almost 2 years ago 3:58pm 7 March 2023 - π¬π§United Kingdom catch
I think this is an issue where we wouldn't gain anything from tests per π± [policy, no patch] Better scoping for bug fix test coverage RTBC .
Committed/pushed to 10.1.x and cherry-picked back through to 9.5.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.