darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-20 22:40:31 -04:00
Code Issues Packages Projects Releases Wiki Activity

Force xlsm files to open with pyexcel-xlsx

Browse Source 
`.xlsm` files are like `.xlxs` files but with macros enabled. They store
data in the same XML-based format as `.xlsx` files.

Pyexcel will try to use the xlrd package to parse `.xlsm` files. This
used to work because xlrd used to support reading `.xlsx` files. xlrd
has dropped support for `.xlsx` files in version 2 because of security
concerns. This means that when pyexcel asks xlrd to parse a `.xlsm` file
it causes an error.

This commit adds some branching to force `.xlsm` files to be opened
with pyexcel-xlsx instead, which does support `.xlsx` files.
This commit is contained in:
Chris Hill-Scott
2022-05-12 15:48:09 +01:00
parent ded87706c7
commit 80854ab2cc
3 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/app/main/views/test_send.py
View File

@@ -300,8 +300,8 @@ def test_set_sender_redirects_if_one_sms_sender(
def test_that_test_files_exist():
assert len(test_spreadsheet_files) == 7
assert len(test_non_spreadsheet_files) == 7
assert len(test_spreadsheet_files) == 8
assert len(test_non_spreadsheet_files) == 6
def test_should_not_allow_files_to_be_uploaded_without_the_correct_permission(