adam/znc
1
0
Fork 0
mirror of https://github.com/znc/znc.git synced 2026-08-07 17:33:34 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix CModeMessage::GetModes()

Browse Source 
Thanks to KindOne for finding the bug. CModeMessage::GetModes()
must trim the colon that CMessage::GetParams() may include.
This commit is contained in:
J-P Nurmi
2015-09-17 13:26:24 +02:00
parent 2c0dfbbe18
commit 57fb58bf71
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
include/znc/Message.h
+1 -1
View File
@@ -144,7 +144,7 @@ public:
class CModeMessage : public CTargetMessage {
public:
CString GetModes() const { return GetParams(1); }
CString GetModes() const { return GetParams(1).TrimPrefix_n(":"); }
};
class CNickMessage : public CMessage {
test/MessageTest.cpp
+5
View File
@@ -249,6 +249,11 @@ TEST(MessageTest, Mode) {
EXPECT_EQ("+k foo", msg.GetModes());
EXPECT_EQ(":nick MODE #chan +k foo", msg.ToString());
msg.Parse(":nick MODE nick :+i");
EXPECT_EQ("+i", msg.GetModes());
EXPECT_EQ(":nick MODE nick :+i", msg.ToString());
}
TEST(MessageTest, Nick) {