Modifiers should only be used for checks

Based on [this pattern](https://docs.soliditylang.org/en/develop/security-considerations.html#use-the-checks-effects-interactions-pattern), the modifiers should only be used to implement checks and not be used to change state or do an external call.