Update security rules to follow new best practices for transferring value

[vince0656]

Name Not thought of a name

Problem Based on https://diligence.consensys.net/posts/2019/09/stop-using-soliditys-transfer-now/ we are now using .call.value() to send value. However, this means that Solium is throwing the following: 72:37 error Consider using 'transfer' in place of 'call.value()'. security/no-call-value

Description We are suppressing this warning for now but would be great if we could fix

Priority Med/Low

[vince0656]

@jamesmorgan

[duaraghav8]

Hi @vince0656 I just noticed this issue, I'm not seeing a lot of notifs on my github, its been messed up for a while. Sorry for responding so late.

To be clear, does Ethereum and/or Consensys encourage the use of call.value()? If yes, is there any authoritative document on this? Or is this diligence document the most authoritative in your opinion?

[jamesmorgan]

@duaraghav8 I think its in the doc @vince0656 linked above https://diligence.consensys.net/posts/2019/09/stop-using-soliditys-transfer-now/