search

bettybop68 / android-rcs-ims-stack

Automatically exported from code.google.com/p/android-rcs-ims-stack
0 stars 1 forks source link

inversion in receiveMessageDeliveryStatus between FToHTTP and chat #157

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
On "InstantMessagingService.receiveMessageDeliveryStatus".

            // Check if message delivery of a FileTransfer
            String ftSessionId = RichMessaging.getInstance().getFileTransferId(msgId);
            if (ftSessionId == null) {
                // Notify the file delivery outside of the chat session
                receiveFileDeliveryStatus(ftSessionId, status);
            } else {
                // Get session associated to the contact
                ...

On the "if", instead of "ftSessionId == null" should be "ftSessionId != null" 
or as I have done "!StringUtils.isEmpty(ftSessionId)" to avoid empty values 
that for some reason can came empty from the DB.
This issue was switching the handling of the received delivery status between a 
message and a file transfer.

Original issue reported on code.google.com by benoit.j...@gmail.com on 1 Oct 2013 at 8:56

GoogleCodeExporter commented 8 years ago 
Corrected in V2.5.11

Original comment by benoit.j...@gmail.com on 1 Oct 2013 at 9:11