From 5ce67bf692c26f46290809886202f5fed3c68600 Mon Sep 17 00:00:00 2001
From: Benjamin Milde <benni@kobrakai.de>
Date: Wed, 22 Jan 2025 18:18:54 +0100
Subject: [PATCH] Allow keyword list options to use nil as key and/or value

---
 lib/phoenix_html/form.ex        | 13 +++++++------
 test/phoenix_html/form_test.exs | 11 +++++++----
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/lib/phoenix_html/form.ex b/lib/phoenix_html/form.ex
index dbbda53..0bf1be9 100644
--- a/lib/phoenix_html/form.ex
+++ b/lib/phoenix_html/form.ex
@@ -334,17 +334,18 @@ defmodule Phoenix.HTML.Form do
         [acc | option(option_key, option_value, [], selected_values)]
 
       options, acc when is_list(options) ->
-        {option_key, options} = Keyword.pop(options, :key)
-
-        option_key ||
+        if not Keyword.has_key?(options, :key) do
           raise ArgumentError,
                 "expected :key key when building <option> from keyword list: #{inspect(options)}"
+        end
 
-        {option_value, options} = Keyword.pop(options, :value)
-
-        option_value ||
+        if not Keyword.has_key?(options, :value) do
           raise ArgumentError,
                 "expected :value key when building <option> from keyword list: #{inspect(options)}"
+        end
+
+        {option_key, options} = Keyword.pop(options, :key)
+        {option_value, options} = Keyword.pop(options, :value)
 
         [acc | option(option_key, option_value, options, selected_values)]
 
diff --git a/test/phoenix_html/form_test.exs b/test/phoenix_html/form_test.exs
index 54fb0d8..a5d3bea 100644
--- a/test/phoenix_html/form_test.exs
+++ b/test/phoenix_html/form_test.exs
@@ -248,9 +248,10 @@ defmodule Phoenix.HTML.FormTest do
 
   describe "options_for_select/2" do
     test "simple" do
-      assert options_for_select(~w(value novalue), "novalue") |> safe_to_string() ==
+      assert options_for_select(["value", "novalue", nil], "novalue") |> safe_to_string() ==
                ~s(<option value="value">value</option>) <>
-                 ~s(<option selected value="novalue">novalue</option>)
+                 ~s(<option selected value="novalue">novalue</option>) <>
+                 ~s(<option value=""></option>)
 
       assert options_for_select(["value", :hr, "novalue"], "novalue") |> safe_to_string() ==
                ~s(<option value="value">value</option>) <>
@@ -261,14 +262,16 @@ defmodule Phoenix.HTML.FormTest do
                [
                  [value: "value", key: "Value", disabled: true],
                  :hr,
-                 [value: "novalue", key: "No Value"]
+                 [value: "novalue", key: "No Value"],
+                 [value: nil, key: nil]
                ],
                "novalue"
              )
              |> safe_to_string() ==
                ~s(<option disabled value="value">Value</option>) <>
                  ~s(<hr/>) <>
-                 ~s(<option selected value="novalue">No Value</option>)
+                 ~s(<option selected value="novalue">No Value</option>) <>
+                 ~s(<option value=""></option>)
 
       assert options_for_select(~w(value novalue), ["value", "novalue"]) |> safe_to_string() ==
                ~s(<option selected value="value">value</option>) <>