Merge pull request #262 from randdusing/discovery

discovery on reconnect

Author: randdusing

readme.md

@@ -1379,7 +1379,7 @@ bluetoothle.isConnected(isConnectedSuccess, isConnectedError, params);
 
 
 ### isDiscovered ###
-Determine whether the device's characteristics and descriptors have been discovered, or error if not initialized or never connected to device.
+Determine whether the device's characteristics and descriptors have been discovered, or error if not initialized or not connected to device. Note, on Android, you can connect, discover and then disconnect. isDiscovered will return an error due to the device not being connected. But if you call reconnect and call isDiscovered again, it will return isDiscovered => true since the device stays discovered until calling close().
 
 ```javascript
 bluetoothle.isDiscovered(isDiscoveredSuccess, isDiscoveredError, params);

src/android/BluetoothLePlugin.java

@@ -1286,7 +1286,7 @@ private void reconnectAction(JSONArray args, CallbackContext callbackContext) {
     }
 
     connection.put(keyState, BluetoothProfile.STATE_CONNECTING);
-    connection.put(keyDiscoveredState, STATE_UNDISCOVERED);
+    //connection.put(keyDiscoveredState, STATE_UNDISCOVERED); //Devices stays discovered even if disconnected (but not closed)
     connection.put(operationConnect, callbackContext);
   }
 
@@ -3505,11 +3505,17 @@ public void onConnectionStateChange(BluetoothGatt gatt, int status, int newState
         returnObj.remove(keyError);
         returnObj.remove(keyMessage);
 
+        //Save the old discovered state
+        Object discoveredState = connection.get(keyDiscoveredState);
+
         //Clear out all the callbacks
         connection = new HashMap<Object, Object>();
         connection.put(keyPeripheral, gatt);
         connection.put(keyState, BluetoothProfile.STATE_DISCONNECTED);
 
+        //Save state in new connection
+        connection.put(keyDiscoveredState, discoveredState);
+
         connections.put(device.getAddress(), connection);
 
         if (callbackContext == null) {