981988 - [Sora][Message][MMS]The media file add to the exist message content .

Status: RESOLVED DUPLICATE of bug 936729

(Firefox OS Graveyard :: Gaia::SMS, defect, P1)

Description

[sync-1]

Firefox OS v1.3
 Mozilla build ID: 20140226004002
 
 DEFECT DESCRIPTION:
 ->The media file add to the exist message content 
 
  REPRODUCING PROCEDURES:
 ->Enter"Message"to create a new MMS,add a media file A successfully;
 ->Tap home key back to idle,enter"Gallary" to select a media file B share to message;
 ->You can find the A and B in the same page.(ko)
 
 note:Beetle lite FF also have this behavior.
 
  EXPECTED BEHAVIOUR:
 ->Should create a another new MMS.
 
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:100%
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[Julien Wajsberg [:julienw]]

Yeah, that's bug 936729: we need to use an inline activity for the sharing activity. Currently it's a window activity.

That's not a regression, this behaves like this since the start.

Duping to Bug 936729.

Comment 2

[Jack Liu]

Hi Julien, 

I think this is not duplicated one of bug#936729.

The emphasis of it is:
There is already a MMS-1 under editing and with one media file A attached, if you share another media file B from either Gallery or Video/Music player, it should create new MMS-2 and attach B in the new message. But now it doesn't create new message, just attach B to MMS-1.

Comment 3

[Steve Chung [:steveck]]

(In reply to Jack Liu from comment #2)
> Hi Julien, 
> 
> I think this is not duplicated one of bug#936729.
> 
> The emphasis of it is:
> There is already a MMS-1 under editing and with one media file A attached,
> if you share another media file B from either Gallery or Video/Music player,
> it should create new MMS-2 and attach B in the new message. But now it
> doesn't create new message, just attach B to MMS-1.

Julien means we currently only use window activity while sharing, which will launch the original message app instead of creating another message app instance for sharing. Once we fix bug 936729, the share action will use inline acttivity and there will be 2 independent message instance with their own state.

Comment 4

[Julien Wajsberg [:julienw]]

Yep, that's it. I agree that's not the same issue, but the fix to bug 936729 will also fix this one.

If you prefer, we can keep this bug open and add a dependency.

Comment 5

[Jack Liu]

How about the solution as Android:

Prompts user to discard or save previous message as draft, no need to create two independent instances.

Comment 6

[Julien Wajsberg [:julienw]]

Of course we can workaround the issue. That would be throwaway work though because we really want to move this activity to inline for a variety of good reasons (and not only bug 936729).