Differential D10875

Differential - allow setting viewPolicy from web ui during diff creation process
ClosedPublic
Actions

Authored by btrahan on Nov 19 2014, 8:00 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 11, 10:53 AM

	Unknown Object (File)
	Mon, Apr 8, 11:39 PM

	Unknown Object (File)
	Mon, Apr 1, 3:46 AM

	Unknown Object (File)
	Feb 14 2024, 9:44 AM

	Unknown Object (File)
	Feb 14 2024, 9:43 AM

	Unknown Object (File)
	Feb 7 2024, 3:28 PM

	Unknown Object (File)
	Feb 7 2024, 3:14 PM

	Unknown Object (File)
	Feb 5 2024, 3:20 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T6237: Use ApplicationTransactions properly in Diff creation
T6152: Show explicit policy controls on web diff creation workflow

Commits

Restricted Diffusion Commit
rP435085862892: Differential - allow setting viewPolicy from web ui during diff creation process

Summary

Fixes T6152, T6237. This introduces a viewPolicy column to the DifferentialDiff, and re-jiggers the DifferentialDiff policy implementation such that things behave as before once associated with a revision, else use the DifferentialDiff policy column value.

Test Plan

made a diff with a non-standard view policy and noted that policy was still selected in the revision step. arc lint.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 26119.Nov 19 2014, 8:00 PM

btrahan added tasks: T6152: Show explicit policy controls on web diff creation workflow, T6237: Use ApplicationTransactions properly in Diff creation.

btrahan retitled this revision from to Differential - allow setting viewPolicy from web ui during diff creation process.

btrahan updated this object.

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

btrahan added a reviewer: epriestley.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 19 2014, 8:00 PM

Harbormaster completed remote builds in B3095: Diff 26119.Nov 19 2014, 8:00 PM

Nice, looks perfect to me.

This revision is now accepted and ready to land.Nov 19 2014, 8:04 PM

Closed by commit rP435085862892: Differential - allow setting viewPolicy from web ui during diff creation process (authored by btrahan, committed by btrahan). · Explain WhyNov 19 2014, 8:16 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20141119.differential.diff.policy.sql

5 lines

src/

applications/

differential/

__tests__/

DifferentialParseRenderTestCase.php

4 lines

conduit/

DifferentialCreateDiffConduitAPIMethod.php

2 lines

DifferentialCreateRawDiffConduitAPIMethod.php

9 lines

controller/

DifferentialDiffCreateController.php

22 lines

DifferentialRevisionEditController.php

7 lines

editor/

DifferentialDiffEditor.php

5 lines

parser/

__tests__/

DifferentialHunkParserTestCase.php

4 lines

query/

DifferentialDiffQuery.php

1 line

storage/

DifferentialDiff.php

39 lines

__tests__/

DifferentialDiffTestCase.php

5 lines

diffusion/

controller/

DiffusionChangeController.php

4 lines

DiffusionCommitController.php

1 line

DiffusionDiffController.php

4 lines

data/

DiffusionPathChange.php

6 lines

engine/

DiffusionCommitHookEngine.php

4 lines

herald/

adapter/

HeraldCommitAdapter.php

4 lines

repository/

worker/

commitmessageparser/

PhabricatorRepositoryCommitMessageParserWorker.php

2 lines

infrastructure/

diff/

PhabricatorDifferenceEngine.php

4 lines

Diff 26120

resources/sql/autopatches/20141119.differential.diff.policy.sql

Loading...

src/applications/differential/tests/DifferentialParseRenderTestCase.php

Loading...

src/applications/differential/conduit/DifferentialCreateDiffConduitAPIMethod.php

Loading...

src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php

Loading...

src/applications/differential/controller/DifferentialDiffCreateController.php

Loading...

src/applications/differential/controller/DifferentialRevisionEditController.php

Loading...

src/applications/differential/editor/DifferentialDiffEditor.php

Loading...

src/applications/differential/parser/tests/DifferentialHunkParserTestCase.php

Loading...

src/applications/differential/query/DifferentialDiffQuery.php

Loading...

src/applications/differential/storage/DifferentialDiff.php

Loading...

src/applications/differential/storage/tests/DifferentialDiffTestCase.php

Loading...

src/applications/diffusion/controller/DiffusionChangeController.php

Loading...

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...

src/applications/diffusion/controller/DiffusionDiffController.php

Loading...

src/applications/diffusion/data/DiffusionPathChange.php

Loading...

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Loading...

src/applications/herald/adapter/HeraldCommitAdapter.php

Loading...

src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php

Loading...

src/infrastructure/diff/PhabricatorDifferenceEngine.php

Loading...