Differential D16415

Avoid double [y/N] when updating a not owned revision
ClosedPublic
Actions

Authored by Luke081515.2 on Aug 17 2016, 4:55 PM.

Tags

Referenced Files

	F15471179: D16415.id39476.diff
	Sat, Apr 5, 3:58 AM

	F15469205: D16415.diff
	Fri, Apr 4, 10:35 AM

	F15440538: D16415.id39476.diff
	Wed, Mar 26, 1:17 PM

	F15438683: D16415.id39474.diff
	Wed, Mar 26, 2:39 AM

	F15437061: D16415.id.diff
	Tue, Mar 25, 5:17 PM

	F15431578: D16415.diff
	Mon, Mar 24, 1:15 PM

	F15430247: D16415.id39476.diff
	Mon, Mar 24, 6:44 AM

	F15429739: D16415.id39474.diff
	Mon, Mar 24, 4:01 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T11489: Arcanist: Duplicate at question

Commits

rARC3ae0cc8d41e2: Avoid double [y/N] when updating a not owned revision

Required Signatures

L28 Phacility Individual Contributor License Agreement

Summary

Since 737f5c0df976fe2b3178aac6ab7feb3d3621d99e arcanist shows [y/N] two times, when arcanist asks you, if you want to proceed, if you want to update a not owned revision. Ths patch fixes
that, so that Arcanist shows [y/N] only once, like at other situations, when Arcanist asks you a question.
Ref T11489

Test Plan

Updated a not owned revision.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Luke081515.2 updated this revision to Diff 39474.Aug 17 2016, 4:55 PM

Luke081515.2 retitled this revision from to Avoid double [y/N] when updating a not owned revision.

Luke081515.2 updated this object.

Luke081515.2 edited the test plan for this revision. (Show Details)

Luke081515.2 added a project: Arcanist.

Luke081515.2 added a task: T11489: Arcanist: Duplicate at question.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 17 2016, 4:55 PM

Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript

Herald added a subscriber: epriestley. · View Herald Transcript

I hope this is ok, since this is my first patch actually...

Thanks!

This revision is now accepted and ready to land.Aug 17 2016, 5:36 PM

Closed by commit rARC3ae0cc8d41e2: Avoid double [y/N] when updating a not owned revision (authored by • Luke081515, committed by epriestley). · Explain WhyAug 17 2016, 5:36 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T11489: Arcanist: Duplicate at question.Aug 17 2016, 5:36 PM

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistDiffWorkflow.php

2 lines

Diff 39476

src/workflow/ArcanistDiffWorkflow.php

Loading...