diff options
author | Danny van Heumen <danny@dannyvanheumen.nl> | 2015-03-06 23:58:45 +0100 |
---|---|---|
committer | Danny van Heumen <danny@dannyvanheumen.nl> | 2015-03-07 00:02:33 +0100 |
commit | 779c5ddbf4a078f8aa236d8327badf65785b40b7 (patch) | |
tree | 385a6fea42f6a8c7d8c31e1d65decfe2def21cdd /src/net/java/sip/communicator/plugin | |
parent | 879f00a06727fb53da5841b49524d50ac40bfd16 (diff) | |
download | jitsi-779c5ddbf4a078f8aa236d8327badf65785b40b7.zip jitsi-779c5ddbf4a078f8aa236d8327badf65785b40b7.tar.gz jitsi-779c5ddbf4a078f8aa236d8327badf65785b40b7.tar.bz2 |
Fix for OTR UI in case of multiple OTR session.
Okay, so this was a real PITA. The otr4j library behaves as expected.
The OTR plugin, on the other hand, hides controls and does not get
updated on the actual selected resouce (chat transport) at all. So as a
consequence the OTR plugin is actually giving a lot of misinformation.
There is still an outstanding issue where you establish an OTR encrypted
session for a particular session ID and the OTR session selector box
does not automatically select the encrypted session. I'm not sure if
this should be the desired behaviour, though. What if there are 2 OTR
encrypted sessions, would you want to switch?
That is something to think about ...
Diffstat (limited to 'src/net/java/sip/communicator/plugin')
-rw-r--r-- | src/net/java/sip/communicator/plugin/otr/ScOtrEngineImpl.java | 3 | ||||
-rw-r--r-- | src/net/java/sip/communicator/plugin/otr/authdialog/OTRv3OutgoingSessionSwitcher.java | 3 |
2 files changed, 1 insertions, 5 deletions
diff --git a/src/net/java/sip/communicator/plugin/otr/ScOtrEngineImpl.java b/src/net/java/sip/communicator/plugin/otr/ScOtrEngineImpl.java index 88668e3..eab7c37 100644 --- a/src/net/java/sip/communicator/plugin/otr/ScOtrEngineImpl.java +++ b/src/net/java/sip/communicator/plugin/otr/ScOtrEngineImpl.java @@ -1304,8 +1304,7 @@ public class ScOtrEngineImpl public List<Session> getSessionInstances(OtrContact otrContact)
{
if (otrContact == null)
- return null;
-
+ return Collections.emptyList();
return getSession(otrContact).getInstances();
}
diff --git a/src/net/java/sip/communicator/plugin/otr/authdialog/OTRv3OutgoingSessionSwitcher.java b/src/net/java/sip/communicator/plugin/otr/authdialog/OTRv3OutgoingSessionSwitcher.java index 528cbf7..ad3f37c 100644 --- a/src/net/java/sip/communicator/plugin/otr/authdialog/OTRv3OutgoingSessionSwitcher.java +++ b/src/net/java/sip/communicator/plugin/otr/authdialog/OTRv3OutgoingSessionSwitcher.java @@ -244,9 +244,6 @@ public class OTRv3OutgoingSessionSwitcher */ public void setCurrentContact(Contact contact, String resourceName) { - if (this.contact != null && this.contact.contact == contact) - return; - if (resourceName == null) { this.contact = |