Paths

Table of Contentst

Differential D18845

Modularize transactions for Drydock Blueprints
ClosedPublic
Actions

Authored by epriestley on Dec 26 2017, 5:44 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Feb 3, 10:14 PM

	Unknown Object (File)
	Mon, Jan 27, 10:14 AM

	Unknown Object (File)
	Fri, Jan 24, 10:57 PM

	Unknown Object (File)
	Fri, Jan 24, 10:57 PM

	Unknown Object (File)
	Fri, Jan 24, 10:57 PM

	Unknown Object (File)
	Sat, Jan 18, 9:08 AM

	Unknown Object (File)
	Fri, Jan 17, 2:48 PM

	Unknown Object (File)
	Fri, Jan 17, 1:24 AM

Subscribers

Restricted Owners Package

Details

Reviewers

Commits

rP83c528c46435: Modularize transactions for Drydock Blueprints

Summary

Ref PHI243. This is a followup to D18822, which added an edit-only drydock.blueprint.edit. By modularizing transactions (here) and then adding a "type" transaction (next change) I intend to remove the "edit-only" limitation and make this API method fully functional.

Test Plan

Created and edited blueprints via the web UI. Edited blueprints via the API. Disabled/enabled blueprints (currently web UI only).

Diff Detail

Repository

Branch

dock1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 18990
Build 25614: Run Core Tests
Build 25613: arc lint + arc unit

Event Timeline

epriestley created this revision.Dec 26 2017, 5:44 PM

Owners added a subscriber: Restricted Owners Package.Dec 26 2017, 5:44 PM

Harbormaster completed remote builds in B18990: Diff 45211.Dec 26 2017, 5:45 PM

epriestley requested review of this revision.Dec 26 2017, 5:45 PM

epriestley added a child revision: D18847: Give EditEngine a Conduit-specific initialization pathway for objects.Dec 26 2017, 5:53 PM

amckinley accepted this revision.Dec 26 2017, 11:47 PM

This revision is now accepted and ready to land.Dec 26 2017, 11:47 PM

Closed by commit rP83c528c46435: Modularize transactions for Drydock Blueprints (authored by epriestley). · Explain WhyJan 4 2018, 6:03 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

8 lines

applications/

drydock/

controller/

DrydockBlueprintDisableController.php

3 lines

editor/

DrydockBlueprintEditEngine.php

2 lines

DrydockBlueprintEditor.php

100 lines

storage/

DrydockBlueprintTransaction.php

35 lines

xaction/

DrydockBlueprintDisableTransaction.php

48 lines

DrydockBlueprintNameTransaction.php

60 lines

DrydockBlueprintTransactionType.php

4 lines

Diff 45211

src/__phutil_library_map__.php

Loading...

src/applications/drydock/controller/DrydockBlueprintDisableController.php

Loading...

src/applications/drydock/editor/DrydockBlueprintEditEngine.php

Loading...

src/applications/drydock/editor/DrydockBlueprintEditor.php

Loading...

src/applications/drydock/storage/DrydockBlueprintTransaction.php

Loading...

src/applications/drydock/xaction/DrydockBlueprintDisableTransaction.php

Loading...

src/applications/drydock/xaction/DrydockBlueprintNameTransaction.php

Loading...

src/applications/drydock/xaction/DrydockBlueprintTransactionType.php

Loading...