budgie
I have this function and the exception is being raised as expected. However, the block is still being evaluated even though the conditional returns false. What is happening here?
def authorize_role!(from, to, required_role, do: block) when required_role in @roles do
actual_role = role_for_schema!(from, to)
if role_index(actual_role) >= role_index(required_role) do
block
else
raise AuthorizationError
end
end
Where Next?
Trending in Questions
Other Trending Topics
Latest on Elixir Forum
Sponsor Spotlight
Catch errors, track performance, monitor hosts and more.
Categories:
Sub Categories:
Forums
Popular Tags
- #ecto
- #liveview
- #troubleshooting
- #learning-elixir
- #library
- #deployment
- #erlang
- #testing
- #genserver
- #mix
- #absinthe
- #remote-other
- #otp
- #plug
- #how-to-question
- #macros
- #postgres
- #elixirconf
- #channels
- #exunit
- #discussion
- #code-sync
- #podcasts
- #javascript
- #onsite
- #dialyzer
- #docker
- #authentication
- #umbrella
- #full-time-contract
- #podcasts-by-brainlid
- #ecto-query
- #ai
- #elixirconf-us
- #blog-post
- #elixir-ls
- #phoenix_html
- #iex
- #graphql
- #genstage
- #websockets
- #supervisor
- #advent-of-code
- #distillery
- #processes
- #api
- #forms
- #hex
- #security
- #metaprogramming
Our Sponsors
Build Elixir applications with speed and confidence.
We build reliable cloud platforms for business-critical systems.
Catch errors, track performance, monitor hosts and more.
Error tracking, logs, uptime, and Just Enough APM. Built for developers.
Elixir, Erlang & BEAM conferences from Code Sync - join us in 2026!
Practical resources that improve the lives of professional developers.
Bringing learning, work and compliance together since 1987.
Producing high quality Elixir screencasts since 2017.
The team behind Membrane, Popcorn, LiveDebugger. Available for hire.
Courses that'll move you from confusion to "Aha, now I get it!"
Showing Posts 1 to 4- Show Best Posts
- Show All (oldest first)
- Show All (newest first)
sodapopcan
It needs to be a macro, not a function.
blockis being evaluated as soon as you pass it as a parameter at the callsite.benwilson512
Yeah to expand on what @sodapopcan if you want to delay the execution of some code without a macro you’d use a function eg:
If you want to use a do block style then yeah that has to be a macro because you’re essentially re-arranging the AST to wrap the code in the
if.sodapopcan
Oh ya, that is even better IMO.
budgie
Thanks! I think a macro is overkill for this usecase, so passing in the callback makes sense. Thanks