Matthias Beyer
8c37fb865a
This patch rebuilds DiaryEntryIterator to be a wrapper around StoreIdIterator and thus `Diary::entries()` to use `Store::entries` and not `Store::retrieve_for_module()`. The `Store::retrieve_for_module()` function is somehow buggy and loads contents of the files into memory and _somehow_ causes the entries to be rewritten without newlines. This bug is fixed by moving away from `Store::retrieve_for_module()`.
142 lines
5 KiB
Rust
142 lines
5 KiB
Rust
//
|
|
// imag - the personal information management suite for the commandline
|
|
// Copyright (C) 2015-2018 Matthias Beyer <mail@beyermatthias.de> and contributors
|
|
//
|
|
// This library is free software; you can redistribute it and/or
|
|
// modify it under the terms of the GNU Lesser General Public
|
|
// License as published by the Free Software Foundation; version
|
|
// 2.1 of the License.
|
|
//
|
|
// This library is distributed in the hope that it will be useful,
|
|
// but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
// Lesser General Public License for more details.
|
|
//
|
|
// You should have received a copy of the GNU Lesser General Public
|
|
// License along with this library; if not, write to the Free Software
|
|
// Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
|
//
|
|
|
|
use std::cmp::Ordering;
|
|
|
|
use libimagstore::store::FileLockEntry;
|
|
use libimagstore::store::Store;
|
|
use libimagerror::trace::trace_error;
|
|
use libimagentryutil::isa::Is;
|
|
|
|
use chrono::offset::Local;
|
|
use chrono::Datelike;
|
|
use itertools::Itertools;
|
|
use chrono::naive::NaiveDateTime;
|
|
use chrono::Timelike;
|
|
|
|
use entry::IsDiaryEntry;
|
|
use diaryid::DiaryId;
|
|
use diaryid::FromStoreId;
|
|
use error::DiaryErrorKind as DEK;
|
|
use error::ResultExt;
|
|
use error::Result;
|
|
use iter::DiaryEntryIterator;
|
|
use iter::DiaryNameIterator;
|
|
|
|
pub trait Diary {
|
|
|
|
// create or get a new entry for today
|
|
fn new_entry_today(&self, diary_name: &str) -> Result<FileLockEntry>;
|
|
|
|
// create or get a new entry for now
|
|
fn new_entry_now(&self, diary_name: &str) -> Result<FileLockEntry>;
|
|
|
|
// Get an iterator for iterating over all entries of a Diary
|
|
fn entries(&self, diary_name: &str) -> Result<DiaryEntryIterator>;
|
|
|
|
fn get_youngest_entry_id(&self, diary_name: &str) -> Option<Result<DiaryId>>;
|
|
|
|
/// Get all diary names
|
|
fn diary_names(&self) -> Result<DiaryNameIterator>;
|
|
|
|
}
|
|
|
|
impl Diary for Store {
|
|
|
|
// create or get a new entry for today
|
|
fn new_entry_today(&self, diary_name: &str) -> Result<FileLockEntry> {
|
|
let dt = Local::now();
|
|
let ndt = dt.naive_local();
|
|
let id = DiaryId::new(String::from(diary_name), ndt.year(), ndt.month(), ndt.day(), 0, 0, 0);
|
|
|
|
let mut entry = self.retrieve(id).chain_err(|| DEK::StoreReadError)?;
|
|
let _ = entry.set_isflag::<IsDiaryEntry>()?;
|
|
Ok(entry)
|
|
}
|
|
|
|
fn new_entry_now(&self, diary_name: &str) -> Result<FileLockEntry> {
|
|
let dt = Local::now();
|
|
let ndt = dt.naive_local();
|
|
let id = DiaryId::new(String::from(diary_name),
|
|
ndt.year(),
|
|
ndt.month(),
|
|
ndt.day(),
|
|
ndt.hour(),
|
|
ndt.minute(),
|
|
ndt.second());
|
|
|
|
let mut entry = self.retrieve(id).chain_err(|| DEK::StoreReadError)?;
|
|
let _ = entry.set_isflag::<IsDiaryEntry>()?;
|
|
Ok(entry)
|
|
}
|
|
|
|
// Get an iterator for iterating over all entries
|
|
fn entries(&self, diary_name: &str) -> Result<DiaryEntryIterator> {
|
|
debug!("Building iterator for module 'diary' with diary name = '{}'", diary_name);
|
|
Store::entries(self)
|
|
.map(|iter| DiaryEntryIterator::new(String::from(diary_name), iter.without_store()))
|
|
.chain_err(|| DEK::StoreReadError)
|
|
}
|
|
|
|
fn get_youngest_entry_id(&self, diary_name: &str) -> Option<Result<DiaryId>> {
|
|
match Diary::entries(self, diary_name) {
|
|
Err(e) => Some(Err(e)),
|
|
Ok(entries) => {
|
|
entries
|
|
.map(|e| DiaryId::from_storeid(&e))
|
|
.sorted_by(|a, b| {
|
|
match (a, b) {
|
|
(&Ok(ref a), &Ok(ref b)) => {
|
|
let a : NaiveDateTime = a.clone().into();
|
|
let b : NaiveDateTime = b.clone().into();
|
|
|
|
a.cmp(&b)
|
|
},
|
|
|
|
(&Ok(_), &Err(ref e)) => {
|
|
trace_error(e);
|
|
Ordering::Less
|
|
},
|
|
(&Err(ref e), &Ok(_)) => {
|
|
trace_error(e);
|
|
Ordering::Greater
|
|
},
|
|
(&Err(ref e1), &Err(ref e2)) => {
|
|
trace_error(e1);
|
|
trace_error(e2);
|
|
Ordering::Equal
|
|
},
|
|
}
|
|
})
|
|
.into_iter()
|
|
//.map(|sidres| sidres.map(|sid| DiaryId::from_storeid(&sid)))
|
|
.next()
|
|
}
|
|
}
|
|
}
|
|
|
|
/// Get all diary names
|
|
fn diary_names(&self) -> Result<DiaryNameIterator> {
|
|
self.retrieve_for_module("diary")
|
|
.chain_err(|| DEK::StoreReadError)
|
|
.map(DiaryNameIterator::new)
|
|
}
|
|
|
|
}
|
|
|