fix: do not cast None to str in ArrowBackend

tconbeer · Oct 31, 2024 · 08ee735 · 08ee735
1 parent a7d1717
commit 08ee735
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@ All notable changes to this project will be documented in this file.
 ## [Unreleased]
 
 - Adds an optional parameter to DataTable to disable rendering of string data as Rich Markup.
+- Fixes a bug where None could be casted to a string and displayed as "None" ([tconbeer/harlequin#658](https://github.com/tconbeer/harlequin/issues/658))
 
 ## [0.9.0] - 2024-07-23
 

diff --git a/src/textual_fastdatatable/backend.py b/src/textual_fastdatatable/backend.py
@@ -252,7 +252,10 @@ def from_pydict(
         except (pal.ArrowInvalid, pal.ArrowTypeError):
             # one or more fields has mixed types, like int and
             # string. Cast all to string for safety
-            new_data = {k: [str(val) for val in v] for k, v in data.items()}
+            new_data = {
+                k: [str(val) if val is not None else None for val in v]
+                for k, v in data.items()
+            }
             tbl = pa.Table.from_pydict(new_data)
         return cls(tbl, max_rows=max_rows)