Remove [commissioner] redundant TC acknowledgement flag

examples/chip-tool/commands/pairing/PairingCommand.cpp

@@ -129,9 +129,6 @@ CommissioningParameters PairingCommand::GetCommissioningParameters()
         params.SetCountryCode(CharSpan::fromCharString(mCountryCode.Value()));
     }
 
-    // Default requiring TCs to false, to preserve release 1.3 chip-tool behavior
-    params.SetRequireTermsAndConditionsAcknowledgement(mRequireTCAcknowledgements.ValueOr(false));
-
     // mTCAcknowledgements and mTCAcknowledgementVersion are optional, but related. When one is missing, default the value to 0, to
     // increase the test tools ability to test the applications.
     if (mTCAcknowledgements.HasValue() || mTCAcknowledgementVersion.HasValue())

examples/chip-tool/commands/pairing/PairingCommand.h

@@ -203,11 +203,6 @@ class PairingCommand : public CHIPCommand,
                         "DSTOffset list to use when setting Time Synchronization cluster's DSTOffset attribute",
                         Argument::kOptional);
 
-            AddArgument("require-tc-acknowledgements", 0, 1, &mRequireTCAcknowledgements,
-                        "Indicates whether Terms and Conditions acknowledgements are required during commissioning. If set to "
-                        "true, the tc-acknowledgements and tc-acknowledgements-version arguments must be provided for the "
-                        "commissioning to succeed. If false, the T&C acknowledgement step will be skipped.");
-
             AddArgument("tc-acknowledgements", 0, UINT16_MAX, &mTCAcknowledgements,
                         "Bit-field value indicating which Terms and Conditions have been accepted by the user. This value is sent "
                         "to the device during commissioning via the General Commissioning cluster");
@@ -272,7 +267,6 @@ class PairingCommand : public CHIPCommand,
     chip::Optional<uint64_t> mICDMonitoredSubject;
     chip::Optional<chip::app::Clusters::IcdManagement::ClientTypeEnum> mICDClientType;
     chip::Optional<uint32_t> mICDStayActiveDurationMsec;
-    chip::Optional<bool> mRequireTCAcknowledgements;
     chip::Optional<uint16_t> mTCAcknowledgements;
     chip::Optional<uint16_t> mTCAcknowledgementVersion;
     chip::app::DataModel::List<chip::app::Clusters::TimeSynchronization::Structs::TimeZoneStruct::Type> mTimeZoneList;

src/controller/CommissioningDelegate.h

@@ -353,12 +353,6 @@ class CommissioningParameters
         return *this;
     }
 
-    CommissioningParameters & SetRequireTermsAndConditionsAcknowledgement(bool requireTermsAndConditionsAcknowledgement)
-    {
-        mRequireTermsAndConditionsAcknowledgement = requireTermsAndConditionsAcknowledgement;
-        return *this;
-    }
-
     CommissioningParameters &
     SetTermsAndConditionsAcknowledgement(TermsAndConditionsAcknowledgement termsAndConditionsAcknowledgement)
     {

src/controller/python/ChipDeviceController-ScriptBinding.cpp

@@ -183,8 +183,6 @@ PyChipError pychip_DeviceController_OpenCommissioningWindow(chip::Controller::De
 bool pychip_DeviceController_GetIPForDiscoveredDevice(chip::Controller::DeviceCommissioner * devCtrl, int idx, char * addrStr,
                                                       uint32_t len);
 
-PyChipError pychip_DeviceController_SetRequireTermsAndConditionsAcknowledgement(bool tcRequired);
-
 PyChipError pychip_DeviceController_SetTermsAcknowledgements(uint16_t tcVersion, uint16_t tcUserResponse);
 
 PyChipError pychip_DeviceController_SetSkipCommissioningComplete(bool skipCommissioningComplete);
@@ -578,12 +576,6 @@ PyChipError pychip_DeviceController_SetDefaultNtp(const char * defaultNTP)
     return ToPyChipError(CHIP_NO_ERROR);
 }
 
-PyChipError pychip_DeviceController_SetRequireTermsAndConditionsAcknowledgement(bool tcRequired)
-{
-    sCommissioningParameters.SetRequireTermsAndConditionsAcknowledgement(tcRequired);
-    return ToPyChipError(CHIP_NO_ERROR);
-}
-
 PyChipError pychip_DeviceController_SetTermsAcknowledgements(uint16_t tcVersion, uint16_t tcUserResponse)
 {
     sCommissioningParameters.SetTermsAndConditionsAcknowledgement(

src/controller/python/chip/ChipDeviceCtrl.py

@@ -2008,9 +2008,6 @@ def _InitLib(self):
             self._dmLib.pychip_DeviceController_SetSkipCommissioningComplete.restype = PyChipError
             self._dmLib.pychip_DeviceController_SetSkipCommissioningComplete.argtypes = [c_bool]
 
-            self._dmLib.pychip_DeviceController_SetRequireTermsAndConditionsAcknowledgement.restype = PyChipError
-            self._dmLib.pychip_DeviceController_SetRequireTermsAndConditionsAcknowledgement.argtypes = [c_bool]
-
             self._dmLib.pychip_DeviceController_SetTermsAcknowledgements.restype = PyChipError
             self._dmLib.pychip_DeviceController_SetTermsAcknowledgements.argtypes = [c_uint16, c_uint16]
 
@@ -2142,13 +2139,6 @@ def SetDSTOffset(self, offset: int, validStarting: int, validUntil: int):
             lambda: self._dmLib.pychip_DeviceController_SetDSTOffset(offset, validStarting, validUntil)
         ).raise_on_error()
 
-    def SetTCRequired(self, tcRequired: bool):
-        ''' Set whether TC Acknowledgements should be set during commissioning'''
-        self.CheckIsActive()
-        self._ChipStack.Call(
-            lambda: self._dmLib.pychip_DeviceController_SetRequireTermsAndConditionsAcknowledgement(tcRequired)
-        ).raise_on_error()
-
     def SetTCAcknowledgements(self, tcAcceptedVersion: int, tcUserResponse: int):
         ''' Set the TC acknowledgements to set during commissioning'''
         self.CheckIsActive()