public function isAuthor($user) {
        if ($user) {
            if($this->trashed()) {
                return null;
            }

            if ($this->users()->find($user->id)) {
                return true;
            } else {
                return false;
            }
        }
        return'mismatch';
    }



чем меньше вложенность ифов, тем удобнее читать код, лучше так:

if (!user) {
     return 'mismatch';
}
...тут остальное без вложенности



и еще - лучше все тексты по типу "mismatch" выносить куда-нибудь в константы