diff options
author | Gurmeen Bindra <gurmeen.bindra@isode.com> | 2013-06-26 13:12:27 (GMT) |
---|---|---|
committer | Gurmeen Bindra <gurmeen.bindra@isode.com> | 2013-06-27 13:27:43 (GMT) |
commit | 1006276e3c78c2098d8efd5dedff0ee87e0e006f (patch) | |
tree | b3ad876f3d5bd815f45e3263f3011aede8ee3767 /src/com/isode/stroke/serializer | |
parent | 7d7ecdbb01daa051472bf40e269b78c4faf7b34e (diff) | |
download | stroke-1006276e3c78c2098d8efd5dedff0ee87e0e006f.zip stroke-1006276e3c78c2098d8efd5dedff0ee87e0e006f.tar.bz2 |
Change the data type of seconds in IQ:last from int to Long
Making it Long allows it to hold an XML-unsignedLong value as well
as null values. Before this patch, it was an int and defaulted to 0.
This was not right as int is too small to hold number of seconds for
last activity time and primitive data types do not allow for null values.
Test-information:
tested using an XMPP client to query last IQ on MUC rooms
Change-Id: I6274403610bd60038fd7c235fad3bc2798f38e19
Reviewer: Kevin Smith <kevin.smith@isode.com>
Diffstat (limited to 'src/com/isode/stroke/serializer')
-rw-r--r-- | src/com/isode/stroke/serializer/payloadserializers/LastSerializer.java | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/com/isode/stroke/serializer/payloadserializers/LastSerializer.java b/src/com/isode/stroke/serializer/payloadserializers/LastSerializer.java index e78a724..8eff41a 100644 --- a/src/com/isode/stroke/serializer/payloadserializers/LastSerializer.java +++ b/src/com/isode/stroke/serializer/payloadserializers/LastSerializer.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011 Isode Limited, London, England. + * Copyright (c) 2011-2013 Isode Limited, London, England. * All rights reserved. */ /* @@ -19,6 +19,9 @@ public class LastSerializer extends GenericPayloadSerializer<Last> { @Override protected String serializePayload(Last last) { - return "<query xmlns='jabber:iq:last' seconds='" + Integer.toString(last.getSeconds()) + "'/>"; + if(last.getSeconds() == null) { + return "<query xmlns='jabber:iq:last'/>"; + } + return "<query xmlns='jabber:iq:last' seconds='" + Long.toString(last.getSeconds()) + "'/>"; } } |