Fix: Entries::in_collection() should be able to return error

This patch changes the Entries::in_collection() interface to return a
Result<()>. This is needed because the fs backend implementation should
be able to check whether a directory actually exists whenever we change
the iterator.

If the implementation detects that the directory does not exist, we can
fail early and error out.

All usages of the interface are adapted by the patch as well.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>

1 files changed, 1 insertions, 1 deletions

diff --git a/lib/domain/libimagtimetrack/src/timetrackingstore.rs b/lib/domain/libimagtimetrack/src/timetrackingstore.rs
index 967c8958..bf25a4df 100644
--- a/lib/domain/libimagtimetrack/src/timetrackingstore.rs
+++ b/lib/domain/libimagtimetrack/src/timetrackingstore.rs
@@ -107,7 +107,7 @@ impl<'a> TimeTrackStore<'a> for Store {
     }
 
     fn get_timetrackings(&'a self) -> Result<TimeTrackingsGetIterator<'a>> {
-        Ok(TimeTrackingsGetIterator::new(self.entries()?.in_collection("timetrack"), self))
+        Ok(TimeTrackingsGetIterator::new(self.entries()?.in_collection("timetrack")?, self))
     }
 
 }