feat: add valueId, node and logger to parse functions

[ahochsteger]

This PR makes the following objects available to the send and receive parse functions:

• logger: The logger object of the gateway
• valueId: The valueId object that contains more information about the affected value
• node: The node object that contains all information about the affected node

It allows something like this which was not possible before:

// Some debugging:
logger.debug('Got the following arguments:')
logger.debug('- value: ' + value)
logger.debug('- valueId: ' + JSON.stringify(valueId))
logger.debug('- node: ' + JSON.stringify(node))

// Check for invalid values:
if (value < -1) {
  // Log a warning if invalid value reports have been fixed:
  logger.warn('Invalid value ' + value + ' detected - setting to -1.')
  value = -1
}
return value

[robertsLando]

@ahochsteger I would also add the node object, before the logger :) ALso remember to update docs

[ahochsteger]

@ahochsteger I would also add the node object, before the logger :) ALso remember to update docs

Good idea, will add it and have a look at the docs ...

[robertsLando]

dito

[ahochsteger]

@ahochsteger I would also add the node object, before the logger :)

How would I get the node object in evalFunction() as it just has access to code, valueId and value?
Or did you mean the valueId object?

[ahochsteger]

Ok, I can add it to evalFunction as 4th argument.
Does it make sense to provide valueId as well as it may contain more interesting information?

[robertsLando]

@ahochsteger Yep I would add in order (value, valueId, node, logger)

[ahochsteger]

@ahochsteger Yep I would add in order (value, valueId, node, logger)

Done locally.
What about the receive parse function?
There doesn't seem to be a node object available in parsePayload(payload, valueId, valueConf).
Should I pass an empty object ({}) or null?

[coveralls]

Pull Request Test Coverage Report for Build 509782018

• 1 of 12 (8.33%) changed or added relevant lines in 1 file are covered.
• 4 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.007%) to 23.369%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
lib/Gateway.js	1	12	8.33%

Files with Coverage Reduction	New Missed Lines	%
lib/Gateway.js	4	18.14%

Totals
Change from base Build 509745234:	-0.007%
Covered Lines:	2009
Relevant Lines:	8798

---

💛 - Coveralls

[ahochsteger]

I'm using an empty object for node for now at the receive part.