<changes>
<release version="3.9-beta1" date="2012-??-??">
+ <action dev="poi-developers" type="fix">53282 - Avoid exception when parsing OPC relationships with non-breaking spaces</action>
<action dev="poi-developers" type="fix">54016 - Avoid exception when parsing workbooks with DConRefRecord in row aggregate</action>
<action dev="poi-developers" type="fix">54008 - Fixed Ant build to support build directories with blanks</action>
<action dev="poi-developers" type="fix">53374 - Avoid exceptions when parsing hyperlinks of type "javascript://"</action>
value = path + "#" + encode(fragment);
}
+ // trailing white spaces must be url-encoded, see Bugzilla 53282
+ if(value.length() > 0 ){
+ StringBuilder b = new StringBuilder();
+ int idx = value.length() - 1;
+ for(; idx >= 0; idx--){
+ char c = value.charAt(idx);
+ if(Character.isWhitespace(c) || c == '\u00A0') {
+ b.append(c);
+ } else {
+ break;
+ }
+ }
+ if(b.length() > 0){
+ value = value.substring(0, idx+1) + encode(b.reverse().toString());
+ }
+ }
+
+ // MS Office can insert URIs with missing authority, e.g. "http://" or "javascript://"
+ // append a forward slash to avoid parse exception
if(missingAuthPattern.matcher(value).matches()){
value += "/";
}
};
private static boolean isUnsafe(int ch) {
- return ch > 0x80 || " ".indexOf(ch) >= 0;
+ return ch > 0x80 || Character.isWhitespace(ch) || ch == '\u00A0';
}
}
import java.io.*;
import java.net.URI;
+import java.util.regex.Pattern;
import junit.framework.TestCase;
assertEquals(rel1.getTargetURI(), rel2.getTargetURI());
assertEquals(rel1.getTargetMode(), rel2.getTargetMode());
}
+
+ public void testTrailingSpacesInURI_53282() throws Exception {
+ InputStream is = OpenXML4JTestDataSamples.openSampleStream("53282.xlsx");
+ OPCPackage pkg = OPCPackage.open(is);
+ is.close();
+
+ PackageRelationshipCollection sheetRels = pkg.getPartsByName(Pattern.compile("/xl/worksheets/sheet1.xml")).get(0).getRelationships();
+ assertEquals(3, sheetRels.size());
+ PackageRelationship rId1 = sheetRels.getRelationshipByID("rId1");
+ assertEquals(TargetMode.EXTERNAL, rId1.getTargetMode());
+ URI targetUri = rId1.getTargetURI();
+ assertEquals("mailto:nobody@nowhere.uk%C2%A0", targetUri.toASCIIString());
+ assertEquals("nobody@nowhere.uk\u00A0", targetUri.getSchemeSpecificPart());
+
+ ByteArrayOutputStream out = new ByteArrayOutputStream();
+ pkg.save(out);
+ out.close();
+
+ pkg = OPCPackage.open(new ByteArrayInputStream(out.toByteArray()));
+ sheetRels = pkg.getPartsByName(Pattern.compile("/xl/worksheets/sheet1.xml")).get(0).getRelationships();
+ assertEquals(3, sheetRels.size());
+ rId1 = sheetRels.getRelationshipByID("rId1");
+ assertEquals(TargetMode.EXTERNAL, rId1.getTargetMode());
+ targetUri = rId1.getTargetURI();
+ assertEquals("mailto:nobody@nowhere.uk%C2%A0", targetUri.toASCIIString());
+ assertEquals("nobody@nowhere.uk\u00A0", targetUri.getSchemeSpecificPart());
+
+ }
}
link = wb.getSheetAt(0).getRow(0).getCell(0).getHyperlink();
assertEquals("javascript:///", link.getAddress());
}
+
+ public void test53282() {
+ XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook("53282.xlsx");
+ XSSFHyperlink link = wb.getSheetAt(0).getRow(0).getCell(14).getHyperlink();
+ assertEquals("mailto:nobody@nowhere.uk%C2%A0", link.getAddress());
+
+ wb = XSSFTestDataSamples.writeOutAndReadBack(wb);
+ link = wb.getSheetAt(0).getRow(0).getCell(14).getHyperlink();
+ assertEquals("mailto:nobody@nowhere.uk%C2%A0", link.getAddress());
+ }
}