Merge pull request #277 from Arkaeriit/quick-fix-plugins

Fix plugings messages with no sender.

Author: Arkaeriit

main.go

@@ -702,15 +702,10 @@ func (u *User) writelnWithImageCache(msg Message, cache map[string]image.Image)
 	case PrivateMessageSend:
 		msg.text = strings.TrimSpace(mdRender(msg.text, lenString(msg.senderName), u.winWidth, cache))
 		msg.text = msg.senderName + " <- " + msg.text
-		if !thisUserIsDMSender {
-			msg.text += "\a"
-		}
 	case PrivateMessageReceive:
 		msg.text = strings.TrimSpace(mdRender(msg.text, lenString(msg.senderName), u.winWidth, cache))
 		msg.text = msg.senderName + " -> " + msg.text
-		if !thisUserIsDMSender {
-			msg.text += "\a"
-		}
+		msg.text += "\a"
 	case DefaultMessage:
 		msg.text = strings.TrimSpace(mdRender(msg.text, lenString(msg.senderName)+2, u.winWidth, cache))
 		msg.text = msg.senderName + ": " + msg.text

rpc.go

@@ -191,13 +191,28 @@ func (s *pluginServer) SendMessage(ctx context.Context, msg *pb.Message) (*pb.Me
 		if !success {
 			return nil, status.Error(codes.NotFound, "Could not find user "+*msg.EphemeralTo)
 		}
-		u.writeln(NewPrivateMessage(msg.GetFrom(), msg.Msg, false))
+		var localMsg Message
+		if msg.GetFrom() == "" {
+			localMsg = NewNoSenderMessage(msg.Msg)
+		} else {
+			localMsg = NewPrivateMessage(msg.GetFrom(), msg.Msg, false)
+		}
+		localMsg = localMsg.dontSendToPlugin()
+		u.writeln(localMsg)
 	} else {
 		r := Rooms[msg.Room]
 		if r == nil {
 			return nil, status.Error(codes.InvalidArgument, "Room does not exist")
 		}
-		r.broadcast(NewMessage(msg.GetFrom(), msg.Msg).dontSendToPlugin())
+		// The plugin API doesn't export a function to send messages with no sender, so we must add support for it here.
+		var localMsg Message
+		if msg.GetFrom() == "" {
+			localMsg = NewNoSenderMessage(msg.Msg)
+		} else {
+			localMsg = NewMessage(msg.GetFrom(), msg.Msg)
+		}
+		localMsg = localMsg.dontSendToPlugin()
+		r.broadcast(localMsg)
 	}
 	return &pb.MessageRes{}, nil
 }