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
F18855602: D8605.id.diff
Sat, Nov 1, 5:53 AM
F18853925: D8605.diff
Fri, Oct 31, 8:39 PM
F18833201: D8605.id.diff
Sun, Oct 26, 1:34 AM
F18736415: D8605.id.diff
Oct 1 2025, 5:50 AM
F18733022: D8605.diff
Sep 30 2025, 8:59 PM
F18698533: D8605.id20395.diff
Sep 27 2025, 11:29 AM
F18621500: D8605.diff
Sep 15 2025, 8:32 AM
F18621291: D8605.diff
Sep 15 2025, 7:40 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).