diff options
author | Matthias Beyer <mail@beyermatthias.de> | 2020-12-10 11:59:13 +0100 |
---|---|---|
committer | Matthias Beyer <mail@beyermatthias.de> | 2020-12-10 11:59:13 +0100 |
commit | 1d038bae2bc6c7215e5e9849fe418c6431e5adbd (patch) | |
tree | 978045760030c5e577a1e4089bf690129505bfcb /src/package | |
parent | 26cee7066649348546599249a1c1a103d130081b (diff) |
Remove feature: State message in "OK" case.
The "OK" case had a state message parameter. This was removed because
"OK" does mean that everything went well. No need to have a message
here.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Diffstat (limited to 'src/package')
-rw-r--r-- | src/package/script.rs | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/package/script.rs b/src/package/script.rs index 805d17f..e7df73a 100644 --- a/src/package/script.rs +++ b/src/package/script.rs @@ -192,7 +192,6 @@ struct StateHelper; impl HelperDef for StateHelper { fn call<'reg: 'rc, 'rc>(&self, h: &Helper, _: &Handlebars, _: &Context, _rc: &mut RenderContext, out: &mut dyn Output) -> HelperResult { - let state_msg = h.param(1).ok_or_else(|| RenderError::new("Required parameter missing: state message"))?; h.param(0) .ok_or_else(|| RenderError::new("Required parameter missing: state"))? .value() @@ -200,12 +199,11 @@ impl HelperDef for StateHelper { .ok_or_else(|| RenderError::new("Required parameter must be a string: state")) .and_then(|state| match state { "OK" => { - out.write("echo '#BUTIDO:STATE:OK:")?; - out.write(state_msg.value().render().as_ref())?; - out.write("'\n")?; + out.write("echo '#BUTIDO:STATE:OK\n")?; Ok(()) }, "ERR" => { + let state_msg = h.param(1).ok_or_else(|| RenderError::new("Required parameter missing: state message"))?; out.write("echo '#BUTIDO:STATE:ERR:")?; out.write(state_msg.value().render().as_ref())?; out.write("'\n")?; |