Remove the incorrect GroupKeySetIDs field from the KeySetReadAllIndices command. (#27044)
* Remove the incorrect GroupKeySetIDs field from the KeySetReadAllIndices command.
This field does not exist in the spec.
This is safe to do in terms of compat because the generated command parsing code
only parses the fields that are actually present and does not error out if
mandatory fields are missing; instead if uses type-default values for them. So
commands updated clients send will still be parse-able by servers that had this
field in their XML.
Fixes part of https://github.com/project-chip/connectedhomeip/issues/25642
For the Darwin codegen, I just special-cased this command in the templates for
now, but if we end up with more commands going from having required fields to
not having them we can figure out a more automated solution.
* Regenerate generated code.
* Address review comments.
diff --git a/examples/chef/devices/rootnode_fan_7N2TobIlOX.matter b/examples/chef/devices/rootnode_fan_7N2TobIlOX.matter
index 0a5f051..0ad9c02 100644
--- a/examples/chef/devices/rootnode_fan_7N2TobIlOX.matter
+++ b/examples/chef/devices/rootnode_fan_7N2TobIlOX.matter
@@ -1162,10 +1162,6 @@
INT16U groupKeySetID = 0;
}
- request struct KeySetReadAllIndicesRequest {
- INT16U groupKeySetIDs[] = 0;
- }
-
response struct KeySetReadResponse = 2 {
GroupKeySetStruct groupKeySet = 0;
}
@@ -1177,7 +1173,7 @@
fabric command access(invoke: administer) KeySetWrite(KeySetWriteRequest): DefaultSuccess = 0;
fabric command access(invoke: administer) KeySetRead(KeySetReadRequest): KeySetReadResponse = 1;
fabric command access(invoke: administer) KeySetRemove(KeySetRemoveRequest): DefaultSuccess = 3;
- fabric command access(invoke: administer) KeySetReadAllIndices(KeySetReadAllIndicesRequest): KeySetReadAllIndicesResponse = 4;
+ fabric command access(invoke: administer) KeySetReadAllIndices(): KeySetReadAllIndicesResponse = 4;
}
/** The Fixed Label Cluster provides a feature for the device to tag an endpoint with zero or more read only
