Page MenuHomePhabricator
Create Task
Maniphest T11910

Cannot unset Build Plan Step's "Wait For Message" option
Closed, ResolvedPublic
Actions

Description

When creating or modifying a Build Plan Step, saving the Step with When Complete set to Wait For Message, this setting will persist regardless of changing it back to Continue Build Normally.

Steps to Reproduce:

  1. Open a new Build Plan Step, or edit an existing one.
  2. Set When Complete to Wait for Message.
  3. Save the Build Plan Step.
  4. Edit the Step again, and set When Complete to Continue Build Normally.
  5. Save the Step.
  6. Observe that When Complete still shows as Wait for Message.
Version Information
phabricator a96f9491bbb98f9d7f9fa343cd33d7d78e79933a (Wed, Nov 16) 
arcanist 8de281a32d5a1dd1c3dad506150eca6be941f7e3 (Wed, Nov 9) 
phutil 1bc8b0ff42bc16453d0c5f48360cb78a15cb3eba (Mon, Nov 14)

Revisions and Commits

rP Phabricator
D16919rPfaf42cbe8f59 Allow "Wait for Message" to be unset on build plans

Related Objects

Event Timeline

jmeador created this task.Nov 22 2016, 8:55 PM
avivey added a subscriber: avivey.Nov 22 2016, 8:59 PM

None of your versions exist here, meaning you have some local changes which might cause this.; Can you reporoduce this on Phacility?

avivey added a comment.Nov 22 2016, 9:02 PM

Actually, I can reproduce this at HEAD using a HTTP request step.

avivey added a project: Custom Fields.Nov 22 2016, 9:14 PM

I suspect this might be because the value for "Continue" is an empty string here: https://secure.phabricator.com/source/phabricator/browse/master/src/applications/harbormaster/customfield/HarbormasterBuildStepCoreCustomField.php$27 ; At least, changing that value to be non-empty seems to solve this.

This appears the be the only Select type custom field we have.

epriestley added a subscriber: epriestley.Nov 22 2016, 9:41 PM

The long-term fix is putting this stuff on EditEngine (some discussion in T11887) but this should be fixable in the short term.

epriestley added a revision: D16919: Allow "Wait for Message" to be unset on build plans.Nov 22 2016, 9:48 PM
epriestley closed this task as Resolved by committing rPfaf42cbe8f59: Allow "Wait for Message" to be unset on build plans.Nov 22 2016, 10:02 PM
epriestley added a commit: rPfaf42cbe8f59: Allow "Wait for Message" to be unset on build plans.
Phabricator · Built by Phacility