Internally, slots are represented as a function, so that is why it was saying it the Safe protocol isn’t implemented for type Function.
I actually have some protection against this tho with regard to rendering attributes, but it seems it doesn’t currently apply when you don’t pass any list literal.
I opened an issue for this: [bug] Don't render the inner_block assign when building element attributes · Issue #132 · mhanberg/temple · GitHub
Oh, I think this actually work as expected. What you were experiencing is when the live_redirect function trying to compile the attrs. you’ll have to make sure that don’t passthrough the inner block key yourself in that case.
This is because it compiles to
<%= case assigns[:disabled] do %>
<% true -> %>
<span <%= Temple.Parser.Utils.runtime_attrs(splat(assigns)) %>>
<%= Temple.Component.__render_block__(@inner_block, {:default, %{}}) %>
</span>
<% _ -> %>
<%= Phoenix.LiveView.Helpers.live_redirect(splat(assigns)) do %>
<%= Temple.Component.__render_block__(@inner_block, {:default, %{}}) %>
<% end %>
<% end %>
as you can see, the splat passed to the span is run through a temple function that will filter out :inner_block, but that can’t happen when you pass the splat to live_redirect. So, you will have to change the Map.delete call in your code snippet to account for :inner_block
That makes sense.
I was considering passing my assigns to Temple.Parser.Util.runtime_attrs() to avoid any future dropbears, but it returns a string, and smells like a private API.
I wonder if a designated clean_attrs_keys or something would be good (or map_to_attrs_list(map()) :: keyword()). I.E:
def splat(assigns) do
assigns
|> munge_etc()
|> Enum.into([])
|> Temple.clean_attributes_keys()
end
Not sure how common the splat pattern is. It’s very useful for wrapping standard phx stuff but I doubt I would want to take on the maintenance burden if I were writing Temple.
I will probably just extract splat into it’s own module and note where to check for keys I should be dropping if something breaks.
I was considering passing my assigns to
Temple.Parser.Util.runtime_attrs()to avoid any future dropbears, but it returns a string, and smells like a private API.
Yes, do not use that function 
.