From 32c5ec1e00d5715ca4af25a880162a7a0abcc8d5 Mon Sep 17 00:00:00 2001 From: Matthias Beyer Date: Sun, 30 Jun 2019 10:57:18 +0200 Subject: [PATCH] Fix: Sort by date, not by id This patch fixes a bug. We stated in the helptext of the "show" command that when showing several logs, we sort by date. This was not true as we sorted by Id, which also included the name of the log, hence we effectively sorted by name. This patch fixes that. Output is not sorted by date. Signed-off-by: Matthias Beyer --- bin/domain/imag-log/src/main.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bin/domain/imag-log/src/main.rs b/bin/domain/imag-log/src/main.rs index 2d0c8b15..58d25730 100644 --- a/bin/domain/imag-log/src/main.rs +++ b/bin/domain/imag-log/src/main.rs @@ -173,7 +173,7 @@ fn show(rt: &Runtime) { }) .filter(|e| e.is_log().map_err_trace_exit_unwrap()) .map(|entry| (entry.diary_id().map_err_trace_exit_unwrap(), entry)) - .sorted_by_key(|tpl| tpl.0.clone()) + .sorted_by_key(|tpl| tpl.0.get_date_representation()) .into_iter() .map(|tpl| { debug!("Found entry: {:?}", tpl.1); tpl }) .map(|(id, entry)| {