Skip to content

Commit 6081370

Browse files
committed
Differential - fix "transaction has no effect error" on diff create
Summary: Fixes T6658. Test Plan: made a diff with no repository and default policy and it worked! Reviewers: epriestley, chad Reviewed By: chad Subscribers: Korvin, epriestley Maniphest Tasks: T6658 Differential Revision: https://secure.phabricator.com/D10910
1 parent 48e90ca commit 6081370

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

src/applications/differential/conduit/DifferentialCreateDiffConduitAPIMethod.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,7 @@ protected function execute(ConduitAPIRequest $request) {
172172
id(new DifferentialDiffEditor())
173173
->setActor($viewer)
174174
->setContentSourceFromConduitRequest($request)
175+
->setContinueOnNoEffect(true)
175176
->applyTransactions($diff, $xactions);
176177

177178
$path = '/differential/diff/'.$diff->getID().'/';

src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ protected function execute(ConduitAPIRequest $request) {
6868
id(new DifferentialDiffEditor())
6969
->setActor($viewer)
7070
->setContentSourceFromConduitRequest($request)
71+
->setContinueOnNoEffect(true)
7172
->setLookupRepository(false) // respect user choice
7273
->applyTransactions($diff, $xactions);
7374

0 commit comments

Comments
 (0)