wallpanel-android icon indicating copy to clipboard operation
wallpanel-android copied to clipboard

fix: check isInitialized on lateinit prop

Open kaqq opened this issue 2 years ago • 3 comments

fix for

FATAL EXCEPTION: main
11-08 17:57:48.587  9310  9310 E AndroidRuntime: Process: xyz.wallpanel.app, PID: 9310 11-08 17:57:48.587  9310  9310 E AndroidRuntime: kotlin.UninitializedPropertyAccessException: lateinit property dialogUtils has not been initialized
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at xyz.wallpanel.app.utils.InternalWebClient.getDialogUtils(InternalWebClient.kt:24)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at xyz.wallpanel.app.utils.InternalWebClient.onReceivedSslError(InternalWebClient.kt:86)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at WV.M6.run(chromium-TrichromeWebViewGoogle.aab-stable-599311131:182)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at android.os.Handler.handleCallback(Handler.java:883)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at android.os.Handler.dispatchMessage(Handler.java:100)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at android.os.Looper.loop(Looper.java:214)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at android.app.ActivityThread.main(ActivityThread.java:7403)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at java.lang.reflect.Method.invoke(Native Method)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:502)
11-08 17:57:48.587  9310  9310 E AndroidRuntime:        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:980)

Not sure if this is a proper fix, based on: https://kotlinlang.org/docs/properties.html#overriding-properties

kaqq avatar Nov 08 '23 17:11 kaqq

Deploy Preview for wallpanel canceled.

Name Link
Latest commit b9c0e709d71d72bfef4997341abfaa23da7cd016
Latest deploy log https://app.netlify.com/sites/wallpanel/deploys/654bc24175a72600087a3719

netlify[bot] avatar Nov 08 '23 17:11 netlify[bot]

Hey! Thanks for the PR, sorry I have totally missed it. Seems to be plausible and I will test it quickly this weekend before merging 👍🏻

TheTimeWalker avatar Feb 24 '24 16:02 TheTimeWalker

Hey! Thanks for the PR, sorry I have totally missed it. Seems to be plausible and I will test it quickly this weekend before merging 👍🏻

Some month ago. :) No problem, but is it possible to merge and perhaps rebuild this?