Page MenuHomePhabricator

Don't create invalid build steps while adding them
ClosedPublic

Authored by epriestley on Mar 23 2014, 9:22 PM.
Tags
None
Referenced Files
F19303309: D8605.id20435.diff
Tue, Dec 23, 11:04 PM
F19251668: D8605.id20435.diff
Mon, Dec 22, 9:08 AM
F19065891: D8605.id20395.diff
Nov 30 2025, 3:16 AM
F18991625: D8605.diff
Nov 18 2025, 11:05 PM
F18855602: D8605.id.diff
Nov 1 2025, 5:53 AM
F18853925: D8605.diff
Oct 31 2025, 8:39 PM
F18833201: D8605.id.diff
Oct 26 2025, 1:34 AM
F18736415: D8605.id.diff
Oct 1 2025, 5:50 AM
Subscribers

Details

Summary

Ref T1049. Currently, the "add" dialog lets you select a build step type, but then immediately creates one. If you "cancel" from the edit screen, you end up with an empty (and almost certainly invalid) build step.

Instead, don't create the step until it's valid.

Test Plan

Add Step -> Pick Type -> Add Step -> Cancel no longer creates empty step.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Don't create invalid build steps while adding them.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 25 2014, 8:52 PM
epriestley updated this revision to Diff 20435.

Closed by commit rP6e3c17e6f9fa (authored by @epriestley).