more comments
This commit is contained in:
parent
905aff6485
commit
7b179ec04c
|
@ -184,11 +184,11 @@ contract Identity is ERC725, ERC735 {
|
||||||
{
|
{
|
||||||
uint256 requiredKey = _to == address(this) ? MANAGEMENT_KEY : ACTION_KEY;
|
uint256 requiredKey = _to == address(this) ? MANAGEMENT_KEY : ACTION_KEY;
|
||||||
if (minimumApprovalsByKeyPurpose[requiredKey] == 1) {
|
if (minimumApprovalsByKeyPurpose[requiredKey] == 1) {
|
||||||
executionId = nonce;
|
executionId = nonce; //(?) useless in this case
|
||||||
nonce++;
|
nonce++; //(?) should increment
|
||||||
require(isKeyPurpose(bytes32(msg.sender), requiredKey));
|
require(isKeyPurpose(bytes32(msg.sender), requiredKey));
|
||||||
_to.call.value(_value)(_data);
|
_to.call.value(_value)(_data); //(?) success not used
|
||||||
emit Executed(executionId, _to, _value, _data);
|
emit Executed(executionId, _to, _value, _data); //no information on success
|
||||||
} else {
|
} else {
|
||||||
executionId = _execute(_to, _value, _data);
|
executionId = _execute(_to, _value, _data);
|
||||||
approve(executionId, true);
|
approve(executionId, true);
|
||||||
|
|
Loading…
Reference in New Issue