Page MenuHomePhabricator
Differential D17172

Make differential.querydiffs more liberal about arguments
ClosedPublic
Actions

Authored by epriestley on Jan 10 2017, 9:45 PM.
Tags
None
Referenced Files
F19936680: D17172.diff
Sun, Apr 12, 3:51 PM
F19902899: D17172.diff
Mon, Mar 30, 9:42 PM
F19869038: D17172.id41300.diff
Mar 15 2026, 4:56 AM
F19826999: D17172.id41300.diff
Mar 9 2026, 4:38 AM
F19819252: D17172.id.diff
Mar 6 2026, 4:47 AM
F19814745: D17172.diff
Mar 4 2026, 3:23 PM
F19804699: D17172.id41300.diff
Mar 2 2026, 9:27 AM
F19798632: D17172.id41300.diff
Feb 27 2026, 9:49 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12092: conduit.querydiffs generates error when optional `id` argument is empty
Commits
rP52d563f8b8cf: Make differential.querydiffs more liberal about arguments
Summary

Fixes T12092. D17164 made DiffQuery more strict about arguments using modern conventions, but differential.querydiffs uses bizarre ancient conventions.

Give it more modern conventions instead.

Test Plan

Made a querydiffs call with only revision IDs.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 10 2017, 9:45 PM
chad accepted this revision.Jan 10 2017, 9:45 PM
This revision is now accepted and ready to land.Jan 10 2017, 9:45 PM
Harbormaster completed remote builds in B15186: Diff 41299.Jan 10 2017, 9:46 PM
Closed by commit rP52d563f8b8cf: Make differential.querydiffs more liberal about arguments (authored by epriestley, committed by epriestley). · Explain WhyJan 10 2017, 9:47 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
conduit/
27 lines

Diff 41300

View Options

src/applications/differential/conduit/DifferentialQueryDiffsConduitAPIMethod.php

Loading...
Phabricator · Built by Phacility