search

mythicalprogrammer / fumigate

0 stars 0 forks source link

perfume_approval user submit perfume check for dupe #269

Closed mythicalprogrammer closed 4 years ago

mythicalprogrammer commented 4 years ago

I don't want the user to spam the same perfume entry over and over creating multiple identical records of perfume_approval.

mythicalprogrammer commented 4 years ago

Does the admin perfume action check for dupes? If so I can copy that code over...

mythicalprogrammer commented 4 years ago

https://elixirforum.com/t/dynamic-queries-in-ecto/14858/11

mythicalprogrammer commented 4 years ago

Done.

Here's the list of things I've done for this issue.

BUG updating admin/perfume_approval with no company selected - DONE

admin/perfume_controller.ex create (model: perfume, user_input: params (map)) - DONE update (model: perfume, internal_db (changeset) & user_input: params (map)) - DONE ISH CHECK for successful edit/update too - DONE NEED TO change SHOW template warning - DONE

admin/perfume_approval_controller.ex new (model: perfume, internal_db (changeset)) - DONE update (model: perfume_approval, internal_db (changeset) & user_input: params (map)) - DONE

user/perfume_controller.ex create (model perfume_approval, user_input: params (map)) - DONE

BUG updating admin/perfume_approval with no company selected

Approval.find_perfume_approval_by_name_con_comp_sex PerfumeApproval.get_all_perfume_approval_by_name_con_sex(name, concentration, gender) get_all_perfume_approval_by_name_con_sex_module(query, name, concentration, gender, Fumigate.Approval.PerfumeApprovalCompanyJoin)

Fragrance.find_perfume_by_name_con_comp_sex Perfume.get_all_perfume_by_name_con_sex(name, concentration, gender) get_all_perfume_by_name_con_sex_module(query, name, concentration, gender, Fumigate.Fragrance.PerfumeCompanyJoin)

def check company = company function () do 208 companies = for company <- companies, do: String.to_integer(company) 209 Enum.sort(results) == Enum.sort(companies)