We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
現在はis_visible_toなどのメソッドを用いて認可処理を行っているが、この処理を行わなくてもコンパイルができてしまう。そこで、Actorを引数にとり、Resultを返すようなメソッドを持つ型を実装し、これを返すことでこの問題を解決できるものと思われる。 一方でこれをするならばリポジトリのインターフェイスを変更する必要があるが、リポジトリの責務ではないと思う。 repositoryのインターフェイスの型はdomainの都合に合わせるべきで、認可はdomainの責務だと思っているので問題ないはず。 ensure!を使用している箇所も同様の解決が可能と思われるが、いい感じの実装方法が思いついていない。
is_visible_to
Actor
Result
ensure!
The text was updated successfully, but these errors were encountered:
arata-nvm
Successfully merging a pull request may close this issue.
現在は
is_visible_to
などのメソッドを用いて認可処理を行っているが、この処理を行わなくてもコンパイルができてしまう。そこで、Actor
を引数にとり、Result
を返すようなメソッドを持つ型を実装し、これを返すことでこの問題を解決できるものと思われる。一方でこれをするならばリポジトリのインターフェイスを変更する必要があるが、リポジトリの責務ではないと思う。repositoryのインターフェイスの型はdomainの都合に合わせるべきで、認可はdomainの責務だと思っているので問題ないはず。ensure!
を使用している箇所も同様の解決が可能と思われるが、いい感じの実装方法が思いついていない。The text was updated successfully, but these errors were encountered: