summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorThanos Doukoudakis <thanos.doukoudakis@isode.com>2017-04-05 16:18:09 (GMT)
committerThanos Doukoudakis <thanos.doukoudakis@isode.com>2017-04-18 09:50:15 (GMT)
commit4656f05cc6e0db2bddb3ec1409c530f1d334e949 (patch)
tree7f245b63fd0003bcefb0459a2a92c49d46ad5a86 /Swift/QtUI
parent4024517891d3a222e71e6632c3609fcda08c875e (diff)
downloadswift-4656f05cc6e0db2bddb3ec1409c530f1d334e949.zip
swift-4656f05cc6e0db2bddb3ec1409c530f1d334e949.tar.bz2
Don't pass the impromptu MUC 'reason' as the room JID
Changing the constructor of the CreateImpromptuMUCUIEvent class not to accept default values to its parameters. This approach is less prone to mistakes, like forgetting an argument. This commit fixes SWIFT-271 bug, where only two arguments were being passed in the constructor. As a result the reason string was used to create a room JID, making the room unusable. By passing an empty string, the room JID created is not valid, and when ChatsManager handles the event, it will generate a new valid room JID. Test information: Unit test pass. Tested the creation of new impromptu chat with with windows 10 (qt5.7). Change-Id: I4ce9ee7db14b9125970a03f19d1ac0c32e1b5a0e
Diffstat (limited to 'Swift/QtUI')
-rw-r--r--Swift/QtUI/UserSearch/QtUserSearchWindow.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/Swift/QtUI/UserSearch/QtUserSearchWindow.cpp b/Swift/QtUI/UserSearch/QtUserSearchWindow.cpp
index e770186..8d15739 100644
--- a/Swift/QtUI/UserSearch/QtUserSearchWindow.cpp
+++ b/Swift/QtUI/UserSearch/QtUserSearchWindow.cpp
@@ -157,7 +157,7 @@ void QtUserSearchWindow::handleAccepted() {
jids.push_back(contact->jid);
}
- eventStream_->send(std::make_shared<CreateImpromptuMUCUIEvent>(jids, Q2PSTRING(firstMultiJIDPage_->reason_->text())));
+ eventStream_->send(std::make_shared<CreateImpromptuMUCUIEvent>(jids, JID(), Q2PSTRING(firstMultiJIDPage_->reason_->text())));
break;
case Type::InviteToChat:
for (Contact::ref contact : contactVector_) {