Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Invalid

[L-2] Missing event on `setNftValue` and `setAssetToPay` in `Trustee`

Description:

A best practice would be to emit an event when such state is changed in order to notify
external systems of the change.

Impact:

Low as there are no funds at risk.

Proof of Concept: N/A

Recommended Mitigation:

function setNftValue(uint256 _index, uint256 _value) public onlyTrustee {
nftValue[_index] = _value;
+ emit NftValueSet(_index, _value);
}
function setAssetToPay(address _asset) external onlyTrustee {
assetToPay = _asset;
+ emit AssetToPaySet(_asset);
}
Updates

Lead Judging Commences

0xtimefliez Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.