fix(android): Add another check in ConnectivityReciever to make sure we have ACCESS_WIFI_STATE permission (#379 by @sweggersen)

Author: sweggersen

android/src/main/java/com/reactnativecommunity/netinfo/ConnectivityReceiver.java

@@ -6,15 +6,12 @@
  */
 package com.reactnativecommunity.netinfo;
 
-import android.Manifest;
 import android.content.Context;
-import android.content.pm.PackageManager;
 import android.net.ConnectivityManager;
 import android.net.wifi.WifiInfo;
 import android.net.wifi.WifiManager;
 import android.telephony.TelephonyManager;
 
-import androidx.core.content.ContextCompat;
 import androidx.core.net.ConnectivityManagerCompat;
 
 import com.facebook.react.bridge.Arguments;
@@ -115,8 +112,10 @@ private WritableMap createConnectivityEventMap(@Nullable final String requestedI
         WritableMap event = Arguments.createMap();
 
         // Add if WiFi is ON or OFF
-        boolean isEnabled = mWifiManager.isWifiEnabled();
-        event.putBoolean("isWifiEnabled", isEnabled);
+        if (NetInfoUtils.isAccessWifiStatePermissionGranted(getReactContext())) {
+            boolean isEnabled = mWifiManager.isWifiEnabled();
+            event.putBoolean("isWifiEnabled", isEnabled);
+        }
 
         // Add the connection type information
         event.putString("type", requestedInterface != null ? requestedInterface : mConnectionType.label);
@@ -161,8 +160,7 @@ private WritableMap createDetailsMap(@Nonnull String detailsInterface) {
                 }
                 break;
             case "wifi":
-                if (ContextCompat.checkSelfPermission(getReactContext(),
-                        Manifest.permission.ACCESS_WIFI_STATE) == PackageManager.PERMISSION_GRANTED) {
+                if (NetInfoUtils.isAccessWifiStatePermissionGranted(getReactContext())) {
                     WifiInfo wifiInfo = mWifiManager.getConnectionInfo();
                     if (wifiInfo != null) {
                         // Get the SSID

android/src/main/java/com/reactnativecommunity/netinfo/NetInfoUtils.java

@@ -6,6 +6,12 @@
  */
 package com.reactnativecommunity.netinfo;
 
+import android.Manifest;
+import android.content.Context;
+import android.content.pm.PackageManager;
+
+import androidx.core.content.ContextCompat;
+
 public class NetInfoUtils {
     public static void reverseByteArray(byte[] array) {
         for (int i = 0; i < array.length / 2; i++) {
@@ -14,4 +20,9 @@ public static void reverseByteArray(byte[] array) {
             array[array.length - i - 1] = temp;
         }
     }
+
+    public static boolean isAccessWifiStatePermissionGranted(Context context) {
+        return ContextCompat.checkSelfPermission(context,
+                Manifest.permission.ACCESS_WIFI_STATE) == PackageManager.PERMISSION_GRANTED;
+    }
 }