sheet naming issue temp fix

[sabbiu]

I found an excel file where fileNames.sheets[sheetRelationId] = /xl/worksheets/sheet1.xml

Content file

[catamphetamine]

Hmm. Can you tell more about the file. Do you know what software was used to generate it?

[sabbiu]

The file that I was provided, unfortunately cannot be shared, because it contains proprietary information. I am trying to replicate the issue by replacing it with test data, and I am not successful to replicate the issue for now.

I will add further information as soon as I find someway of replicating the issue.

[catamphetamine]

The file that I was provided, unfortunately cannot be shared, because it contains proprietary information. I am trying to replicate the issue by replacing it with test data, and I am not successful to replicate the issue for now.

No need to share, just ask them what software did they use for generating the file.

[sabbiu]

Sure, I'll get back to you regarding that.

[catamphetamine]

So far, I'm not convinced that a workaround like this should exist. I'd blame it on the software that was used to generate the file, unless it's something popular which it doesn't seem like it is. Perhaps it was generated programmatically by some other library or a home-written script.

[sabbiu]

AFAIK, it was generated using software. I was also able to import this file in google sheets, and it was working fine. But I will comment as soon I am provided with more details.

[catamphetamine]

You could also see if some other libraries open such files, like "sheet js", for example. https://sheetjs.com/demo

[sabbiu]

It works with sheetjs as well.

[catamphetamine]

See if this code change works:

function parseFileNames(content, xml) {
  const document = ...
  const fileNames = ...

    const filePath = relationship.getAttribute('Target')
    // There has been one weird case when file path started with a `/`.
    // https://github.com/catamphetamine/read-excel-file/pull/95
      .replace(/^\//, '')

  ...
}

in readXlsx.js

[sabbiu]

It works, but I had to make changes here,

https://github.com/catamphetamine/read-excel-file/blob/master/source/read/readXlsx.js#L583

const filePath = relationship.getAttribute('Target').replace(/^\/xl\//, '')

[catamphetamine]

That would be strange because the issue you're having seems to be a leading slash that shouldn't be there. Then why're you also stripping the xl part.

[sabbiu]

As you can see here,

https://github.com/catamphetamine/read-excel-file/blob/master/source/read/readXlsx.js#L87

It is prepending the xl part, and thus I have to remove it during parsing :sweat_smile:

[catamphetamine]

published read-excel-file@5.2.7 https://gitlab.com/catamphetamine/read-excel-file/-/commit/f1cfbe40d1ae80696e5da1c6253373383e836349