app: Migrate group_details to CommonComponent

This commit is contained in:
Valentin Tolmer 2021-10-29 12:32:52 +09:00 committed by nitnelave
parent 5a5baf883f
commit ba72e622c2

View File

@ -4,13 +4,16 @@ use crate::{
remove_user_from_group::RemoveUserFromGroupComponent, remove_user_from_group::RemoveUserFromGroupComponent,
router::{AppRoute, Link}, router::{AppRoute, Link},
}, },
infra::api::HostService, infra::{
common_component::{CommonComponent, CommonComponentParts},
api::HostService,
}
}; };
use anyhow::{bail, Error, Result}; use anyhow::{bail, Error, Result};
use graphql_client::GraphQLQuery; use graphql_client::GraphQLQuery;
use yew::{ use yew::{
prelude::*, prelude::*,
services::{fetch::FetchTask, ConsoleService}, services::ConsoleService,
}; };
#[derive(GraphQLQuery)] #[derive(GraphQLQuery)]
@ -27,15 +30,10 @@ pub type User = get_group_details::GetGroupDetailsGroupUsers;
pub type AddGroupMemberUser = add_group_member::User; pub type AddGroupMemberUser = add_group_member::User;
pub struct GroupDetails { pub struct GroupDetails {
link: ComponentLink<Self>, common: CommonComponentParts<Self>,
props: Props,
/// The group info. If none, the error is in `error`. If `error` is None, then we haven't /// The group info. If none, the error is in `error`. If `error` is None, then we haven't
/// received the server response yet. /// received the server response yet.
group: Option<Group>, group: Option<Group>,
/// Error message displayed to the user.
error: Option<Error>,
// Used to keep the request alive long enough.
_task: Option<FetchTask>,
} }
/// State machine describing the possible transitions of the component state. /// State machine describing the possible transitions of the component state.
@ -55,11 +53,11 @@ pub struct Props {
impl GroupDetails { impl GroupDetails {
fn get_group_details(&mut self) { fn get_group_details(&mut self) {
self._task = HostService::graphql_query::<GetGroupDetails>( self.common.task = HostService::graphql_query::<GetGroupDetails>(
get_group_details::Variables { get_group_details::Variables {
id: self.props.group_id, id: self.common.group_id,
}, },
self.link.callback(Msg::GroupDetailsResponse), self.common.link.callback(Msg::GroupDetailsResponse),
"Error trying to fetch group details", "Error trying to fetch group details",
) )
.map_err(|e| { .map_err(|e| {
@ -69,33 +67,6 @@ impl GroupDetails {
.ok(); .ok();
} }
fn handle_msg(&mut self, msg: <Self as Component>::Message) -> Result<bool> {
match msg {
Msg::GroupDetailsResponse(response) => match response {
Ok(group) => self.group = Some(group.group),
Err(e) => {
self.group = None;
bail!("Error getting user details: {}", e);
}
},
Msg::OnError(e) => return Err(e),
Msg::OnUserAddedToGroup(user) => {
self.group.as_mut().unwrap().users.push(User {
id: user.id,
display_name: user.display_name,
});
}
Msg::OnUserRemovedFromGroup((user_id, _)) => {
self.group
.as_mut()
.unwrap()
.users
.retain(|u| u.id != user_id);
}
}
Ok(true)
}
fn view_messages(&self, error: &Option<Error>) -> Html { fn view_messages(&self, error: &Option<Error>) -> Html {
if let Some(e) = error { if let Some(e) = error {
html! { html! {
@ -124,8 +95,8 @@ impl GroupDetails {
<RemoveUserFromGroupComponent <RemoveUserFromGroupComponent
username=user_id username=user_id
group_id=g.id group_id=g.id
on_user_removed_from_group=self.link.callback(Msg::OnUserRemovedFromGroup) on_user_removed_from_group=self.common.link.callback(Msg::OnUserRemovedFromGroup)
on_error=self.link.callback(Msg::OnError)/> on_error=self.common.link.callback(Msg::OnError)/>
</td> </td>
</tr> </tr>
} }
@ -174,38 +145,60 @@ impl GroupDetails {
<AddGroupMemberComponent <AddGroupMemberComponent
group_id=g.id group_id=g.id
users=users users=users
on_error=self.link.callback(Msg::OnError) on_error=self.common.link.callback(Msg::OnError)
on_user_added_to_group=self.link.callback(Msg::OnUserAddedToGroup)/> on_user_added_to_group=self.common.link.callback(Msg::OnUserAddedToGroup)/>
} }
} }
} }
impl CommonComponent<GroupDetails> for GroupDetails {
fn handle_msg(&mut self, msg: <Self as Component>::Message) -> Result<bool> {
match msg {
Msg::GroupDetailsResponse(response) => match response {
Ok(group) => self.group = Some(group.group),
Err(e) => {
self.group = None;
bail!("Error getting user details: {}", e);
}
},
Msg::OnError(e) => return Err(e),
Msg::OnUserAddedToGroup(user) => {
self.group.as_mut().unwrap().users.push(User {
id: user.id,
display_name: user.display_name,
});
}
Msg::OnUserRemovedFromGroup((user_id, _)) => {
self.group
.as_mut()
.unwrap()
.users
.retain(|u| u.id != user_id);
}
}
Ok(true)
}
fn mut_common(&mut self) -> &mut CommonComponentParts<Self> {
&mut self.common
}
}
impl Component for GroupDetails { impl Component for GroupDetails {
type Message = Msg; type Message = Msg;
type Properties = Props; type Properties = Props;
fn create(props: Self::Properties, link: ComponentLink<Self>) -> Self { fn create(props: Self::Properties, link: ComponentLink<Self>) -> Self {
let mut table = Self { let mut table = Self {
link, common: CommonComponentParts::<Self>::create(props, link),
props,
_task: None,
group: None, group: None,
error: None,
}; };
table.get_group_details(); table.get_group_details();
table table
} }
fn update(&mut self, msg: Self::Message) -> ShouldRender { fn update(&mut self, msg: Self::Message) -> ShouldRender {
self.error = None; CommonComponentParts::<Self>::update(self, msg)
match self.handle_msg(msg) {
Err(e) => {
ConsoleService::error(&e.to_string());
self.error = Some(e);
true
}
Ok(b) => b,
}
} }
fn change(&mut self, _: Self::Properties) -> ShouldRender { fn change(&mut self, _: Self::Properties) -> ShouldRender {
@ -213,7 +206,7 @@ impl Component for GroupDetails {
} }
fn view(&self) -> Html { fn view(&self) -> Html {
match (&self.group, &self.error) { match (&self.group, &self.common.error) {
(None, None) => html! {{"Loading..."}}, (None, None) => html! {{"Loading..."}},
(None, Some(e)) => html! {<div>{"Error: "}{e.to_string()}</div>}, (None, Some(e)) => html! {<div>{"Error: "}{e.to_string()}</div>},
(Some(u), error) => { (Some(u), error) => {