Skip to content

Commit 52f5808

Browse files
committed
Make private messages structure given by writeln method instead of caller
1 parent 89cc396 commit 52f5808

3 files changed

Lines changed: 13 additions & 7 deletions

File tree

commands.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,15 +175,15 @@ func dmCMD(rest string, u *User) {
175175
return
176176
}
177177
msg := strings.TrimSpace(strings.TrimPrefix(rest, restSplit[0]))
178-
u.writeln(NewPrivateMessage(peer.Name+" <- ", msg, true))
178+
u.writeln(NewPrivateMessage(peer.Name, msg, true))
179179
if u == peer {
180180
devbotRespond(u.room, []string{"You must be really lonely, DMing yourself.",
181181
"Don't worry, I won't judge :wink:",
182182
"srsly?",
183183
"what an idiot"}, 30)
184184
return
185185
}
186-
peer.writeln(NewPrivateMessage(u.Name+" -> ", msg, false))
186+
peer.writeln(NewPrivateMessage(u.Name, msg, false))
187187
}
188188

189189
func hangCMD(rest string, u *User) {
@@ -248,15 +248,15 @@ func usersCMD(_ string, u *User) {
248248
}
249249

250250
func dmRoomCMD(line string, u *User) {
251-
u.writeln(NewPrivateMessage(u.messaging.Name+" <- ", line, true))
251+
u.writeln(NewPrivateMessage(u.messaging.Name, line, true))
252252
if u == u.messaging {
253253
devbotRespond(u.room, []string{"You must be really lonely, DMing yourself.",
254254
"Don't worry, I won't judge :wink:",
255255
"srsly?",
256256
"what an idiot"}, 30)
257257
return
258258
}
259-
u.messaging.writeln(NewPrivateMessage(u.Name+" -> ", line, false))
259+
u.messaging.writeln(NewPrivateMessage(u.Name, line, false))
260260
}
261261

262262
// named devmonk at the request of a certain ced

main.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -699,9 +699,15 @@ func (u *User) writelnWithImageCache(msg Message, cache map[string]image.Image)
699699
msg.text = strings.ReplaceAll(msg.text, `\`+"\n", `\n`) // let people escape newlines
700700
thisUserIsDMSender := msg.messageType == PrivateMessageSend
701701
switch msg.messageType {
702-
case PrivateMessageSend, PrivateMessageReceive:
702+
case PrivateMessageSend:
703703
msg.text = strings.TrimSpace(mdRender(msg.text, lenString(msg.senderName), u.winWidth, cache))
704-
msg.text = msg.senderName + msg.text
704+
msg.text = msg.senderName + " <- " + msg.text
705+
if !thisUserIsDMSender {
706+
msg.text += "\a"
707+
}
708+
case PrivateMessageReceive:
709+
msg.text = strings.TrimSpace(mdRender(msg.text, lenString(msg.senderName), u.winWidth, cache))
710+
msg.text = msg.senderName + " -> " + msg.text
705711
if !thisUserIsDMSender {
706712
msg.text += "\a"
707713
}

rpc.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ func (s *pluginServer) SendMessage(ctx context.Context, msg *pb.Message) (*pb.Me
191191
if !success {
192192
return nil, status.Error(codes.NotFound, "Could not find user "+*msg.EphemeralTo)
193193
}
194-
u.writeln(NewPrivateMessage(msg.GetFrom()+" -> ", msg.Msg, false))
194+
u.writeln(NewPrivateMessage(msg.GetFrom(), msg.Msg, false))
195195
} else {
196196
r := Rooms[msg.Room]
197197
if r == nil {

0 commit comments

Comments
 (0)