Include Empty Rows

Hi,

First of all thank you for the module. Its one of the rare modules that

• support both xls and xlsx files
• gives the data cell by cell

One problem I found is that it ignores empty lines by default. My app’s business logic requires me to pick up rows based upon user-given index values (which include empty lines).

It would be great if you could make it config based rather the current approach.

I looked into the code and found I need to comment following lines in order enable empty lines:

           // if (columnData === '') {
           //     continue;
           // }

        // Cleaning empty
        // rows = rows.filter(v => v !== null && v !== undefined);

Can you please confirm that I am not missing any other lines?

Thanks.

[DiegoZoracKy]

Thanks for your answer and your PR @harshitgupta. It is already merged and published on npm (v 1.5.0)

@NunoChumbo might want to take a look.

[DiegoZoracKy]

You are right. It does that by default. What do you think about a config to indicate when it should return the empty values (being the current behavior of not returning the empty the default value)?

As you are already on the right snippet of the code, would you like to implement it?