[GH-ISSUE #386] SMS Messages may be synched multiple times #276

Closed
opened 2026-02-28 01:23:58 +03:00 by kerem · 0 comments
Owner

Originally created by @alxndrsn on GitHub (Apr 6, 2016).
Original GitHub issue: https://github.com/ushahidi/SMSSync/issues/386

Similar to: #382

If there is more than one pending PostMessage, each pending post message will be synched multiple times at github.com/ushahidi/SMSSync@a1f2224bbb/smssync/src/main/java/org/addhen/smssync/data/message/PostMessage.java (L176):

    public boolean syncPendingMessages(final String uuid) {
        ...
        if (!TextUtils.isEmpty(uuid)) {
            ...
        } else {
            final List<Message> messages = mMessageDataSource.syncFetchPending();
            if (messages != null && messages.size() > 0) {
                for (Message message : messages) {
                    status = postMessage(messages);
                }
            }
        }
        ...
    }
Originally created by @alxndrsn on GitHub (Apr 6, 2016). Original GitHub issue: https://github.com/ushahidi/SMSSync/issues/386 Similar to: #382 If there is more than one pending `PostMessage`, each pending post message will be synched multiple times at https://github.com/ushahidi/SMSSync/blob/a1f2224bbb4488a9772eb35a1d8648e0dc350a59/smssync/src/main/java/org/addhen/smssync/data/message/PostMessage.java#L176: ``` public boolean syncPendingMessages(final String uuid) { ... if (!TextUtils.isEmpty(uuid)) { ... } else { final List<Message> messages = mMessageDataSource.syncFetchPending(); if (messages != null && messages.size() > 0) { for (Message message : messages) { status = postMessage(messages); } } } ... } ```
kerem closed this issue 2026-02-28 01:23:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/SMSSync#276
No description provided.