network: show rationale for permissions (fixes #2089)

[Okuro3499]

fixes #2089

Screen_recording_20241111_194922.mp4

[xyb994]

I wonder if jumping to location permission (if not granted already) is the optimal flow. It sort of reminded me of myPlanet jumping to "Usage access" directly without warning upon opening. Would a pop up message asking if the user would like to be taking to location permission settings for the app to scan for wifi SSID be better?

Aside from that, it seem to crash when I try to scan -> get to the SSID list -> click on cancel -> use the wifi icon to scan again

FATAL EXCEPTION: main
Process: io.treehouses.remote, PID: 21886
java.lang.RuntimeException: Error receiving broadcast Intent { act=android.net.wifi.SCAN_RESULTS flg=0x4000010 pkg=io.treehouses.remote (has extras) } in io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment$wifiBroadcastReceiver$1@8832dbf
	at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1581)
	at android.app.-$$Lambda$LoadedApk$ReceiverDispatcher$Args$_BumDX2UKsnxLVrE6UJsJZkotuA.run(Unknown Source:2)
	at android.os.Handler.handleCallback(Handler.java:938)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:233)
	at android.app.ActivityThread.main(ActivityThread.java:8068)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:631)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:978)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String android.content.Context.getPackageName()' on a null object reference
	at android.widget.Toast.<init>(Toast.java:167)
	at android.widget.Toast.makeText(Toast.java:492)
	at android.widget.Toast.makeText(Toast.java:480)
	at io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment.scanFailure(WifiDialogFragment.kt:127)
	at io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment.access$scanFailure(WifiDialogFragment.kt:21)
	at io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment$wifiBroadcastReceiver$1.onReceive(WifiDialogFragment.kt:85)
	at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1566)
	at android.app.-$$Lambda$LoadedApk$ReceiverDispatcher$Args$_BumDX2UKsnxLVrE6UJsJZkotuA.run(Unknown Source:2) 
	at android.os.Handler.handleCallback(Handler.java:938) 
	at android.os.Handler.dispatchMessage(Handler.java:99) 
	at android.os.Looper.loop(Looper.java:233) 
	at android.app.ActivityThread.main(ActivityThread.java:8068) 
	at java.lang.reflect.Method.invoke(Native Method) 
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:631) 
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:978) 

[Okuro3499]

I wonder if jumping to location permission (if not granted already) is the optimal flow. It sort of reminded me of myPlanet jumping to "Usage access" directly without warning upon opening. Would a pop up message asking if the user would like to be taking to location permission settings for the app to scan for wifi SSID be better?

Aside from that, it seem to crash when I try to scan -> get to the SSID list -> click on cancel -> use the wifi icon to scan again

FATAL EXCEPTION: main
Process: io.treehouses.remote, PID: 21886
java.lang.RuntimeException: Error receiving broadcast Intent { act=android.net.wifi.SCAN_RESULTS flg=0x4000010 pkg=io.treehouses.remote (has extras) } in io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment$wifiBroadcastReceiver$1@8832dbf
	at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1581)
	at android.app.-$$Lambda$LoadedApk$ReceiverDispatcher$Args$_BumDX2UKsnxLVrE6UJsJZkotuA.run(Unknown Source:2)
	at android.os.Handler.handleCallback(Handler.java:938)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:233)
	at android.app.ActivityThread.main(ActivityThread.java:8068)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:631)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:978)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String android.content.Context.getPackageName()' on a null object reference
	at android.widget.Toast.<init>(Toast.java:167)
	at android.widget.Toast.makeText(Toast.java:492)
	at android.widget.Toast.makeText(Toast.java:480)
	at io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment.scanFailure(WifiDialogFragment.kt:127)
	at io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment.access$scanFailure(WifiDialogFragment.kt:21)
	at io.treehouses.remote.fragments.dialogfragments.WifiDialogFragment$wifiBroadcastReceiver$1.onReceive(WifiDialogFragment.kt:85)
	at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1566)
	at android.app.-$$Lambda$LoadedApk$ReceiverDispatcher$Args$_BumDX2UKsnxLVrE6UJsJZkotuA.run(Unknown Source:2) 
	at android.os.Handler.handleCallback(Handler.java:938) 
	at android.os.Handler.dispatchMessage(Handler.java:99) 
	at android.os.Looper.loop(Looper.java:233) 
	at android.app.ActivityThread.main(ActivityThread.java:8068) 
	at java.lang.reflect.Method.invoke(Native Method) 
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:631) 
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:978) 

the scanning crash could be fixed in another issue. for the pop up message we already have that as shown in the video the rationale is shown for gps

Screen_recording_20241125_211729.mp4