Added guard to peripheral:didUpdateNotificationStateForCharacteristic…

…:error: that checks if charcteristic.value is non-nil (plugin crashes if value is nil).
evothings · Dec 16, 2015 · 55456d4 · 55456d4
1 parent 247af97
commit 55456d4
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/ios/EVOBLE.m b/src/ios/EVOBLE.m
@@ -667,7 +667,11 @@ - (void) peripheral: (CBPeripheral *)peripheral
 	didUpdateNotificationStateForCharacteristic: (CBCharacteristic *)characteristic
 	error: (NSError *)error
 {
-	[self peripheral:peripheral didUpdateValueForCharacteristic:characteristic error:error];
+	// Call didUpdateValueForCharacteristic only when we have a value.
+	if (characteristic.value)
+	{
+		[self peripheral:peripheral didUpdateValueForCharacteristic:characteristic error:error];
+	}
 }
 
 - (void)peripheral:(CBPeripheral *) peripheral
@@ -1173,6 +1177,8 @@ - (void) enableNotification: (CDVInvokedUrlCommand*)command
 	if (nil == characteristic) return; // Error.
 
 	// Result is delivered in:
+	//  peripheral:didUpdateValueForCharacteristic:error:
+	// TODO Confirm this: Possibly results are also delived in:
 	//	peripheral:didUpdateNotificationStateForCharacteristic:error:
 	[myPeripheral
 		addCallbackForCharacteristic: characteristic