Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error trying to open .xlsx file not created by Excel using ParseXLSX version 0.27 #7

Open
acggith opened this issue Jan 4, 2024 · 2 comments
Labels
feature New feature or request

Comments

@acggith
Copy link

acggith commented Jan 4, 2024

Bug related to parsing of file Example_Book1.xlsx using ParseXLSX version 0.27. The .xlsx file was NOT created with Excel.

perl code used to generate the bug:

use strict;
use warnings;
use Spreadsheet::ParseXLSX;

my $UUIDName='Example_Book1.xlsx';
my $parser = Spreadsheet::ParseXLSX->new();
my $workbook = $parser->parse($UUIDName);

Result:
Can't call method "att" on an undefined value at /usr/share/perl5/vendor_perl/Spreadsheet/ParseXLSX.pm line 900.

Comments:
1.) Bug is related to the variable $type_base in sub _extract_files of /usr/share/perl5/vendor_perl/Spreadsheet/ParseXLSX.pm.
2.) Opening and saving the file with Excel resolves the problem.

@MichaelDaum MichaelDaum self-assigned this Jan 4, 2024
@MichaelDaum MichaelDaum added bug Something isn't working feature New feature or request and removed bug Something isn't working labels Jan 4, 2024
@MichaelDaum
Copy link
Owner

MichaelDaum commented Jan 4, 2024

Basically purl.oclc.org (strict ooxml namespace) is not supported, only schemas.openxmlformats.org. Pull requests welcome.

MichaelDaum added a commit that referenced this issue Jan 4, 2024
... which fails atm as we don't support it yet. see issue #7
@MichaelDaum MichaelDaum removed their assignment Jan 8, 2024
@MichaelDaum
Copy link
Owner

The unit test in t/bug-md-7.t has been disabled temporarily unless this feature has been implemented.

MichaelDaum added a commit that referenced this issue Jan 17, 2024
reported by @phvietan, fixes issue #10

also:
  - added unit test for support of purl.oclc.org namespace ... see issue #7
  - removed tidyness tests again as it fails on weaver'ed code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants