Skip to content

Commit

Permalink
Merge pull request #383 from OpenDataServices/ods-bug
Browse files Browse the repository at this point in the history
Fix another number formatting .ods bug
  • Loading branch information
Bjwebb authored Mar 24, 2021
2 parents 57af89b + e1f58ec commit 69e906b
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 3 deletions.
10 changes: 10 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,16 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

## [Unreleased]

## [0.16.0] - 2021-03-24

### Fixed

- Fix another number formatting .ods bug https://github.com/OpenDataServices/flatten-tool/pull/383

### Changed

- Reduce memory footprint of flattening https://github.com/OpenDataServices/flatten-tool/pull/376

## [0.15.4] - 2021-03-08

### Fixed
Expand Down
2 changes: 1 addition & 1 deletion flattentool/ODSReader.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ def readSheet(self, sheet):
"value",
)
)
if "." in str(cell):
if "." in str(value):
arrCells[count] = float(value)
else:
arrCells[count] = int(value)
Expand Down
4 changes: 3 additions & 1 deletion flattentool/tests/test_input_SpreadsheetInput.py
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,9 @@ def test_xlsx_input_types(self):
"colFloat": 1000.2,
"colFloatComma": 1000.2,
"colDate": datetime.datetime(2020, 3, 5),
"colDateTime": datetime.datetime(2020, 2, 7, 16, 41, 0, 1),
"colDateTime": datetime.datetime(
2020, 2, 7, 16, 41, 0, 1 if sys.version_info < (3, 6) else 0
),
None: None,
}
]
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ def run(self):

setup(
name="flattentool",
version="0.15.4",
version="0.16.0",
author="Open Data Services",
author_email="code@opendataservices.coop",
packages=["flattentool"],
Expand Down

0 comments on commit 69e906b

Please sign in to comment.