944600 - [MMS] 4 different issue about sending and receiving

Status: RESOLVED DUPLICATE of bug 943460

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

Description

[Walter Chen[:ypwalter][:wachen]]

Most up-to-date PVT user build on 11/28/2013
with v1.2 base & MozRIL of Buri
failed on daily smoke test
Gaia:     0d57ec2801ae125ec855a19cf956ab118660d694
Gecko:    http://hg.mozilla.org/mozilla-central/rev/a5e7f611546f
BuildID   20131128040201
Version   28.0a1
ro.build.version.incremental=eng.archermind.20131114.105818

Step Set 1:
a. launch messages app
b. attached a picture to the message

expected result:
the picture has preview before you send it
actual result:
there is not preview

Step Set 2: (Do Set 1 first)
c. send the message and redo a-c again

expected result:
the image showed up
actaul result:
no image shown

Step Set 3: (Do set1 -> set2 first)
d. tap the top-left button "<"
e. tap the message just sent

expected result:
1.the message title shows the number and content
2.you can get back to message contents
actual result:
the tile is empty and you can't go back

Step Set 4:
f. kill the app
g. relaunch the message app
h. go into that message

expected result:
2 message should be sent and it should show 2 message
actual result:
only one sent

Comment 1

[Jason Smith [:jsmith]]

Please file separate bugs for each issue you hit - don't file a single bug representing multiple bugs.

Comment 2

[Walter Chen[:ypwalter][:wachen]]

This is a continuous action
It should be just one bug.

Comment 3

[Julien Wajsberg [:julienw]]

Hi,

The 2 first steps have already a bug for this: bug 943460.

I'd like that you try to separate steps 3 and 4 into 2 different bugs, it would be easier to fix.

Thanks Walter.

Comment 4

[Jason Smith [:jsmith]]

(In reply to Walter Chen[:ypwalter][:wachen] from comment #2)
> This is a continuous action
> It should be just one bug.

No. Continuous action does not justify a single bug - it'll be confusing to track. When a fault is hit, you should have a bug specific to that issue. Each fault found represents a single bug with specific STR.

Comment 5

[Walter Chen[:ypwalter][:wachen]]

Jason, you can be more helpful than this. You do know that, right?

Here are some suggestion that you can do better:
1. If a fix doesn't fix a thing, that means "reopen", and don't do "resolved fixed". If you want to do "resolved fixed", get me a patch..

2. "I don't understand this bug", "I don't fully understand this bug", and so on... Those are bad sentences. You can be more helpful to just watch the video we provided, look at the picture we provide, or just go through STR. If you still have other questions. Here is the solution -> tell us what you don't understand if you do follow all the things. Don't tell me that others needs to know details from me. I gave out the record and anyone can ask me question if he tries to reproduce and fails.

3. "Resolved invalid" is not helping at all, either. Come on, you can do better than a nonsense people. If you think this should be separate, you can help out by separate these bugs and set up the standard.

4. Back out changes is not a good practice, either. Jason, can you negotiate with people before you do something? Once we agree and get to know each others' thoughts, we can agree on something like finding the root causes or something else that we can both agree.

However, doing things like this is not good at all. If you have free time to just saying "I don't understand" or "resolved invalid" or "backing out other people's patches"... Spend more time doing meaningful or helpful stuffs will be even better.

You graduated from RIT. I think you can do much better than what you are doing now. Please be more helpful. I will appreciate your help. Perhaps, others can understand and find your works more valuable than ever.


And, yes, Julien, I will be separating the bugs, and let you know. I was so much in a hurry to complete some jobs. I apologize for that. I was finally cover like almost 2 smoke test runs with other stuffs. Thanks for your help. I will dup this to the bug you opened.

Comment 6

[Walter Chen[:ypwalter][:wachen]]

Jason,
What I commented in comment 5 wasn't intended to mess with you or any intention for making things worse. I'd like to apologize here for that I shouldn't mention your school and other related personal stuffs. Sorry for some bad words in comment 5, but what I meant was you are good in every aspect but we can communicate a little bit more.

Comment 7

[Julien Wajsberg [:julienw]]

Hey Walter,

I'd like to comment about backouts, because in comment 5 you implies that backouts are bad things. Well, they are not. If a backout is right, this makes the phone work again. If a backout is wrong, it's super easy to revert it.

Thanks for filing new bugs :)