jandrew / Spreadsheet-XLSX-Reader-LibXML

Read spreadsheet files with xlsx extentions
Other
4 stars 2 forks source link

$setting_node undef at the Workbook level #74

Closed jandrew closed 9 years ago

jandrew commented 9 years ago

Submitted via email

Second, a very similar change in LibXML.pm when $setting_node would be undef:

--- DIST/LibXML.pm 2015-08-24 19:49:26.000000000 -0400 +++ SITE_PERL/LibXML.pm 2015-09-14 10:10:20.000000000 -0400 @@ -731,7 +731,7 @@ my ( $list, $sheet_ref, $rel_lookup, $id_lookup ); my $position = 0; my ( $setting_node ) = $dom->getElementsByTagName( 'workbookPr' );

Thanks Todd

jandrew commented 9 years ago

I will try and assign a milestone once I asses the test case

jandrew commented 9 years ago

OK, This is another one that the work of noodling a test file for fail -> pass testing seems more effort than it's worth since the test is just an additional data safe check.

jandrew commented 9 years ago

Closed with release of v0.38.16