Describe the contribution
For commands that rely on CF_TsnChanAction(), this is documented as returning a count of the number of transactions affected. Therefore, a return value >0 means that the operation was successful (or at least it matched something), and a return value <=0 means failure.
Fixes a case where the function did not adhere to its documented return
Fixes passing the result to CF_CmdCond() which needs a translation.
Fixes #125
Testing performed
Build and sanity check CF
Expected behavior changes
Correct counters being incremented now
System(s) tested on
Ubuntu 21.10
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.
Describe the contribution For commands that rely on CF_TsnChanAction(), this is documented as returning a count of the number of transactions affected. Therefore, a return value >0 means that the operation was successful (or at least it matched something), and a return value <=0 means failure.
Fixes #125
Testing performed Build and sanity check CF
Expected behavior changes Correct counters being incremented now
System(s) tested on Ubuntu 21.10
Contributor Info - All information REQUIRED for consideration of pull request Joseph Hickey, Vantage Systems, Inc.