fix: prefer not to mutate the ScanResult object (#268)

JuanSeBestia · May 26, 2023 · 840c80d · 840c80d
1 parent 49a59ba
commit 840c80d
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/android/src/main/java/com/reactlibrary/rnwifi/mappers/WifiScanResultsMapper.java b/android/src/main/java/com/reactlibrary/rnwifi/mappers/WifiScanResultsMapper.java
@@ -13,15 +13,20 @@ public class WifiScanResultsMapper {
     private WifiScanResultsMapper() {
     }
 
+    private static String parseSSID(final ScanResult scanResult) {
+        if (scanResult.SSID.equals("")) {
+            return "(hidden SSID)";
+        }
+
+        return scanResult.SSID;
+    }
+
     public static WritableArray mapWifiScanResults(final List<ScanResult> scanResults) {
         final WritableArray wifiArray = new WritableNativeArray();
 
         for (ScanResult result : scanResults) {
             final WritableMap wifiObject = new WritableNativeMap();
-            if (result.SSID.equals("")) {
-                result.SSID = "(hidden SSID)";
-            }
-            wifiObject.putString("SSID", result.SSID);
+            wifiObject.putString("SSID", parseSSID(result));
             wifiObject.putString("BSSID", result.BSSID);
             wifiObject.putString("capabilities", result.capabilities);
             wifiObject.putInt("frequency", result.frequency);