matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Fix: 'start-time' cannot be None

Browse source 
The UI is configured to require the 'start-time' parameter, so we do not
need to check for None here.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
This commit is contained in:
Matthias Beyer 2019-02-06 01:38:26 +01:00
parent afe275692e
commit 340dab18f0
1 changed files with 13 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
bin/domain/imag-timetrack/src/start.rs
View file

@ -32,15 +32,19 @@ pub fn start(rt: &Runtime) -> i32 {
let (_, cmd) = rt.cli().subcommand(); let (_, cmd) = rt.cli().subcommand();
let cmd = cmd.unwrap(); // checked in main() let cmd = cmd.unwrap(); // checked in main()
let start = match cmd.value_of("start-time") { let start = {
None | Some("now") => ::chrono::offset::Local::now().naive_local(), let startstr = cmd.value_of("start-time").unwrap(); // safe by clap
Some(ndt) => match NaiveDateTime::from_str(ndt).map_err(Error::from) { if startstr == "now" {
Ok(ndt) => ndt, ::chrono::offset::Local::now().naive_local()
Err(e) => { } else {
trace_error(&e); match NaiveDateTime::from_str(startstr).map_err(Error::from) {
error!("Cannot continue, not having start time"); Ok(ndt) => ndt,
return 1 Err(e) => {
}, trace_error(&e);
error!("Cannot continue, not having start time");
return 1
},
}
} }
}; };