912814 - [Settings] User can change the PIN code in'Change PIN' menu when SIM state is 'pukRequired'

Status: RESOLVED DUPLICATE of bug 897310

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

Description

[Leo]

Attachment: simcard_dialog.patch

[STR]
1. SIM lock is enable
2. Try to unlock and input the wrong PIN code 3 times
3. Close the PUK screen
4. Go to 'Change PIN'

You can chage the PIN code without PUK code


I uploaded a patch. 
User can change PIN code after input PUK code.

Please check it

Comment 1

[Evelyn Hung]

Comment on attachment 799909 [details] [diff] [review]
simcard_dialog.patch

r=me. Thanks for the patch, but I'm afraid that it's not properly solve the bug.
This patch force displayed screen to be 'unlock' but doesn't update user action to correct 'unlock' state, that may lead to an inconsistency. Furthermore, this transition is not smooth and I think it's better to redirect user to unlock puk screen when the user clicking on "change pin" button.
BTW, please use `IccHelper.cardState` instead of `this.mobileConnection.cardState`, we do a run-time detection in IccHelper to know which interface we should query.

Comment 2

[Leo]

Thanks for review.
`IccHelper.cardState` do not use in v1-train.

This issue should be fixed in the next version of leo.
This patch was temporary to fix quickly and not applied in leo.

Please check this issue.