Page MenuHomePhorge
Differential D25444

Fix regression in DiffusionDiffQueryConduitAPIMethod
ClosedPublic
Actions

Authored by valerio.bozzolan on Oct 2 2023, 14:51.
Tags
None
Referenced Files
F3341548: D25444.1743653995.diff
Wed, Apr 2, 04:19
F3337964: D25444.1743596776.diff
Tue, Apr 1, 12:26
F3335545: D25444.1743564231.diff
Tue, Apr 1, 03:23
F3333537: D25444.1743536618.diff
Mon, Mar 31, 19:43
F3333353: D25444.1743534725.diff
Mon, Mar 31, 19:12
F3330455: D25444.1743489566.diff
Mon, Mar 31, 06:39
F3327065: D25444.1743431927.diff
Sun, Mar 30, 14:38
F3322755: D25444.1743376891.diff
Sat, Mar 29, 23:21
View All 40 Files
Subscribers
Cigaryno
Matthew
speck
tobiaswiese

Details

Reviewers
avivey
speck
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15649: Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected
Commits
rPf727f17bc24a: Fix regression in DiffusionDiffQueryConduitAPIMethod
Summary

Fix the following error you may encounter in production:

Too few arguments to function DiffusionDiffQueryConduitAPIMethod::getDefaultParser(), 0 passed in /var/www/phorge/src/applications/diffusion/conduit/DiffusionDiffQueryConduitAPIMethod.php on line 156 and exactly 1 expected

It was caused by the lack of a new mandatory parameter in a method.

Closes T15649

Test Plan

Check with your big eyes that a ConduitAPIRequest is passed to getDefaultParser().

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSizePackages
src/
applications/
diffusion/
conduit/
2 linesO1: Blessed Committers

Diff 1409

View Options

src/applications/diffusion/conduit/DiffusionDiffQueryConduitAPIMethod.php

Loading...
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0