[Octave-bug-tracker] [bug #43331] io-2.2.4 package - function xlsread -> function __OCT_xlsx2oct__ in inst/private/__OCT_xlsx2oct__.m

[anonymous]

Follow-up Comment #5, bug #43331 (project octave):

Dear Philip&Markus,

the quick fix works (patch #32206) for the time being. Thanks!

BUT: I would recommend extracting the entire Target="..." attribute from the
worksheet.xml.rels file's <Relationship .../> tag. In the described special
case of the bug43331-addnumber.xlsx file, the r:id="+b+...-b-" happens to be
the same as the sheet+b+...-b-.xml number, but this is not required.

What IS required from the ISO specs is that the r:Id attribute value of the
workbook.xml <sheet .../> tag has a corresponding Id attribute value in the
workbook.xml.rels file's <Relationship .../> tag.

So, in my view, the bug could be fixed by having xls.sheets.rid(id) return
"rId1" (in the case of bug43331-addnumber.xlsx) rather than only "1". Then,
this "rId1" is searched for in workbook.xml.rels as the corresponding Id
attribute value of the <Relationship .../> tag, and the entire attribute value
of Target is fed into the function call for the worksheet =
fopen(sprintf('%s', xls.workbook, +b+wsh-b-)), where +b+wsh-b- would have to
be replaced by the attribute value of Target.

Hope that helps.

Best,
Marco

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?43331>

_______________________________________________
  Nachricht gesendet von/durch Savannah
  http://savannah.gnu.org/