doy / spreadsheet-parsexlsx

parse XLSX files
http://metacpan.org/release/Spreadsheet-ParseXLSX
27 stars 35 forks source link

unvisible part of merged cell is not undefined #16

Closed Tux closed 10 years ago

Tux commented 10 years ago

in Spreadsheet::ParseExcel, if A2 and A3 are merged, A2 will return the value and A3 will be undef. In ParseXLSX, A2 reveals the value, but A3 return the empty string ""

doy commented 10 years ago

Are you sure? Doing some basic testing with Spreadsheet::ParseExcel seems to indicate that it also returns an empty string. Can you give me an example?

Tux commented 10 years ago

On Wed, 22 Jan 2014 09:35:53 -0800, Jesse Luehrs notifications@github.com wrote:

Are you sure? Doing some basic testing with Spreadsheet::ParseExcel seems to indicate that it also returns an empty string. Can you give me an example?

You are right. The difference does stem from Spreadsheet::Read, which returns an empty string, but from Spreadsheet::XLSX, which returns an undef. I will fix my tests.

H.Merijn Brand http://tux.nl Perl Monger http://amsterdam.pm.org/ using perl5.00307 .. 5.19 porting perl5 on HP-UX, AIX, and openSUSE http://mirrors.develooper.com/hpux/ http://www.test-smoke.org/ http://qa.perl.org http://www.goldmark.org/jeff/stupid-disclaimers/