880243 - [MMS] A reply to a multi-recipient MMS is not grouped in the same thread

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: RIL, defect)

Description

[isabelrios]

Attachment: Two threads created

Bug seen on Unagi v1-train
Gecko-65bbcee
Gaia-13c6246

PROCEDURE
1. DuT A: Send MMS to  multi recipients with international format: B(+34xxxxxxxxx) and C(+34yyyyyyyyy)
2. B replies to the group MMS
3. Verify whether the reply is correctly manage in the thread

EXPECTED
DuT A shows only one thread with the original MMS and the different replies from B and/or C inside it.

ACTUAL
DuT A shows two different threads, the original group MMS and a new on with B international format number

Comment 1

[Corey Frang [:gnarf]]

Impossible to do - SMS replies are sent individually to each recipient which is why they are in their own thread.  If you reply with an MMS it shows up correctly in the thread.

If we were to force group messages to be sent as MMS this works, but we aren't doing this.

needinfo? on ayman to verify

Comment 2

[isabelrios]

Hi Corey,

The reply was also an MMS, please see attached file. There you could see the original MMS and the reply, also an MMS.

Thanks

Comment 3

[ayman maat :maat]

(In reply to Isabel Rios [:isabel_rios] from comment #2)
> Hi Corey,
> 
> The reply was also an MMS, please see attached file. There you could see the
> original MMS and the reply, also an MMS.
> 
> Thanks

I was about to point that out as well and ask Isabel if that was indeed the case. But she has just confirmed it.

needinfo me again if you require my input corey

Comment 4

[Lukas Blakk [:lsblakk] use ?needinfo]

Blocking for now, MMS work.

Comment 5

[Gabriele Svelto [:gsvelto]]

I was looking at this bug and bug 880251 too because I have the feeling that they may be caused by the same root issue, namely that the MMS threadId field is not set correctly.

However since I'm having a bit of trouble reproducing it on my device mostly because I haven't yet got MMS functionality to work correctly (carrier configuration instructions are... confusing) I was wondering if you could post the communication app indexedDB database and attach it to this bug. This way I should be able to reproduce the problem simply by loading the DB with the test messages on my phone.

To pull the DB first enable remote debugging via the settings app then look for a file called ***communications.gaiamobile.org under the /data/local/indexedDB folder using adb; on my phone it looks like this:

$ adb shell ls /data/local/indexedDB | grep communications.gaiamobile.org
6+f+app+++communications.gaiamobile.org

Once you have the correct filename pull it via adb:

$ adb pull /data/local/indexedDB/6+f+app+++communications.gaiamobile.org 6+f+app+++communications.gaiamobile.org

This should yield a folder with all the relevant data in it; zip it and attach it to the bug.

Comment 6

[Rick Waldron [:rwaldron]]

Corey and I discussed this last week and determined that somehow the threadId needs to be sent along with the outgoing message, in a way that it will be preserved/returned in any corresponding incoming messages.

Comment 7

[isabelrios]

Hi Gabriele,

Sorry for the late reply. Attached you can find the files you asked for. Hope that helps.

Comment 8

[isabelrios]

Attachment: comms files

Comment 9

[bov]

This issue tested on Leo 1.1 (Test Run Firefox OS 1.1.0 Full – Leo)
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/8e3f39363c54
Gaia: ea18de80fd04110756becaed214656642388401d
Platform Version: 18.0

STEPS
1. From the DUT send an MMS to multiple recipients
2. Receive an answer from one of the recipients
3. Check where this answer is stored and shown

EXPECTED
DuT shows only one thread with the original MMS and the answer should be shown on the original MMS thread.

ACTUAL
DuT shows two different threads, the original group MMS sent and a new one with the answer received

Link to failed test case:
https://moztrap.mozilla.org/manage/case/7974/

Comment 10

[Julien Wajsberg [:julienw]]

I'd say that we should be able to map an array of recipients to a thread id. The array of recipients should be matched whatever the order is.

This is a Gecko problem, I'll try to investigate and see if I can come up with a patch.

Comment 11

[Julien Wajsberg [:julienw]]

of course this is what's already done. I've switched DEBUG mode in dom/mobilemessage/src/ril/MobileMessageDatabaseService.js.

Comment 12

[Borja Salguero [:borjasalguero]]

It seems that all these bugs are related... Wait until next week for fixing them with Taipei Team!.

Comment 13

[bhavana bajaj [:bajaj]]

Triage Comment : Passing on to David to help check the status and have the right assignee here.

Comment 14

[Julien Wajsberg [:julienw]]

As decided in Taipei, we won't probably keep any of the "groupmms" bugs in leo.

Comment 15

[Alex Keybl [:akeybl]]

leo? given comment 14

Comment 16

[Dietrich Ayala (:dietrich)]

Per triage decision, all groupmms moving to koi? as the feature is no longer in leo.

Comment 17

[KTucker [:KTucker][Inactive 3/4/2016]]

This also occurs when sending a group SMS on buri v1.2. The sent group messages are split up into different threads instead of being in one thread.

Environmental Variables
Build ID: 20130916040205
Gecko: http://hg.mozilla.org/mozilla-central/rev/c4bcef90cef9
Gaia: a0079597d510ce8ea0b9cbb02c506030510b9eeb
Platform Version: 26.0a1

Comment 18

[KTucker [:KTucker][Inactive 3/4/2016]]

Please disregard comment 17. After further research it was determined that the SMS is a separate issue and should be logged accordingly.

Comment 19

[Noemí Freire (:noemi)]

All groupmms moving to comms backlog, Bug 891754

Comment 20

[Julien Wajsberg [:julienw]]

ckreinbring> this feature was removed from v1.2.

Comment 21

[Kevin Grandon :kgrandon]

Julien - I noticed that if I am involved in a multi-user message with two android phones which one android phone initiates, they will not be grouped in the same thread. They will be in two different threads. I believe the message should be using MMS as it's multi-user.

I am just wondering if that is due to this bug, or if I should file a new bug?

Comment 22

[Julien Wajsberg [:julienw]]

Yeah, this is this bug (or at least closely related).

As you might see we stopped working on groupmms and put it in a "sort of working" state, with clear issues as the one you see.

The main technical issue is bug 880251 -- we'd need to find out how other platforms are dealing with this.

Comment 23

[BMO Automation]

Firefox OS is not being worked on