Empty cells with style aren't skipped anymore

[azukaar]

Fixing #7

[protobi]

@azukaar Great catch, thanks!

The way the code is structured I'll make the change to the source file, which is located in bits/67_wsxml.js and that'll get compiled to ./xlsx.js.

Before I do that, can you send or describe a test case that I can use to (a) verify it fails now and (b) see it succeed after implementing? The comment suggests this was a feature that was dropped but I don't think I changed that function, so I just want to be careful to get it right.

[azukaar]

@protobi
Sorry about the title may be the "anymore" might be misleading, I just meant that after the fix they aren't skipped anymore.
To test a fail case, simply create a file with obvious pattern :

Then open it and resave it using js-xlsx :

window.workbookTemplate = XLSX.read(e.target.result, {type:"binary" ,cellStyles : true, bookVBA : true});

Then

wbout = XLSX.write(window.workbookTemplate,
    bookType: 'xlsx'
    bookSST: false
    type: 'binary')
saveAs new Blob([  s2ab(wbout)  ], type: 'application/octet-stream'), "test.xlsx"

Finally reopen the result in Excel and it will be :

This is actually a real issue for a lot of Excel file, for exemple :

Here, those strokes will disappear no matter what, because the inset cell will always be empty even after the report will be filled.
Oh also, sorry about fixing the bad file, I wasn't quite confident about the structure of the project !

[protobi]

@azukaar Very helpful. Your changes are made and addressed under issue #7