summaryrefslogtreecommitdiffstats
path: root/src/package
diff options
context:
space:
mode:
authorMatthias Beyer <mail@beyermatthias.de>2020-12-10 11:59:13 +0100
committerMatthias Beyer <mail@beyermatthias.de>2020-12-10 11:59:13 +0100
commit1d038bae2bc6c7215e5e9849fe418c6431e5adbd (patch)
tree978045760030c5e577a1e4089bf690129505bfcb /src/package
parent26cee7066649348546599249a1c1a103d130081b (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.rs6
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")?;