Closed Bug 1118701 Opened 6 years ago Closed 6 years ago

[FFOS2.0][Woodduck][STK]There has no indication given to the user during the REFRESH

Categories

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

defect

Tracking

(blocking-b2g:-, b2g-v2.0 affected, b2g-v2.0M affected, b2g-v2.1 affected, b2g-v2.2 affected)

RESOLVED INVALID
blocking-b2g -
Tracking Status
b2g-v2.0 --- affected
b2g-v2.0M --- affected
b2g-v2.1 --- affected
b2g-v2.2 --- affected

People

(Reporter: sync-1, Assigned: selee)

References

Details

Attachments

(1 file)

DEFECT DESCRIPTION:[STK]There has no indication  given to the user during the REFRESH 
 
  REPRODUCING PROCEDURES:
 1. Load a simcard to the phone which can send "refresh".
 2. Executed "refresh-STM07002",during command refresh executed, there has no indication  given to the user.--ko
 
  EXPECTED BEHAVIOUR:
 2. Executed "refresh-STM07002",during command refresh executed, there should have indication  given to the user.
 
 Reporter's phone number: 0752-2639695
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 
  For FT PR, Please list reference mobile's behavior:
Hi Sean,
Could you please help to check the problem? Thanks!
blocking-b2g: --- → 2.0M+
Flags: needinfo?(selee)
Hi Pengfei,

This patch provides a feature that a notification will be shown when a refresh command is received.

Thanks.
Flags: needinfo?(selee) → needinfo?(pengfei.huang.hz)
Hi Sean lee,
  Your patch works well. But after refreshing, the notification still exist.
  Besides there is a big question about operator pushing message. VAL confirm different operators may push various message to phone which including some message(refresh command will executed) shouldn't inform user. And we couldn't confirm what exactly type of refresh operator will push. So, user may receive notifications suddenly.
  I will check android code to see how they implements this feature.Thanks
Flags: needinfo?(pengfei.huang.hz)
Flags: needinfo?(selee)
Assignee: nobody → selee
Flags: needinfo?(selee)
Hi Pengfei,

In my patch at comment 2, this is a sample patch for you.
The notification will be closed if the card state is changed to 'ready' .
If the condition of notification closed is confirmed, the design can be improved then.
(In reply to Sean Lee [:seanlee] from comment #4)
> Hi Pengfei,
> 
> In my patch at comment 2, this is a sample patch for you.
> The notification will be closed if the card state is changed to 'ready' .
> If the condition of notification closed is confirmed, the design can be
> improved then.

  Yes, I try your patch on comment3. We receive the notification but it isn't remove after a while when refresh command executed. I add some DUMP log and will try again tonight.
    
+    DUMP('command.commandQualifier:' + message.command.commandQualifier);
     if (message.command.commandQualifier === 0x04) {
       createResetNotification();
       if (!this.stkResetHandler) {
         this.stkResetHandler = function() {
           var iccManager = window.navigator.mozIccManager;
           var iccCard = iccManager.getIccById(message.iccId);
           var cardState = iccCard.cardState;
+          DUMP('cardState:' + cardState);
           if (cardState === 'ready') {
             resetNotification.close();
           }
         };


  I ask android developer and they confirm there is no any code to show the refresh icon at application layer but it exists on android. So, android developer guess it's Android OS implements that but they don't know the detail too.
In my sample code (this is not a final design, just a sample.),
the notification will be closed when the two case happened:
1. User closes it manually.
2. The card state is changed to 'ready' .

Could you help to confirm whether 'cardstatechange' event is triggered?

If it is not triggered, the notification not closed automatically is expected.
(In reply to Sean Lee [:seanlee] from comment #6)
> In my sample code (this is not a final design, just a sample.),
> the notification will be closed when the two case happened:
> 1. User closes it manually.
> 2. The card state is changed to 'ready' .
> 
> Could you help to confirm whether 'cardstatechange' event is triggered?
> 
> If it is not triggered, the notification not closed automatically is
> expected.

  The event not triggered.
Hi Sean,
  VAL can't find another good idea for us to implements so that we can ignore this bug.
  Many thanks.
Per comment 8, this issue can be closed.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
blocking-b2g: 2.0M+ → ---
blocking-b2g: --- → -
Dear Wayne Teng,
   we can close this bug, please see TCL bug862838.
   Thanks
You need to log in before you can comment on or make changes to this bug.