Add an optional "separator" field for coordinates (#8)

Author: bdebinska

lib/schemas.ex

@@ -71,6 +71,11 @@ defmodule ValueFormatters.Schemas do
         default: true,
         description:
           "Whether the formatter should include the radius/accuracy information (if present)."
+      },
+      separator: %{
+        type: :string,
+        default: ", ",
+        description: "The string used to separate latitude, longitude and radius values."
       }
     }
   end
@@ -256,7 +261,7 @@ defmodule ValueFormatters.Schemas.Format do
             }
           },
           required: [:field],
-          additionalProperties: false
+          additionalProperties: true
         }
       ]
     }

lib/value_formatters.ex

@@ -365,19 +365,22 @@ defmodule ValueFormatters do
            format_number(lat, %{"format" => "number", "precision" => 5}, opts),
          {:ok, lng_formatted} <-
            format_number(lng, %{"format" => "number", "precision" => 5}, opts) do
+      separator = get_in(coordinate_definition, ["separator"]) || ", "
+
       if get_in(coordinate_definition, ["radius_display"]) != false and radius != nil do
         with {:ok, radius_formatted} <-
                format_number(
                  radius,
                  %{"format" => "number", "precision" => 0, "unit" => "m"},
                  opts
                ) do
-          {:ok, "#{lat_formatted}\u{00B0}, #{lng_formatted}\u{00B0}, #{radius_formatted}"}
+          {:ok,
+           "#{lat_formatted}\u{00B0}#{separator}#{lng_formatted}\u{00B0}#{separator}#{radius_formatted}"}
         else
           {:error, reason} -> {:error, reason}
         end
       else
-        {:ok, "#{lat_formatted}\u{00B0}, #{lng_formatted}\u{00B0}"}
+        {:ok, "#{lat_formatted}\u{00B0}#{separator}#{lng_formatted}\u{00B0}"}
       end
     else
       {:error, _reason} -> {:error, "Value #{value} cannot be parsed as a coordinate"}