Handle relationships with targets given as absolute path

Author: xavier

lib/xlsx_reader/parsers/relationships_parser.ex

@@ -42,7 +42,7 @@ defmodule XlsxReader.Parsers.RelationshipsParser do
   def handle_event(:start_element, {"Relationship", attributes}, state) do
     with %{id: id, target: target, type: type} <- extract_relationship_attributes(attributes),
          {:ok, key} <- Map.fetch(@types, type) do
-      {:ok, Map.update!(state, key, fn rels -> Map.put_new(rels, id, target) end)}
+      {:ok, Map.update!(state, key, fn rels -> Map.put_new(rels, id, sanitize_target(target)) end)}
     else
       _ ->
         {:ok, state}
@@ -75,4 +75,8 @@ defmodule XlsxReader.Parsers.RelationshipsParser do
   defp extract_relationship_attributes(attributes) do
     Utils.map_attributes(attributes, @relationship_attributes_mapping)
   end
+
+  # Remove leading "/" to deal with file containing absolute paths
+  defp sanitize_target("/" <> target), do: target
+  defp sanitize_target(target), do: target
 end

test/fixtures/package/xl/_rels/workbook.xml.rels

@@ -5,5 +5,6 @@
   <Relationship Id="rId3" Target="theme/theme1.xml" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/theme"/>
   <Relationship Id="rId4" Target="worksheets/sheet1.xml" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet"/>
   <Relationship Id="rId5" Target="worksheets/sheet2.xml" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet"/>
-  <Relationship Id="rId6" Target="worksheets/sheet3.xml" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet"/>
+  <!-- Leading / added to test sanitization of absolute path -->
+  <Relationship Id="rId6" Target="/worksheets/sheet3.xml" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet"/>
 </Relationships>