Closed Bug 876003 Opened 11 years ago Closed 11 years ago

[MMS] Sending mms to unknown contact via activity will lose the number

Categories

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

Product:
Firefox OS Graveyard
Component:
Gaia::SMS
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:leo+, b2g18 fixed)

Status:
RESOLVED FIXED
Project Flags:
blocking-b2g leo+
Tracking Flags:
Tracking Status
b2g18 --- fixed

People

(Reporter: rwaldron, Assigned: rwaldron)

References

(
URL
)

Details

Attachments

(1 file)

Fix for 876003
45 bytes, text/x-github-pull-request
gnarf
: review+
Details | Review 
This was overlooked because it was not a tested activity handler case.
Assignee: nobody → waldron.rick
clarified on irc. broken. blocking+.
blocking-b2g: --- → leo+
From IRC: 

Imagine a third party app that says "send messages to a number" and you enter "999" and tap it's silly "go" button and that initiates the "new" activity handled by Messages app—the number "999" will not be in the recipients
Attached file Fix for 876003 — 

        Attachment #754008 -
        Flags: review?(gnarf37)

    
    

    
  
Comment on attachment 754008 [details] [review]
Fix for 876003

Couple of test related requests on the pull.

        Attachment #754008 -
        Flags: review?(gnarf37) → review+

    
    

    
  
r=me with tests still passing obviously

    
    

    
  
master: https://github.com/mozilla-b2g/gaia/commit/fc32391128052b42416427cc2a2c061150210c70
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED

    
  
Depends on: 867885, 876758

    
    

    
  
the two dependencies are for bugs that are required to pick this to train

    
    

    
  
Uplifted fc32391128052b42416427cc2a2c061150210c70 to:
v1-train: 6a490bacd2aeaee0cec46c6f741c68a57aa69f59

          status-b2g18:
          --- → fixed

    
  
Flags: in-moztrap?

    
  
Flags: in-moztrap? → in-moztrap+

    
  
URL: https://moztrap.mozilla.org/manage/ca...
QA Contact: amiller
Attachment mime type: text/plain → text/x-github-pull-request

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: