Mirrors/knx
1
0
Fork 0
mirror of https://github.com/thelsing/knx.git synced 2024-12-18 19:08:18 +01:00
Code Issues Packages Projects Releases Wiki Activity

(Prepare for devel) Fix: Do NOT Send when Updating KO Failed on Value Conversion

Browse Source
This commit is contained in:
Cornelius Köpp 2024-11-02 23:31:08 +01:00
parent a7d344fae9
commit 9ee6e11ca5
1 changed files with 26 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/knx/group_object/group_object.h
View File

@ -478,14 +478,29 @@ namespace Knx
ComFlag _commFlag : 7;
uint8_t* _data = 0;
uint8_t _dataLength = 0;
/**
* set the current value of the group object and show success.
* @param value the value the group object is set to
* @param type the datapoint type used for the conversion.
*
* The parameters must fit the group object. Otherwise it will stay unchanged.
*
* @returns true if the value of the group object was updated after successful conversion.
*/
template<class DPT> bool _valueNoSend(const DPT& value);
};
bool operator==(const GroupObject& lhs, const GroupObject& rhs);
template<class DPT> void GroupObject::value(const DPT& value)
{
valueNoSend(value);
objectWritten();
if (_valueNoSend(value))
{
// write on successful conversion/setting value only
objectWritten();
}
}
template<class DPT> DPT GroupObject::value()
@ -508,6 +523,12 @@ namespace Knx
}
template<class DPT> void GroupObject::valueNoSend(const DPT& value)
{
// ignore actual updating TODO check replacing valueNoSend with _valueNoSend
_valueNoSend(value);
}
template<class DPT> bool GroupObject::_valueNoSend(const KNXValue& value)
{
if (value.size() != sizeCode())
return;
@ -518,6 +539,8 @@ namespace Knx
// initialize on succesful conversion only
if (encodingDone && _uninitialized)
commFlag(Ok);
return encodingDone;
}
template<class DPT> bool GroupObject::valueNoSendCompare(const DPT& value)
@ -528,8 +551,7 @@ namespace Knx
if (_uninitialized)
{
// always set first value
this->valueNoSend(value);
return true;
return _valueNoSend(value);
}
else
{