Skip to content

Commit fdd6436

Browse files
deduping new entries to secret fields
1 parent d85cdb1 commit fdd6436

File tree

2 files changed

+12
-11
lines changed

2 files changed

+12
-11
lines changed

src/ui/details/detail_view.rs

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -53,16 +53,19 @@ pub fn save_edit(params: SaveEdit) {
5353
config,
5454
} = params;
5555

56-
config.db.write().unwrap().edit_field(id, &field, value.get());
57-
if field == DbFields::Title {
58-
let new_list = config.db.read().unwrap().get_list();
59-
set_list.update(|list: &mut im::Vector<(usize, &'static str, usize)>| {
60-
*list = new_list;
61-
});
62-
}
56+
let last_val = config.db.read().unwrap().get_last_by_field(&id, &field);
57+
if last_val != value.get() {
58+
config.db.write().unwrap().edit_field(id, &field, value.get());
59+
if field == DbFields::Title {
60+
let new_list = config.db.read().unwrap().get_list();
61+
set_list.update(|list: &mut im::Vector<(usize, &'static str, usize)>| {
62+
*list = new_list;
63+
});
64+
}
6365

64-
dates.set(config.db.read().unwrap().get_history_dates(&id, &field));
65-
input_id.request_focus();
66+
dates.set(config.db.read().unwrap().get_history_dates(&id, &field));
67+
input_id.request_focus();
68+
}
6669

6770
if is_secret {
6871
value.set(String::from(SECRET_PLACEHOLDER));

src/ui/details/list_item.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -181,8 +181,6 @@ pub fn list_item(param: ListItem) -> impl View {
181181
tooltip_signals,
182182
},
183183
move |_| {
184-
// TODO: compare to reset_text and don't save if the same
185-
186184
if save_btn_visible.get() {
187185
reset_text.set(field_value.get());
188186
if is_secret {

0 commit comments

Comments
 (0)