Fibaroextra 0.955 - event error

[Neo Andersson 230]

Jan, can you please check this..

It throws me error when updated to latest Fibaroextra, and it was working months before update.

 

Please login or register to see this code.

main.lua:15: attempt to call a nil value (field 'event')

Edited January 22, 2023 by Neo Andersson

[jgab 2,641]

27 minutes ago, Neo Andersson said:

Jan, can you please check this..

It throws me error when updated to latest Fibaroextra, and it was working months before update.

 

Please login or register to see this code.

main.lua:15: attempt to call a nil value (field 'event')

 

Well, you were lucky in the past. 

fibaro.event(...) should not be used before QuickApp:onInit() is called. The reason is that the order of the loading of the files of the QuickApp is not guaranteed.

If 'main' is loaded before 'fibaroExtra' you can't call fibaro.event(...) on top-level in 'main' because fibaro.event(...) is not defined yet.

 

The way to do it is either lifting in the code above inside QuickApp:onInit() or wrap it in a function called from :onInit()

Ex.

 

Please login or register to see this code.

 

Edited January 22, 2023 by jgab

[Neo Andersson 230]

3 minutes ago, jgab said:

 

Well, you were lucky in the past. 

fibaro.event(...) should not be used before QuickApp:onInit() is called. The reason is that the order of the loading of the files of the QuickApp is not guaranteed.

If 'main' is loaded before 'fibaroExtra' you can't call fibaro.event(...) on top-level in 'main' because fibaro.event(...) is not defined yet.

 

The way to do it is either lifting in the code above inside QuickApp:onInit() or wrap it in a function called from :onInit()

Ex.

 

Please login or register to see this code.

 

ohh okay, thanks, i think i cant update all my QAs to latest Fibaroextra..as i used event function calls many times like in the example...

[jgab 2,641]

2 minutes ago, Neo Andersson said:

ohh okay, thanks, i think i cant update all my QAs to latest Fibaroextra..as i used event function calls many times like in the example...

Well, if you do that you are playing file-order-roulette and your existing QAs may start to throw errors at restarts. You have just been lucky so far.

Just now, jgab said:

Well, if you do that you are playing file-order-roulette and your existing QAs may start to throw errors at restarts. You have just been lucky so far.

 

It's actually a general rule when developing QAs. If you have a multi-file QA, don't use functions defined in the other files until your QuickApp:onInit() has been called.

[Neo Andersson 230]

2 minutes ago, jgab said:

Well, if you do that you are playing file-order-roulette and your existing QAs may start to throw errors at restarts. You have just been lucky so far.

 

It's actually a general rule when developing QAs. If you have a multi-file QA, don't use functions defined in the other files until your QuickApp:onInit() has been called.

Thank you for the advice..

[Pica2017 2]

]9873