david234
Cannot invoke remote function Access.get/2 inside guards
I have this form
<%= form_for @conn, Routes.admin_products_path(@conn, :create_product), [as: :create_product, id: "product-form", class: "mt-5 mb-5 login-input", method: :post, multipart: :true], fn f -> %>
<div class="form-group">
<%= text_input f, :name, placeholder: "Name", class: "form-control" %>
</div>
<div class="form-group">
<%= text_input f, :description, placeholder: "Description", class: "form-control" %>
</div>
<h4 class="card-title">Image Upload</h4>
<div class="form-group">
<%= file_input f, :image_files, name: "create_product[images][]", multiple: true, class: "form-control", placeholder: "Upload Image" %>
</div>
<h4 class="card-title">Video Upload</h4>
<div class="form-group">
<%= file_input f, :video_files, name: "create_product[videos][]", multiple: true, class: "form-control", placeholder: "Upload Image" %>
</div>
<br/><br/>
<input type="submit" value="Save" class="btn btn-dark mb-2" />
<% end %>
I have this three function to filter and process through Guard
def create_product(conn, %{"create_product" => product_params}) when is_list(product_params["images"]) == true and is_list(product_params["videos"]) == true do
def create_product(conn, %{"create_product" => product_params}) when is_list(product_params["images"]) == true and is_list(product_params["videos"]) == false do
def create_product(conn, %{"create_product" => product_params}) when is_list(product_params["images"]) == false and is_list(product_params["videos"]) == true do
Below is my inspect output
%{
"description" => "sdfdf",
"images" => [
[
%Plug.Upload{
content_type: "image/jpeg",
filename: "deuteronomy-2223-eggstransvestite-transvestism-cross-dressing-cult-prostitutes-lost-and-found-35-638.jpg",
path: "/tmp/plug-1586/multipart-1586529347-447222130427589-1"
}
]
],
"name" => "Sony"
}
I can see this error is due to accessing product_params["images"].
Can anyone give me insight on how to solve this? Or is there any better way to filter this. Your help is greatly appreciated
Marked As Solved
benwilson512
Hi @david234, you need to write it like this:
def create_product(conn, %{"create_product" => %{"images" => images, "videos" => videos} = product_params)
when is_list(images) and not is_list(videos) do
Basically, you need to pattern match out the values you want to use in the guards. Do note that the clause won’t match at all unless the product params include BOTH images and videos. If you need to handle them not being present at all, I’d consider doing this just inside the function body.
Also Liked
benwilson512
Without know what you’re doing in the function body it’s hard to say. Personally I’d probably do something more like:
def create_product(conn, %{"create_product" => product_params}) do
images = product_params["images"] || []
videos = product_params["videos"] || []
But again, it depends on what you’re doing. If each of those clauses is a wildly different flow then go ahead with the pattern you have. If you’re just using it to normalize the data or something then I think that can be accomplished more succinctly by working with the parameters more individually.
Popular in Questions
Other popular topics
Latest on Elixir Forum
Sponsor Spotlight
The deployment platform built for Elixir: PaaS ease, VPS control.
Categories:
Sub Categories:
Forums
Our Sponsors
Build Elixir applications with speed and confidence.
Supporting innovation across the BEAM ecosystem.
We build reliable cloud platforms for business-critical systems.
Catch errors, track performance, monitor hosts and more.
Practical resources that improve the lives of professional developers.
Develop your skills with books, videos, and courses.
Producing high quality Elixir screencasts since 2017.
Courses that'll move you from confusion to "Aha, now I get it!"