Merge pull request #543 from muscardinus/muscardinus/bugfix/disable-c…

…haracteristic-notifications Bug fix: Write correct descriptor value when attempting to disable notifications
NordicSemiconductor · Feb 14, 2024 · 3ef4709 · 3ef4709
2 parents 3112b52 + 6125ba8
commit 3ef4709
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/ble/src/main/java/no/nordicsemi/android/ble/BleManagerHandler.java b/ble/src/main/java/no/nordicsemi/android/ble/BleManagerHandler.java
@@ -969,7 +969,7 @@ private boolean internalDisableNotifications(@Nullable final BluetoothGattCharac
 			if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
 				log(Log.DEBUG, () ->
 						"gatt.writeDescriptor(00002902-0000-1000-8000-00805f9b34fb, value=0x00-00)");
-				return gatt.writeDescriptor(descriptor, BluetoothGattDescriptor.ENABLE_NOTIFICATION_VALUE) == BluetoothStatusCodes.SUCCESS;
+				return gatt.writeDescriptor(descriptor, BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE) == BluetoothStatusCodes.SUCCESS;
 			} else {
 				log(Log.DEBUG, () -> "descriptor.setValue(0x00-00)");
 				descriptor.setValue(BluetoothGattDescriptor.DISABLE_NOTIFICATION_VALUE);