trinitrix/core
trinitrix
/
core
Archived
5
4
Fork 2
Code Issues 8 Pull Requests Packages Projects Releases Wiki Activity

Fix(handlers::command): Only send success on real success 

The `room_message_send()` function told the user about a successful sent
room message, even if that was not the case.
This commit is contained in:
Benedikt Peetz 2023-07-26 21:15:53 +02:00
parent f7b161fb55
commit a80245c523
Signed by: bpeetz
GPG Key ID: A5E94010C3A642AD
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/tui_app/app/events/event_types/event/handlers/command.rs
View File

@ -112,12 +112,12 @@ pub async fn handle(
Command::RoomMessageSend(msg) => { Command::RoomMessageSend(msg) => {
if let Some(room) = app.status.room_mut() { if let Some(room) = app.status.room_mut() {
room.send(msg.clone()).await?; room.send(msg.clone()).await?;
send_status_output!("Sent message: `{}`", msg);
} else { } else {
// TODO(@Soispha): Should this raise a lua error? It could be very confusing, // TODO(@Soispha): Should this raise a lua error? It could be very confusing,
// when a user doesn't read the log. // when a user doesn't read the log.
warn!("Can't send message: `{}`, as there is no open room!", &msg); warn!("Can't send message: `{}`, as there is no open room!", &msg);
} }
send_status_output!("Send message: `{}`", msg);
EventStatus::Ok EventStatus::Ok
} }
Command::Greet(name) => { Command::Greet(name) => {