The view model can no longer be initialized without an app

This commit is contained in:
Savanni D'Gerinel 2024-01-29 09:18:36 -05:00
parent 7d5d639ed9
commit 9874b6081b
3 changed files with 85 additions and 76 deletions

View File

@ -197,7 +197,7 @@ impl DayDetail {
pub struct DayEditPrivate { pub struct DayEditPrivate {
on_finished: RefCell<Box<dyn Fn()>>, on_finished: RefCell<Box<dyn Fn()>>,
workout_rows: RefCell<gtk::Box>, workout_rows: RefCell<gtk::Box>,
view_model: RefCell<DayDetailViewModel>, view_model: RefCell<Option<DayDetailViewModel>>,
} }
impl Default for DayEditPrivate { impl Default for DayEditPrivate {
@ -210,7 +210,7 @@ impl Default for DayEditPrivate {
.hexpand(true) .hexpand(true)
.build(), .build(),
), ),
view_model: RefCell::new(DayDetailViewModel::default()), view_model: RefCell::new(None),
} }
} }
} }
@ -239,7 +239,7 @@ impl DayEdit {
s.set_orientation(gtk::Orientation::Vertical); s.set_orientation(gtk::Orientation::Vertical);
s.set_hexpand(true); s.set_hexpand(true);
*s.imp().on_finished.borrow_mut() = Box::new(on_finished); *s.imp().on_finished.borrow_mut() = Box::new(on_finished);
*s.imp().view_model.borrow_mut() = view_model.clone(); *s.imp().view_model.borrow_mut() = Some(view_model.clone());
let workout_buttons = workout_buttons(view_model.clone(), { let workout_buttons = workout_buttons(view_model.clone(), {
let s = s.clone(); let s = s.clone();
@ -320,7 +320,11 @@ impl DayEdit {
_ => panic!("Record type {:?} is not a Time/Distance record", type_), _ => panic!("Record type {:?} is not a Time/Distance record", type_),
}; };
let record = Record { id, data }; let record = Record { id, data };
self.imp().view_model.borrow().update_record(record); let view_model = self.imp().view_model.borrow();
let view_model = view_model
.as_ref()
.expect("DayEdit has not been initialized with a view model");
view_model.update_record(record);
} }
} }

View File

