Skip to content

Commit 14c9145

Browse files
yufengwangcagmarcosb
authored andcommittedMar 4, 2025
Do not pass cluster ID to CommissionerControlServer (project-chip#37644)
1 parent 8137219 commit 14c9145

File tree

4 files changed

+5
-6
lines changed

4 files changed

+5
-6
lines changed
 

‎examples/fabric-bridge-app/linux/include/CommissionerControlDelegate.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ inline constexpr EndpointId kAggregatorEndpointId = 1;
3131
class CommissionerControlDelegate : public Delegate
3232
{
3333
public:
34-
CommissionerControlDelegate() : mCommissionerControlServer(this, kAggregatorEndpointId, CommissionerControl::Id) {}
34+
CommissionerControlDelegate() : mCommissionerControlServer(this, kAggregatorEndpointId) {}
3535

3636
CHIP_ERROR HandleCommissioningApprovalRequest(const CommissioningApprovalRequest & request) override;
3737
// TODO(#35627) clientNodeId should move towards ScopedNodeId.

‎examples/fabric-sync/bridge/include/CommissionerControlDelegate.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ class CommissionerControlDelegate : public Delegate
3333
{
3434
public:
3535
CommissionerControlDelegate(bridge::FabricAdminDelegate * fabricAdmin) :
36-
mFabricAdmin(fabricAdmin), mCommissionerControlServer(this, kAggregatorEndpointId, CommissionerControl::Id)
36+
mFabricAdmin(fabricAdmin), mCommissionerControlServer(this, kAggregatorEndpointId)
3737
{}
3838

3939
CHIP_ERROR HandleCommissioningApprovalRequest(const CommissioningApprovalRequest & request) override;

‎src/app/clusters/commissioner-control-server/commissioner-control-server.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,8 @@ namespace app {
6969
namespace Clusters {
7070
namespace CommissionerControl {
7171

72-
CommissionerControlServer::CommissionerControlServer(Delegate * delegate, EndpointId endpointId, ClusterId clusterId) :
73-
CommandHandlerInterface(MakeOptional(endpointId), clusterId)
72+
CommissionerControlServer::CommissionerControlServer(Delegate * delegate, EndpointId endpointId) :
73+
CommandHandlerInterface(MakeOptional(endpointId), Id)
7474
{
7575
mDelegate = delegate;
7676
}

‎src/app/clusters/commissioner-control-server/commissioner-control-server.h

+1-2
Original file line numberDiff line numberDiff line change
@@ -110,9 +110,8 @@ class CommissionerControlServer : public CommandHandlerInterface
110110
* @param delegate A pointer to the delegate to be used by this server.
111111
* Note: the caller must ensure that the delegate lives throughout the instance's lifetime.
112112
* @param endpointId The endpoint on which this cluster exists. This must match the zap configuration.
113-
* @param clusterId The ID of the Microwave Oven Control cluster to be instantiated.
114113
*/
115-
CommissionerControlServer(Delegate * delegate, EndpointId endpointId, ClusterId clusterId);
114+
CommissionerControlServer(Delegate * delegate, EndpointId endpointId);
116115

117116
~CommissionerControlServer() override;
118117

0 commit comments

Comments
 (0)