1038806 - Avoid calling `init` panel functions twice in Contacts

Status: RESOLVED WONTFIX

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

Description

[Sergi Mansilla [:sergi] (Telenor) [PTO]]

`initSettings`, `initDetails` and `initForm` could all be called twice (for example by quickly tapping on the settings button inside Contacts) and then the callback would be executed twice, with unexpected consequences.

Comment 1

[Sergi Mansilla [:sergi] (Telenor) [PTO]]

Attachment: Github PR

This patch makes the three init functions (initDetails, initForm and initSettings) self-contained, without the need of external state variables, and avoids the possibility of loading dependencies twice or, more importantly, executing callbacks twice.

Comment 2

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8456275 [details] [review]
Github PR

Just a couple of nit on the github pr.

Could we add unit test to check that we call the functions just once?

Thanks!

Comment 3

[Sergi Mansilla [:sergi] (Telenor) [PTO]]

Comment on attachment 8456275 [details] [review]
Github PR

Corrected nits and added tests that check that the callback is called twice.

Comment 4

[Sergi Mansilla [:sergi] (Telenor) [PTO]]

About testing for the function to be run just once, since parts in the contacts test use these methods, it is not possible to test if it is run once, since it has been memoized already. I accept suggestions on how to test it. The only way I see right now is to make an entire new suit only for these cases. Is that acceptable?

Comment 5

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8456275 [details] [review]
Github PR

Forwarding the review to Jose to speed up the process.

Comment 6

[Jose Manuel Cantera]

As per conversation with Sergi we are going to rely on a simpler solution that does not involve functions that return functions

Comment 7

[BMO Automation]

Firefox OS is not being worked on