{"id":2792,"date":"2016-03-18T13:50:25","date_gmt":"2016-03-18T13:50:25","guid":{"rendered":"http:\/\/www.ericwhite.com\/home2\/bm8qcmjy\/public_html\/blog\/?page_id=2792"},"modified":"2016-03-18T13:50:25","modified_gmt":"2016-03-18T13:50:25","slug":"handling-invalid-hyperlinks-openxmlpackageexception-in-the-open-xml-sdk","status":"publish","type":"page","link":"https:\/\/www.ericwhite.com\/blog\/handling-invalid-hyperlinks-openxmlpackageexception-in-the-open-xml-sdk\/","title":{"rendered":"Handling Invalid Hyperlinks (OpenXmlPackageException) in the Open XML SDK"},"content":{"rendered":"

One of the pernicious problems around the Open XML SDK is that if a document contains a hyperlink that has an invalid Uri<\/b>, then Word, Excel, or PowerPoint will happily open the document, whereas System.IO.Packaging<\/b> (and therefore the Open XML SDK) will throw an exception.<\/p>\n

The problem that we face with this is that we can\u2019t fix this in the Open XML SDK; it is thrown from deep inside the classes in System.IO.Packaging<\/b>.  This problem occurs because System.IO.Packaging<\/b> creates a Uri <\/b>object for every external relationship, and if the target of that relationship is not a valid Uri<\/b>, then we see the thrown exception.  System.IO.Packaging<\/b> has been frozen for quite some time, and it currently isn\u2019t possible to submit this as a bug and expect it to be fixed.  This is understandable in that there are hundreds of thousands of programs that rely on these classes, and changing semantics in the smallest way very well might cause compatibility problems.<\/p>\n

However, there is a way to work around this in a fairly clean way.  I have written a small method that uses the classes in System.IO.Compression <\/b>to open an Open XML document (of any type \u2013 DOCX, XLSX, or PPTX), examine all external relationships and if the relationship does not contain a valid Uri<\/b>, the method calls a callback with the invalid target.  You can write this callback to return any valid Uri<\/b> that you want, so long as it is a valid Uri<\/b>.  A possible candidate would be the target of http:\/\/broken-link\/<\/b>.  The FixInvalidUri <\/b>method then updates the target of the external relationship with the valid Uri<\/b>.  You can then open and process the document as usual.<\/p>\n

Following is the complete listing of the class UriFixer<\/b>, as well as the code to use it.  The approach that you take when using this class is to first attempt to open the document as usual, catching OpenXmlPackageException<\/b>.  If that exception is thrown, and if the text of that exception contains \u201cInvalid Hyperlink\u201d, then the code calls UriFixer.FixInvalidUri<\/b>.  After calling FixInvalidUri<\/b>, the code then opens the fixed document (or spreadsheet \/ presentation) as usual.<\/p>\n

using<\/span> <\/span>System<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>Collections<\/span>.<\/span>Generic<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>IO<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>IO<\/span>.<\/span>Compression<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>Linq<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>Text<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>Threading<\/span>.<\/span>Tasks<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>Xml<\/span>;<\/span>
<\/span>using<\/span> <\/span>System<\/span>.<\/span>Xml<\/span>.<\/span>Linq<\/span>;<\/span>
<\/span>using<\/span> <\/span>DocumentFormat<\/span>.<\/span>OpenXml<\/span>.<\/span>Packaging<\/span>;<\/span>

<\/span>class<\/span> <\/span>Program<\/span>
<\/span>{<\/span>
    <\/span>static<\/span> <\/span>void<\/span> <\/span>Main<\/span>(<\/span>string<\/span>[]<\/span> args<\/span>)<\/span>
    <\/span>{<\/span>
        <\/span>var<\/span> fileName <\/span>=<\/span> <\/span>@<\/span>\"..\\..\\..\\Test.docx\"<\/span>;<\/span>
        <\/span>var<\/span> newFileName <\/span>=<\/span> <\/span>@<\/span>\"..\\..\\..\\Fixed.docx\"<\/span>;<\/span>
        <\/span>var<\/span> newFileInfo <\/span>=<\/span> <\/span>new<\/span> <\/span>FileInfo<\/span>(<\/span>newFileName<\/span>);<\/span>

        <\/span>if<\/span> <\/span>(<\/span>newFileInfo<\/span>.<\/span>Exists<\/span>)<\/span>
            newFileInfo<\/span>.<\/span>Delete<\/span>();<\/span>

        <\/span>File<\/span>.<\/span>Copy<\/span>(<\/span>fileName<\/span>,<\/span> newFileName<\/span>);<\/span>

        <\/span>WordprocessingDocument<\/span> wDoc<\/span>;<\/span>
        <\/span>try<\/span>
        <\/span>{<\/span>
            <\/span>using<\/span> <\/span>(<\/span>wDoc <\/span>=<\/span> <\/span>WordprocessingDocument<\/span>.<\/span>Open<\/span>(<\/span>newFileName<\/span>,<\/span> <\/span>true<\/span>))<\/span>
            <\/span>{<\/span>
                <\/span>ProcessDocument<\/span>(<\/span>wDoc<\/span>);<\/span>
            <\/span>}<\/span>
        <\/span>}<\/span>
        <\/span>catch<\/span> <\/span>(<\/span>OpenXmlPackageException<\/span> e<\/span>)<\/span>
        <\/span>{<\/span>
            <\/span>if<\/span> <\/span>(<\/span>e<\/span>.<\/span>ToString<\/span>().<\/span>Contains<\/span>(<\/span>\"Invalid Hyperlink\"<\/span>))<\/span>
            <\/span>{<\/span>
                <\/span>using<\/span> <\/span>(<\/span>FileStream<\/span> fs <\/span>=<\/span> <\/span>new<\/span> <\/span>FileStream<\/span>(<\/span>newFileName<\/span>,<\/span> <\/span>FileMode<\/span>.<\/span>OpenOrCreate<\/span>,<\/span> <\/span>FileAccess<\/span>.<\/span>ReadWrite<\/span>))<\/span>
                <\/span>{<\/span>
                    <\/span>UriFixer<\/span>.<\/span>FixInvalidUri<\/span>(<\/span>fs<\/span>,<\/span> brokenUri <\/span>=><\/span> <\/span>FixUri<\/span>(<\/span>brokenUri<\/span>));<\/span>
                <\/span>}<\/span>
                <\/span>using<\/span> <\/span>(<\/span>wDoc <\/span>=<\/span> <\/span>WordprocessingDocument<\/span>.<\/span>Open<\/span>(<\/span>newFileName<\/span>,<\/span> <\/span>true<\/span>))<\/span>
                <\/span>{<\/span>
                    <\/span>ProcessDocument<\/span>(<\/span>wDoc<\/span>);<\/span>
                <\/span>}<\/span>
            <\/span>}<\/span>
        <\/span>}<\/span>
    <\/span>}<\/span>

    <\/span>private<\/span> <\/span>static<\/span> <\/span>Uri<\/span> <\/span>FixUri<\/span>(<\/span>string<\/span> brokenUri<\/span>)<\/span>
    <\/span>{<\/span>
        <\/span>return<\/span> <\/span>new<\/span> <\/span>Uri<\/span>(<\/span>\"http:\/\/broken-link\/\"<\/span>);<\/span>
    <\/span>}<\/span>

    <\/span>private<\/span> <\/span>static<\/span> <\/span>void<\/span> <\/span>ProcessDocument<\/span>(<\/span>WordprocessingDocument<\/span> wDoc<\/span>)<\/span>
    <\/span>{<\/span>
        <\/span>var<\/span> elementCount <\/span>=<\/span> wDoc<\/span>.<\/span>MainDocumentPart<\/span>.<\/span>Document<\/span>.<\/span>Descendants<\/span>().<\/span>Count<\/span>();<\/span>
        <\/span>Console<\/span>.<\/span>WriteLine<\/span>(<\/span>elementCount<\/span>);<\/span>
    <\/span>}<\/span>
<\/span>}<\/span>

<\/span>public<\/span> <\/span>static<\/span> <\/span>class<\/span> <\/span>UriFixer<\/span>
<\/span>{<\/span>
    <\/span>public<\/span> <\/span>static<\/span> <\/span>void<\/span> <\/span>FixInvalidUri<\/span>(<\/span>Stream<\/span> fs<\/span>,<\/span> <\/span>Func<\/span><<\/span>string<\/span>,<\/span> <\/span>Uri<\/span>><\/span> invalidUriHandler<\/span>)<\/span>
    <\/span>{<\/span>
        <\/span>XNamespace<\/span> relNs <\/span>=<\/span> <\/span>\"http:\/\/schemas.openxmlformats.org\/package\/2006\/relationships\"<\/span>;<\/span>
        <\/span>using<\/span> <\/span>(<\/span>ZipArchive<\/span> za <\/span>=<\/span> <\/span>new<\/span> <\/span>ZipArchive<\/span>(<\/span>fs<\/span>,<\/span> <\/span>ZipArchiveMode<\/span>.<\/span>Update<\/span>))<\/span>
        <\/span>{<\/span>
            <\/span>foreach<\/span> <\/span>(<\/span>var<\/span> entry <\/span>in<\/span> za<\/span>.<\/span>Entries<\/span>.<\/span>ToList<\/span>())<\/span>
            <\/span>{<\/span>
                <\/span>if<\/span> <\/span>(!<\/span>entry<\/span>.<\/span>Name<\/span>.<\/span>EndsWith<\/span>(<\/span>\".rels\"<\/span>))<\/span>
                    <\/span>continue<\/span>;<\/span>
                <\/span>bool<\/span> replaceEntry <\/span>=<\/span> <\/span>false<\/span>;<\/span>
                <\/span>XDocument<\/span> entryXDoc <\/span>=<\/span> <\/span>null<\/span>;<\/span>
                <\/span>using<\/span> <\/span>(<\/span>var<\/span> entryStream <\/span>=<\/span> entry<\/span>.<\/span>Open<\/span>())<\/span>
                <\/span>{<\/span>
                    <\/span>try<\/span>
                    <\/span>{<\/span>
                        entryXDoc <\/span>=<\/span> <\/span>XDocument<\/span>.<\/span>Load<\/span>(<\/span>entryStream<\/span>);<\/span>
                        <\/span>if<\/span> <\/span>(<\/span>entryXDoc<\/span>.<\/span>Root<\/span> <\/span>!=<\/span> <\/span>null<\/span> <\/span>&&<\/span> entryXDoc<\/span>.<\/span>Root<\/span>.<\/span>Name<\/span>.<\/span>Namespace<\/span> <\/span>==<\/span> relNs<\/span>)<\/span>
                        <\/span>{<\/span>
                            <\/span>var<\/span> urisToCheck <\/span>=<\/span> entryXDoc
                                <\/span>.<\/span>Descendants<\/span>(<\/span>relNs <\/span>+<\/span> <\/span>\"Relationship\"<\/span>)<\/span>
                                <\/span>.<\/span>Where<\/span>(<\/span>r <\/span>=><\/span> r<\/span>.<\/span>Attribute<\/span>(<\/span>\"TargetMode\"<\/span>)<\/span> <\/span>!=<\/span> <\/span>null<\/span> <\/span>&&<\/span> <\/span>(<\/span>string<\/span>)<\/span>r<\/span>.<\/span>Attribute<\/span>(<\/span>\"TargetMode\"<\/span>)<\/span> <\/span>==<\/span> <\/span>\"External\"<\/span>);<\/span>
                            <\/span>foreach<\/span> <\/span>(<\/span>var<\/span> rel <\/span>in<\/span> urisToCheck<\/span>)<\/span>
                            <\/span>{<\/span>
                                <\/span>var<\/span> target <\/span>=<\/span> <\/span>(<\/span>string<\/span>)<\/span>rel<\/span>.<\/span>Attribute<\/span>(<\/span>\"Target\"<\/span>);<\/span>
                                <\/span>if<\/span> <\/span>(<\/span>target <\/span>!=<\/span> <\/span>null<\/span>)<\/span>
                                <\/span>{<\/span>
                                    <\/span>try<\/span>
                                    <\/span>{<\/span>
                                        <\/span>Uri<\/span> uri <\/span>=<\/span> <\/span>new<\/span> <\/span>Uri<\/span>(<\/span>target<\/span>);<\/span>
                                    <\/span>}<\/span>
                                    <\/span>catch<\/span> <\/span>(<\/span>UriFormatException<\/span>)<\/span>
                                    <\/span>{<\/span>
                                        <\/span>Uri<\/span> newUri <\/span>=<\/span> invalidUriHandler<\/span>(<\/span>target<\/span>);<\/span>
                                        rel<\/span>.<\/span>Attribute<\/span>(<\/span>\"Target\"<\/span>).<\/span>Value<\/span> <\/span>=<\/span> newUri<\/span>.<\/span>ToString<\/span>();<\/span>
                                        replaceEntry <\/span>=<\/span> <\/span>true<\/span>;<\/span>
                                    <\/span>}<\/span>
                                <\/span>}<\/span>
                            <\/span>}<\/span>
                        <\/span>}<\/span>
                    <\/span>}<\/span>
                    <\/span>catch<\/span> <\/span>(<\/span>XmlException<\/span>)<\/span>
                    <\/span>{<\/span>
                        <\/span>continue<\/span>;<\/span>
                    <\/span>}<\/span>
                <\/span>}<\/span>
                <\/span>if<\/span> <\/span>(<\/span>replaceEntry<\/span>)<\/span>
                <\/span>{<\/span>
                    <\/span>var<\/span> fullName <\/span>=<\/span> entry<\/span>.<\/span>FullName<\/span>;<\/span>
                    entry<\/span>.<\/span>Delete<\/span>();<\/span>
                    <\/span>var<\/span> newEntry <\/span>=<\/span> za<\/span>.<\/span>CreateEntry<\/span>(<\/span>fullName<\/span>);<\/span>
                    <\/span>using<\/span> <\/span>(<\/span>StreamWriter<\/span> writer <\/span>=<\/span> <\/span>new<\/span> <\/span>StreamWriter<\/span>(<\/span>newEntry<\/span>.<\/span>Open<\/span>()))<\/span>
                    <\/span>using<\/span> <\/span>(<\/span>XmlWriter<\/span> xmlWriter <\/span>=<\/span> <\/span>XmlWriter<\/span>.<\/span>Create<\/span>(<\/span>writer<\/span>))<\/span>
                    <\/span>{<\/span>
                        entryXDoc<\/span>.<\/span>WriteTo<\/span>(<\/span>xmlWriter<\/span>);<\/span>
                    <\/span>}<\/span>
                <\/span>}<\/span>
            <\/span>}<\/span>
        <\/span>}<\/span>
    <\/span>}<\/span>
<\/span>}<\/span><\/pre>\n
We are considering including this method in the Open XML SDK itself.  We would make a few overloads of the WordprocessingDocument.Open<\/b> method, the SpreadsheetDocument.Open<\/b> method, and the PresentationDocument.Open<\/b> method.  These overloads would take the callback as an argument, just as in the above example.  These new methods would first attempt to open the document in the normal way.  If the attempt to open is successful, then these methods would return the newly opened document.  However, if System.IO.Packaging <\/b>throws the OpenXmlPackageException<\/b>, and if the document were opened for writing, then the method would open, modify, and save a fixed document.  It would then attempt to open again, and return the newly opened document.<\/span><\/p>\n
With this approach, the idiom to open the document would be almost identical to the current approach to opening a document.  The only difference would be the inclusion of the callback method as an argument.<\/p>\n
If the document was opened for read-only access, then the various methods would create a copy of the document in memory, fix the broken Uri<\/b> objects, and then open and return the fixed document (for read-only access).<\/p>\n
Please feel free to comment about how this approach would work for you.  If we have agreement on this approach, then in a month or two, we will make the change to the open source version of the Open XML SDK.<\/p>\n","protected":false},"excerpt":{"rendered":"
One of the pernicious problems around the Open XML SDK is that if a document contains a hyperlink that has an invalid Uri, then Word, Excel, or PowerPoint will happily open the document, whereas System.IO.Packaging (and therefore the Open XML SDK) will throw an exception. The problem that we face with this is that we […]<\/p>\n","protected":false},"author":10567,"featured_media":0,"parent":0,"menu_order":0,"comment_status":"closed","ping_status":"closed","template":"","meta":{"_bbp_topic_count":0,"_bbp_reply_count":0,"_bbp_total_topic_count":0,"_bbp_total_reply_count":0,"_bbp_voice_count":0,"_bbp_anonymous_reply_count":0,"_bbp_topic_count_hidden":0,"_bbp_reply_count_hidden":0,"_bbp_forum_subforum_count":0,"_s2mail":"","footnotes":""},"class_list":["post-2792","page","type-page","status-publish","hentry"],"_links":{"self":[{"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/pages\/2792","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/pages"}],"about":[{"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/types\/page"}],"author":[{"embeddable":true,"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/users\/10567"}],"replies":[{"embeddable":true,"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/comments?post=2792"}],"version-history":[{"count":1,"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/pages\/2792\/revisions"}],"predecessor-version":[{"id":2794,"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/pages\/2792\/revisions\/2794"}],"wp:attachment":[{"href":"https:\/\/www.ericwhite.com\/blog\/wp-json\/wp\/v2\/media?parent=2792"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}