896346 - [Buri][STK][DUAL IMSI]no TR return for provide local information command

Status: RESOLVED DUPLICATE of bug 817952

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

Description

[sync-1]

AU_LINUX_GECKO_ICS_STRAWBERRY.01.01.00.019.158
 Firefox os  v1.1
 Mozilla build ID:20130709070206
 
 
 
 DEFECT DESCRIPTION:
 [STK][DUAL IMSI]no TR return for provide local information command
 
  REPRODUCING PROCEDURES:
 1 check scenario 31 about roaming broker with VIVO dual IMSI card ,
 network configuration:
 - Two (2) at 2100 MHz 3G: 234/10 (O2) and 234/50 (JT-Wave)
 - One (1) 2G network in 900 MHz: 234/10 (O2)
 - One (1) 2G network on 1800 MHz: 234/15 (Vodafone)
 - Location Update Reject: Roaming Not Allowed in This Area Location
 - PIN 1 Enabled
 2 after send provide local inofrmation ,no TR return--->KO
 
  EXPECTED BEHAVIOUR:
 provide local information can execute rightly,and ME's behavior should be same as roaming broker word's requirement.(see the attached file)
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: Roaming_Broker word

Comment 3

[sync-1]

Clone from brother

Comment 4

[sync-1]

Attachment: QXDM log

Comment 5

[Lukas Blakk [:lsblakk] use ?needinfo]

Given that this depends on a bug which is not a blocker it's hard to see how this could be a blocker.

Comment 6

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

Can you verify if patch https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this issue?

Comment 7

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

(In reply to Fernando R. Sela [:frsela] from comment #6)
> Can you verify if patch
> https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this
> issue?

Patch landed in master so you can test with master branch directly

Comment 8

[buri.blff]

(In reply to Fernando R. Sela [:frsela] from comment #7)
> (In reply to Fernando R. Sela [:frsela] from comment #6)
> > Can you verify if patch
> > https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this
> > issue?
> 
> Patch landed in master so you can test with master branch directly

thanks, we will verify this patch, thanks.

Comment 9

[buri.blff]

(In reply to Fernando R. Sela [:frsela] from comment #7)
> (In reply to Fernando R. Sela [:frsela] from comment #6)
> > Can you verify if patch
> > https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this
> > issue?
> 
> Patch landed in master so you can test with master branch directly

can you tell me the mozilla buid id??
our mozilla build id is:20130730070228
we find there is too much difference to add the patch

Comment 10

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

(In reply to buri.blff from comment #9)
> (In reply to Fernando R. Sela [:frsela] from comment #7)
> > (In reply to Fernando R. Sela [:frsela] from comment #6)
> > > Can you verify if patch
> > > https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this
> > > issue?
> > 
> > Patch landed in master so you can test with master branch directly
> 
> can you tell me the mozilla buid id??
> our mozilla build id is:20130730070228
> we find there is too much difference to add the patch

Only two files had been changed (https://github.com/mozilla-b2g/gaia/pull/11310/files), this patch cann't be merged?

Comment 11

[buri.blff]

(In reply to Fernando R. Sela (no CC, needinfo please) [:frsela] from comment #10)
> (In reply to buri.blff from comment #9)
> > (In reply to Fernando R. Sela [:frsela] from comment #7)
> > > (In reply to Fernando R. Sela [:frsela] from comment #6)
> > > > Can you verify if patch
> > > > https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this
> > > > issue?
> > > 
> > > Patch landed in master so you can test with master branch directly
> > 
> > can you tell me the mozilla buid id??
> > our mozilla build id is:20130730070228
> > we find there is too much difference to add the patch
> 
> Only two files had been changed
> (https://github.com/mozilla-b2g/gaia/pull/11310/files), this patch cann't be
> merged?


can you merge the patch to v1.1??

Comment 12

[Fernando R. Sela (no CC, needinfo please) [:frsela]]

(In reply to buri.blff from comment #11)
> (In reply to Fernando R. Sela (no CC, needinfo please) [:frsela] from
> comment #10)
> > (In reply to buri.blff from comment #9)
> > > (In reply to Fernando R. Sela [:frsela] from comment #7)
> > > > (In reply to Fernando R. Sela [:frsela] from comment #6)
> > > > > Can you verify if patch
> > > > > https://bugzilla.mozilla.org/attachment.cgi?id=784883 from 817952 fixes this
> > > > > issue?
> > > > 
> > > > Patch landed in master so you can test with master branch directly
> > > 
> > > can you tell me the mozilla buid id??
> > > our mozilla build id is:20130730070228
> > > we find there is too much difference to add the patch
> > 
> > Only two files had been changed
> > (https://github.com/mozilla-b2g/gaia/pull/11310/files), this patch cann't be
> > merged?
> 
> 
> can you merge the patch to v1.1??

Hi,

after cherry-picking in v1.1.0hd:

 git cherry-pick -m 1 32c4349d1194e5588f093c6cd3896049c8c2899d

there is a conflict in the file:

 both modified:      apps/system/js/icc_events.js

Anyway this is a trivial conflict you can solve removing the <<<<<<< HEAD, ========= and >>>>>>>>> ... lines in conflicting file:

 computita:gaia (<cherry-pick>test^)$ git diff apps/system/js/icc_events.js
 diff --cc apps/system/js/icc_events.js
 index a5b4193,e3efdad..0000000
 --- a/apps/system/js/icc_events.js
 +++ b/apps/system/js/icc_events.js
 @@@ -86,6 -86,15 +86,18 @@@ var icc_events = 
       });
     },
  
 ++<<<<<<< HEAD
 ++=======
 +   handleLanguageSelectionEvent:
 +     function icc_events_handleLanguageSelectionEvent(evt) {
 +       DUMP(' STK Language selection = ' + evt.settingValue);
 +       this.downloadEvent({
 +         eventType: icc._icc.STK_EVENT_TYPE_LANGUAGE_SELECTION,
 +         language: evt.settingValue.substring(0, 2)
 +       });
 +   },
 + 
 ++>>>>>>> 32c4349... Merge pull request #11310 from frsela/STK/Bug817952
     register: function icc_events_register(eventList) {
       DUMP('icc_events_register - Events list:', eventList);
       for (var evt in eventList) {