@ -93,12 +93,9 @@ impl<T: Clone + emseries::Recordable> Deref for RecordState<T> {
} }
} }
#[derive(Default)] #[derive(Clone)]
struct DayDetailViewModelInner {}
#[derive(Clone, Default)]
pub struct DayDetailViewModel { pub struct DayDetailViewModel {
app: Option<App>, app: App,
pub date: chrono::NaiveDate, pub date: chrono::NaiveDate,
weight: Arc<RwLock<Option<RecordState<ft_core::Weight>>>>, weight: Arc<RwLock<Option<RecordState<ft_core::Weight>>>>,
steps: Arc<RwLock<Option<RecordState<ft_core::Steps>>>>, steps: Arc<RwLock<Option<RecordState<ft_core::Steps>>>>,
@ -110,7 +107,7 @@ pub struct DayDetailViewModel {
impl DayDetailViewModel { impl DayDetailViewModel {
pub fn new(date: chrono::NaiveDate, records: Vec<Record<TraxRecord>>, app: App) -> Self { pub fn new(date: chrono::NaiveDate, records: Vec<Record<TraxRecord>>, app: App) -> Self {
let s = Self { let s = Self {
app: Some(app), app,
date, date,
weight: Arc::new(RwLock::new(None)), weight: Arc::new(RwLock::new(None)),
@ -230,63 +227,63 @@ impl DayDetailViewModel {
glib::spawn_future({ glib::spawn_future({
let s = self.clone(); let s = self.clone();
async move { async move {
if let Some(app) = s.app { let weight_record = s.weight.read().unwrap().clone();
let weight_record = s.weight.read().unwrap().clone(); match weight_record {
match weight_record { Some(RecordState::New(Record { data, .. })) => {
Some(RecordState::New(Record { data, .. })) => { let _ = s.app.put_record(TraxRecord::Weight(data)).await;
let _ = app.put_record(TraxRecord::Weight(data)).await;
}
Some(RecordState::Original(_)) => {}
Some(RecordState::Updated(weight)) => {
let _ = app
.update_record(Record {
id: weight.id,
data: TraxRecord::Weight(weight.data),
})
.await;
}
Some(RecordState::Deleted(_)) => {}
None => {}
} }
Some(RecordState::Original(_)) => {}
let steps_record = s.steps.read().unwrap().clone(); Some(RecordState::Updated(weight)) => {
match steps_record { let _ = s
Some(RecordState::New(Record { data, .. })) => { .app
let _ = app.put_record(TraxRecord::Steps(data)).await; .update_record(Record {
} id: weight.id,
Some(RecordState::Original(_)) => {} data: TraxRecord::Weight(weight.data),
Some(RecordState::Updated(steps)) => { })
let _ = app .await;
.update_record(Record {
id: steps.id,
data: TraxRecord::Steps(steps.data),
})
.await;
}
Some(RecordState::Deleted(_)) => {}
None => {}
} }
Some(RecordState::Deleted(_)) => {}
None => {}
}
let records = s let steps_record = s.steps.read().unwrap().clone();
.records match steps_record {
.write() Some(RecordState::New(Record { data, .. })) => {
.unwrap() let _ = s.app.put_record(TraxRecord::Steps(data)).await;
.drain() }
.map(|(_, record)| record) Some(RecordState::Original(_)) => {}
.collect::<Vec<RecordState<TraxRecord>>>(); Some(RecordState::Updated(steps)) => {
let _ = s
.app
.update_record(Record {
id: steps.id,
data: TraxRecord::Steps(steps.data),
})
.await;
}
Some(RecordState::Deleted(_)) => {}
None => {}
}
for record in records { let records = s
println!("saving record: {:?}", record); .records
match record { .write()
RecordState::New(Record { data, .. }) => { .unwrap()
let _ = app.put_record(data).await; .drain()
} .map(|(_, record)| record)
RecordState::Original(_) => {} .collect::<Vec<RecordState<TraxRecord>>>();
RecordState::Updated(r) => {
let _ = app.update_record(r.clone()).await; for record in records {
} println!("saving record: {:?}", record);
RecordState::Deleted(_) => unimplemented!(), match record {
RecordState::New(Record { data, .. }) => {
let _ = s.app.put_record(data).await;
} }
RecordState::Original(_) => {}
RecordState::Updated(r) => {
let _ = s.app.update_record(r.clone()).await;
}
RecordState::Deleted(_) => unimplemented!(),
} }
} }
} }

View File

@ -25,7 +25,7 @@ use std::cell::RefCell;
#[derive(Default)] #[derive(Default)]
pub struct DayDetailViewPrivate { pub struct DayDetailViewPrivate {
container: Singleton, container: Singleton,
view_model: RefCell<DayDetailViewModel>, view_model: RefCell<Option<DayDetailViewModel>>,
} }
#[glib::object_subclass] #[glib::object_subclass]
@ -47,7 +47,7 @@ glib::wrapper! {
impl DayDetailView { impl DayDetailView {
pub fn new(view_model: DayDetailViewModel) -> Self { pub fn new(view_model: DayDetailViewModel) -> Self {
let s: Self = Object::builder().build(); let s: Self = Object::builder().build();
*s.imp().view_model.borrow_mut() = view_model; *s.imp().view_model.borrow_mut() = Some(view_model);
s.append(&s.imp().container); s.append(&s.imp().container);
@ -57,20 +57,28 @@ impl DayDetailView {
} }
fn view(&self) { fn view(&self) {
self.imp() let view_model = self.imp().view_model.borrow();
.container let view_model = view_model
.swap(&DayDetail::new(self.imp().view_model.borrow().clone(), { .as_ref()
let s = self.clone(); .expect("DayDetailView has not been initialized with a view_model")
move || s.edit() .clone();
}));
self.imp().container.swap(&DayDetail::new(view_model, {
let s = self.clone();
move || s.edit()
}));
} }
fn edit(&self) { fn edit(&self) {
self.imp() let view_model = self.imp().view_model.borrow();
.container let view_model = view_model
.swap(&DayEdit::new(self.imp().view_model.borrow().clone(), { .as_ref()
let s = self.clone(); .expect("DayDetailView has not been initialized with a view_model")
move || s.view() .clone();
}));
self.imp().container.swap(&DayEdit::new(view_model, {
let s = self.clone();
move || s.view()
}));
} }
} }