[Fabric-Sync] Fix subscription for non-ICD devices (#36799)

arkq · web-flow · commit 41a9dea02eff · 2024-12-11T18:53:27.000Z
diff --git a/examples/fabric-sync/admin/DeviceSynchronization.cpp b/examples/fabric-sync/admin/DeviceSynchronization.cpp
@@ -274,7 +274,7 @@ void DeviceSynchronizer::SynchronizationCompleteAddDevice()
     bridge::FabricBridge::Instance().AddSynchronizedDevice(mCurrentDeviceData);
 
     // TODO(#35077) Figure out how we should reflect CADMIN values of ICD.
-    if (!mCurrentDeviceData.isIcd)
+    if (!mCurrentDeviceData.isIcd.value_or(false))
     {
         VerifyOrDie(mController);
         ScopedNodeId scopedNodeId(mNodeId, mController->GetFabricIndex());

Original file line number	Diff line number	Diff line change
`@@ -274,7 +274,7 @@ void DeviceSynchronizer::SynchronizationCompleteAddDevice()`
`274`	`274`	`bridge::FabricBridge::Instance().AddSynchronizedDevice(mCurrentDeviceData);`
`275`	`275`
`276`	`276`	`// TODO(#35077) Figure out how we should reflect CADMIN values of ICD.`
`277`		`- if (!mCurrentDeviceData.isIcd)`
	`277`	`+ if (!mCurrentDeviceData.isIcd.value_or(false))`
`278`	`278`	`{`
`279`	`279`	`VerifyOrDie(mController);`
`280`	`280`	`ScopedNodeId scopedNodeId(mNodeId, mController->GetFabricIndex());`