would it be better to use `if current_user.editor?` here?

If so, there are other places in the code where a similar change could happen.