Modifiers can be named better
Modifiers need to clearly express what they are checking, enforcing or requiring or not allowing etc.
The modifier moreThanZero(..) needs to be clear that its amount that must be enforced to be more than zero
The modifier isAllowedToken() makes it seem as if its saying token is allowed vs is token allowed?, it's not clear
Informational: Naming them right helps other developers, auditors . maintainers impacting better readability, maintanability and auditability of code.
Manual analysis
It is recommended that modifiers use consistent, descriptive naming that signifies clarity as to what is being checked, enforced or required etc. It may be best practise to consider using 'only' format for modifiers enforcing check of what is allowed and what is not allowed e.g
DSCEngine.sol line 95 moreThanZero -> onlyAmountsAboveZero(...)
DSCEngine.sol line 102 isAllowedToken -> onlyAllowedToken(...)
With better naming monitoring and error reading and assessing becomes easier and faster
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.