[Fix] Optimize preset validation by removing redundant command checks in SetActivePreset

Author: lboue

src/app/clusters/thermostat-server/thermostat-server-presets.cpp

@@ -302,15 +302,10 @@ Status ThermostatAttrAccess::SetActivePreset(EndpointId endpoint, DataModel::Nul
         return Status::InvalidInState;
     }
 
-    // If the preset handle passed in the command is not present in the Presets attribute, return INVALID_COMMAND.
-    if (!presetHandle.IsNull() && !IsPresetHandlePresentInPresets(delegate, presetHandle.Value()))
-    {
-        return Status::InvalidCommand;
-    }
-
     if (!presetHandle.IsNull())
     {
         PresetStructWithOwnedMembers matchingPreset;
+        // Look up the preset once and reuse it to apply any setpoints it carries.
         if (!GetMatchingPresetInPresets(delegate, presetHandle, matchingPreset))
         {
             return Status::InvalidCommand;