address comment

jmartinez-silabs · jmartinez-silabs · commit 4ed3d8b81a06 · 2024-07-18T15:13:29.000-04:00
diff --git a/src/app/clusters/on-off-server/on-off-server.cpp b/src/app/clusters/on-off-server/on-off-server.cpp
@@ -615,7 +615,7 @@ bool OnOffServer::offWithEffectCommand(app::CommandHandler * commandObj, const a
     chip::EndpointId endpoint = commandPath.mEndpointId;
     Status status             = Status::Success;
 
-    if (isKnownEnumValue(effectId))
+    if (effectId != EffectIdentifierEnum::kUnknownEnumValue)
     {
         // Depending on effectId value, effectVariant enum type varies.
         // The following check validates that effectVariant value is valid in relation to the applicable enum type.

Original file line number	Diff line number	Diff line change
`@@ -615,7 +615,7 @@ bool OnOffServer::offWithEffectCommand(app::CommandHandler * commandObj, const a`
`615`	`615`	`chip::EndpointId endpoint = commandPath.mEndpointId;`
`616`	`616`	`Status status = Status::Success;`
`617`	`617`
`618`		`- if (isKnownEnumValue(effectId))`
	`618`	`+ if (effectId != EffectIdentifierEnum::kUnknownEnumValue)`
`619`	`619`	`{`
`620`	`620`	`// Depending on effectId value, effectVariant enum type varies.`
`621`	`621`	`// The following check validates that effectVariant value is valid in relation to the applicable enum type.`