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

Lack of Event Emissions

Summary

The setResult function does not emit an event, making it difficult to track and verify when match results are updated. This lack of event emission reduces transparency and hinders the ability to monitor changes in the contract's state.

function setResult(uint256 matchNumber, Result result) public onlyOwner {
results[matchNumber] = result;
}

Impact: Low. Event emissions are crucial for off-chain monitoring

Tools Used

Manual Review

Recommendations

// Example: emit an event in setResult
event ResultSet(uint256 matchNumber, Result result);
function setResult(uint256 matchNumber, Result result) public onlyOwner {
results[matchNumber] = result;
emit ResultSet(matchNumber, result);
}
Updates

Lead Judging Commences

NightHawK Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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