Elixlsx Fork - Patch issue with nil/empty cells unable to be styled

README.md

@@ -1,5 +1,11 @@
 # Elixlsx
 
+> ### Rum&Code
+> ℹ️ This repo is a fork of [xou/elixlsx](https://github.com/xou/elixlsx/tree/master) to fix some of the shortcomings with the original library.
+> #### Changelog & differences
+> - Fix issue where passing `nil` to a cell's content ignores that cell's styling & formatting props
+>   - Add missing pattern in Elixlsx.Util.to_excel_datetime/1 to support empty cells
+
 [![Build Status](https://travis-ci.com/xou/elixlsx.svg?branch=master)](https://travis-ci.org/xou/elixlsx)
 [![Module Version](https://img.shields.io/hexpm/v/elixlsx.svg)](https://hex.pm/packages/elixlsx)
 [![Hex Docs](https://img.shields.io/badge/hex-docs-lightgreen.svg)](https://hexdocs.pm/elixlsx/)

lib/elixlsx/util.ex

@@ -230,6 +230,9 @@ defmodule Elixlsx.Util do
   def to_excel_datetime({:formula, value}) do
     {:formula, value}
   end
+
+  @spec to_excel_datetime(any()) :: any()
+  def to_excel_datetime(value), do: value
 
   @doc ~S"""
   Replace_all(input, [{search, replace}]).

lib/elixlsx/xml_templates.ex

@@ -221,68 +221,71 @@ defmodule Elixlsx.XMLTemplates do
       |> List.foldl({"", 1}, fn cell, {acc, colidx} ->
         {content, styleID, cellstyle} = split_into_content_style(cell, wci)
 
-        if is_nil(content) do
-          {acc, colidx + 1}
-        else
-          content =
-            if CellStyle.is_date?(cellstyle) do
-              U.to_excel_datetime(content)
-            else
-              content
-            end
-
-          cv = get_content_type_value(content, wci)
-
-          {content_type, content_value, content_opts} =
-            case cv do
-              {t, v} ->
-                {t, v, []}
-
-              {t, v, opts} ->
-                {t, v, opts}
-
-              :error ->
-                raise %ArgumentError{
-                  message:
-                    "Invalid column content at " <>
-                      U.to_excel_coords(rowidx, colidx) <> ": " <> inspect(content)
-                }
-            end
-
-          cell_xml =
-            case content_type do
-              :formula ->
-                value =
-                  if not is_nil(content_opts[:value]),
-                    do: "<v>#{content_opts[:value]}</v>",
-                    else: ""
-
-                """
-                <c r="#{U.to_excel_coords(rowidx, colidx)}"
-                s="#{styleID}">
-                <f>#{content_value}</f>
-                #{value}
-                </c>
-                """
-
-              :empty ->
-                """
-                <c r="#{U.to_excel_coords(rowidx, colidx)}"
-                s="#{styleID}">
-                </c>
-                """
-
-              type ->
-                """
-                <c r="#{U.to_excel_coords(rowidx, colidx)}"
-                s="#{styleID}" t="#{type}">
-                <v>#{content_value}</v>
-                </c>
-                """
-            end
-
-          {acc <> cell_xml, colidx + 1}
-        end
+        content =
+          if is_nil(content) do
+            ""
+          else
+            content
+          end
+
+        content =
+          if CellStyle.is_date?(cellstyle) do
+            U.to_excel_datetime(content)
+          else
+            content
+          end
+
+        cv = get_content_type_value(content, wci)
+
+        {content_type, content_value, content_opts} =
+          case cv do
+            {t, v} ->
+              {t, v, []}
+
+            {t, v, opts} ->
+              {t, v, opts}
+
+            :error ->
+              raise %ArgumentError{
+                message:
+                  "Invalid column content at " <>
+                  U.to_excel_coords(rowidx, colidx) <> ": " <> inspect(content)
+              }
+          end
+
+        cell_xml =
+          case content_type do
+            :formula ->
+              value =
+                if not is_nil(content_opts[:value]),
+                   do: "<v>#{content_opts[:value]}</v>",
+                   else: ""
+
+              """
+              <c r="#{U.to_excel_coords(rowidx, colidx)}"
+              s="#{styleID}">
+              <f>#{content_value}</f>
+              #{value}
+              </c>
+              """
+
+            :empty ->
+              """
+              <c r="#{U.to_excel_coords(rowidx, colidx)}"
+              s="#{styleID}">
+              </c>
+              """
+
+            type ->
+              """
+              <c r="#{U.to_excel_coords(rowidx, colidx)}"
+              s="#{styleID}" t="#{type}">
+              <v>#{content_value}</v>
+              </c>
+              """
+          end
+
+        {acc <> cell_xml, colidx + 1}
       end)
 
     updated_row