numbers-parser 4.4.7__tar.gz → 4.4.8__tar.gz

{numbers_parser-4.4.7 → numbers_parser-4.4.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: numbers-parser
-Version: 4.4.7
+Version: 4.4.8
 Summary: Read and write Apple Numbers spreadsheets
 Home-page: https://github.com/masaccio/numbers-parser
 License: MIT
@@ -19,7 +19,6 @@ Classifier: Topic :: Office/Business :: Financial :: Spreadsheet
 Requires-Dist: compact-json (>=1.1.3,<2.0.0)
 Requires-Dist: pendulum (>=3.0,<4.0)
 Requires-Dist: protobuf (>=4.21.1,<5.0.0)
-Requires-Dist: pylama (>=8.4.1,<9.0.0)
 Requires-Dist: python-snappy (>=0.6.1,<0.7.0)
 Requires-Dist: regex (>=2022.9.13,<2023.0.0)
 Requires-Dist: roman (>=3.3,<4.0)
@@ -284,6 +283,8 @@ with open (cell.style.bg_image.filename, "wb") as f:
     f.write(cell.style.bg_image.data)
 ```
 
+Due to a limitation in Python's [ZipFile](https://docs.python.org/3/library/zipfile.html), Python versions older than 3.11 do not support image filenames with UTF-8 characters (see [issue 69](https://github.com/masaccio/numbers-parser/issues/69)). `cell.style.bg_image` returns `None` for such files and issues a `RuntimeWarning`.
+
 ### Borders
 
 `numbers-parser` supports reading and writing cell borders, though the interface for each differs. Individual cells can have each of their four borders tested, but when drawing new borders, these are set for the table to allow for drawing borders across multiple cells. Setting the border of merged cells is not possible unless the edge of the cells is at the end of the merged region.

{numbers_parser-4.4.7 → numbers_parser-4.4.8}/README.md

@@ -253,6 +253,8 @@ with open (cell.style.bg_image.filename, "wb") as f:
     f.write(cell.style.bg_image.data)
 ```
 
+Due to a limitation in Python's [ZipFile](https://docs.python.org/3/library/zipfile.html), Python versions older than 3.11 do not support image filenames with UTF-8 characters (see [issue 69](https://github.com/masaccio/numbers-parser/issues/69)). `cell.style.bg_image` returns `None` for such files and issues a `RuntimeWarning`.
+
 ### Borders
 
 `numbers-parser` supports reading and writing cell borders, though the interface for each differs. Individual cells can have each of their four borders tested, but when drawing new borders, these are set for the table to allow for drawing borders across multiple cells. Setting the border of merged cells is not possible unless the edge of the cells is at the end of the merged region.

{numbers_parser-4.4.7 → numbers_parser-4.4.8}/pyproject.toml

@@ -12,7 +12,7 @@ name = "numbers-parser"
 packages = [{include = "numbers_parser", from = "src"}]
 readme = "README.md"
 repository = "https://github.com/masaccio/numbers-parser"
-version = "4.4.7"
+version = "4.4.8"
 
 [tool.poetry.scripts]
 cat-numbers = "numbers_parser._cat_numbers:main"
@@ -28,7 +28,6 @@ regex = "^2022.9.13"
 roman = "^3.3"
 sigfig = "^1.3.2"
 setuptools = "^69.0.3"
-pylama = "^8.4.1"
 
 [tool.poetry.group.dev.dependencies]
 black = {version = "^22.10.0", allow-prereleases = true}
@@ -42,10 +41,10 @@ pytest-console-scripts = "^1.3.1"
 pytest-cov = "^4.0.0"
 pytest-profiling = "^1.7.0"
 pytest-xdist = "^3.3.1"
-python-magic = "^0.4.27"
 ruff = "^0.0.292"
 termcolor = "^2.2.0"
 tox = "^4.11.4"
+python-magic = "^0.4.27"
 
 [build-system]
 build-backend = "poetry.core.masonry.api"
@@ -69,9 +68,16 @@ legacy_tox_ini = """
 isolated_build = true
 envlist = py38, py39, py310, py311, py312
 [testenv]
-allowlist_externals = poetry
-commands_pre = poetry install --no-root --sync
-commands = poetry run -vvv pytest --import-mode importlib -n logical --no-cov
+deps =
+  pytest
+  pytest-check
+  pytest-console-scripts
+  pytest-cov
+  pytest-xdist
+  python-magic
+  psutil
+commands =
+  pytest tests/ --import-mode importlib -n logical --no-cov
 """
 
 [tool.isort]

{numbers_parser-4.4.7 → numbers_parser-4.4.8}/src/numbers_parser/cell_storage.py

@@ -291,8 +291,15 @@ class CellStorage(Cacheable):
         stored_filename = [x.file_name for x in datas if x.identifier == image_id][0]
         preferred_filename = [x.preferred_file_name for x in datas if x.identifier == image_id][0]
         all_paths = self.model.objects.file_store.keys()
-        image_pathname = [x for x in all_paths if x == f"Data/{stored_filename}"][0]
-        return (self.model.objects.file_store[image_pathname], preferred_filename)
+        image_pathnames = [x for x in all_paths if x == f"Data/{stored_filename}"]
+        if len(image_pathnames) == 0:
+            warn(
+                f"Cannot find file '{preferred_filename}' in Numbers archive",
+                RuntimeWarning,
+                stacklevel=3,
+            )
+        else:
+            return (self.model.objects.file_store[image_pathnames[0]], preferred_filename)
 
     def custom_format(self) -> str:
         if self.text_format_id is not None and self.type == CellType.TEXT: