Page MenuHomePhorge
Differential D25444

Fix regression in DiffusionDiffQueryConduitAPIMethod
ClosedPublic
Actions

Authored by valerio.bozzolan on Oct 2 2023, 14:51.
Tags
None
Referenced Files
F3529778: D25444.1744866109.diff
Wed, Apr 16, 05:01
F3496680: D25444.1744790460.diff
Tue, Apr 15, 08:01
F3393627: D25444.1744499009.diff
Fri, Apr 11, 23:03
F3372449: D25444.1744249883.diff
Wed, Apr 9, 01:51
F3369212: D25444.1744213579.diff
Tue, Apr 8, 15:46
F3368403: D25444.1744208757.diff
Tue, Apr 8, 14:25
F3363337: D25444.1744122982.diff
Mon, Apr 7, 14:36
F3362586: D25444.1744087144.diff
Mon, Apr 7, 04:39
View All 49 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