907507 - [Buri][PLMN]Networt name in PLMN list is not same as idle show

Status: RESOLVED WONTFIX

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

Description

[sync-1]

AU_LINUX_GECKO_ICS_STRAWBERRY.01.01.00.019.190
 Firefox os  v1.1
 Mozilla build ID:20130812041203
 
 DEFECT DESCRIPTION:
 Networt name in PLMN list is not same as idle show.
 
  REPRODUCING PROCEDURES:
 Case1:
 SIM with EF_SPN 00 
 When register on HPLMN, in idle only SPN is shown, but in PLMN search list, static name is shown;-->KO
 
 Case2:
 SIM with OPL/PNN
 When register on PLMN that in OPL list, in idle only PNN is shown, but in PLMN search list, static name is shown;-->KO
 
  EXPECTED BEHAVIOUR:
 Network name in search list should be same as idle
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 5/5
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: radio log

Comment 3

[sync-1]

Clone from brother

Comment 4

[sync-1]

Attachment: QXDM log

Comment 5

[yangshjuan]

Hi, 
for test case3: short name is showed instead of longname in plmn list.

I find in newlistitem() in gaia/app/setting/carrier.js for 852035. Why? 
-name.textContent = network.longName;
+name.textContent = network.shortName || network.longName;

I need to change to :
name.textContent = network.longName || network.longName;

Will it be ok?

Comment 6

[Andrew Overholt [:overholt]]

Evelyn, could this be a regression from bug 852038?

I'm also wondering if comment 5 meant to say:

name.textContent = network.longName || network.shortName; // not longName twice

?

Comment 7

[Andrew Overholt [:overholt]]

[e-mail], please see my second question in comment 6.

Comment 8

[Preeti Raghunath(:Preeti)]

Andrew,

Assigning this bug to you for further analysis, since you  are already looking into it.

Comment 9

[yangshjuan]

Hi, 
I mean, at first the code was:
name.textContent = network.longName;

later, for bug 852035, it is modified like following:
name.textContent = network.shortName || network.longName;

now, bug 907507 asks to show longName first. i need to change to:
name.textContent = network.longName || network.shortName;

Will this affect bug 852035?

Comment 10

[bhavana bajaj [:bajaj]]

Can someone help confirm if this is the same behaviour in 1.0.1 .If its a regression from bug 852038, then it would be and I think it is ok to ship it in the same way for 1.1 as well given where we are.(needinfo  	
Evelyn Hung )

Comment 11

[Evelyn Hung]

No, it's not a regression from bug 852038.

Comment 12

[Evelyn Hung]

(In reply to yangshjuan from comment #9)
> Hi, 
> I mean, at first the code was:
> name.textContent = network.longName;
> 
> later, for bug 852035, it is modified like following:
> name.textContent = network.shortName || network.longName;
> 
> now, bug 907507 asks to show longName first. i need to change to:
> name.textContent = network.longName || network.shortName;
> 
> Will this affect bug 852035?

I don't see any patch for bug 852035. From the comments on that bug, it seems a totally different problem that trying to do ADN editing.
I believe switching network.longName and network.shortName doesn't matter in this case. Would you like to provide a patch?

Comment 13

[Preeti Raghunath(:Preeti)]

Evelyn

What is the user impact if the bug is not fixed. Sounds like a naming convention.

Comment 14

[Evelyn Hung]

I'm not sure about the user impact... From my point of view, yes, it's more like a convention.

Comment 15

[Preeti Raghunath(:Preeti)]

very minimal user impact and hence minus

Comment 16

[Evelyn Hung]

clear ni flag, and label good first bug for someone who is looking for a easier start.

Comment 17

[Evelyn Hung]

I'd like to be a mentor of this bug.

Comment 18

[Anindya-Pandey]

Hi,
I'm new to open source.
I'm looking for my first bug.
Can u assign this bug to me?

Comment 19

[Mike Hoye [:mhoye]]

Hi, Anindya-Pandey; welcome aboard. We'd love to assign this to you, and as soon as you've got a patch ready to go we can do that. 

Building FirefoxOS can be a challenge, but our documentation is pretty good. You can get started here:

https://developer.mozilla.org/en-US/Firefox_OS/Building_and_installing_Firefox_OS

and as soon as you're able to build from source successfully, you'll be on your way.

Evelyn, listed in Mentor's field above, can answer questions about what this change looks like. Thanks for your effort, and good luck!

Comment 20

[Anindya-Pandey]

Hi, Mike Hoye. I've already built and installed the Gaia user interface. What else do I need?

Comment 21

[Mike Hoye [:mhoye]]

Evelyn can answer that question, I think - I've called her attention to this with the "need more info" box you see below, and selecting "mentor". 

Evelyn, what do you think?

Comment 22

[BMO Automation]

Firefox OS is not being worked on