[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-bug-tracker] [bug #43331] io-2.2.4 package - function xlsread ->
From: |
anonymous |
Subject: |
[Octave-bug-tracker] [bug #43331] io-2.2.4 package - function xlsread -> function __OCT_xlsx2oct__ in inst/private/__OCT_xlsx2oct__.m |
Date: |
Thu, 02 Oct 2014 11:15:06 +0000 |
User-agent: |
Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; WOW64; Trident/5.0) |
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/