Skip to content

Commit 310652a

Browse files
committed
Applied some clippy advice.
1 parent f77c06f commit 310652a

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

src/planner/core/src/api/actions/create.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,17 +7,17 @@ pub fn create(title: &str, size: u8, status: WorkItemStatus) -> Result<ActionTyp
77
match status {
88
WorkItemStatus::Ready => {
99
let wi = Ready::new(title, size);
10-
let _ = save_single(&title.to_string(), &wi)?;
11-
Ok(ActionTypes::Ready(Ready::new(&title, size)))
10+
save_single(title, &wi)?;
11+
Ok(ActionTypes::Ready(Ready::new(title, size)))
1212
}
1313
WorkItemStatus::InProgress => {
1414
let wi = InProgress::new(title, size);
15-
let _ = save_single(&title.to_string(), &wi)?;
15+
save_single(title, &wi)?;
1616
Ok(ActionTypes::InProgress(InProgress::new(title, size)))
1717
}
1818
WorkItemStatus::Completed => {
1919
let wi = Completed::new(title, size);
20-
let _ = save_single(&title.to_string(), &wi)?;
20+
save_single(title, &wi)?;
2121
Ok(ActionTypes::Completed(Completed::new(title, size)))
2222
}
2323
}
@@ -56,7 +56,7 @@ impl Display for ActionTypes {
5656

5757
#[cfg(test)]
5858
mod tests {
59-
use crate::api::actions::create::{ActionTypes, create};
59+
use crate::api::actions::create::create;
6060
use crate::enums::WorkItemStatus;
6161

6262
#[test]

src/planner/dal/src/json_repository.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ fn get_store() -> Result<File, String> {
1313
.read(true)
1414
.write(true)
1515
.create(true)
16+
.truncate(true)
1617
.open(&file_path)
1718
.map_err(|e| e.to_string())?;
1819

@@ -69,7 +70,7 @@ pub fn delete<T>(key: &str) -> Result<(), String>
6970
where
7071
T: Serialize + DeserializeOwned + Clone,
7172
{
72-
let mut work_items = get_all::<T>().unwrap_or(HashMap::new());
73+
let mut work_items = get_all::<T>().unwrap_or_default();
7374
work_items.remove(key);
7475
save_all(&work_items)
7576
}

0 commit comments

Comments
 (0)