From 8adccebb0e4f70483fac9eb71d59530cd26418f2 Mon Sep 17 00:00:00 2001
From: Jan Marvin Garbuszus <jan.garbuszus@rub.de>
Date: Sun, 15 Sep 2024 20:01:11 +0200
Subject: [PATCH] [clone] fix cloning hyperlinks

---
 R/class-workbook.R                   | 59 +++++++++++++++-------------
 tests/testthat/test-cloneWorksheet.R |  6 +--
 2 files changed, 35 insertions(+), 30 deletions(-)

diff --git a/R/class-workbook.R b/R/class-workbook.R
index 5157d1cd4..409228928 100644
--- a/R/class-workbook.R
+++ b/R/class-workbook.R
@@ -3715,47 +3715,52 @@ wbWorkbook <- R6::R6Class(
 
       self$worksheets[[sheet]]$sheet_data$cc <- cc
 
-      # # FIXME need to think about this some more.
-      # ### add hyperlinks ---
-      # if (length(self$worksheets[[from_sheet]]$relships$hyperlinks)) {
+      # FIXME need to think about this some more.
+      ### add hyperlinks ---
+      if (length(self$worksheets[[from_sheet]]$relships$hyperlink)) {
 
-      #   ws_hyls <- self$worksheets[[from_sheet]]$hyperlinks
-      #   ws_rels <- self$worksheets_rels[[self$worksheets[[from_sheet]]$relships$hyperlinks]]
+        ws_hyls <- self$worksheets[[from_sheet]]$hyperlinks
+        ws_rels <- self$worksheets_rels[[self$worksheets[[from_sheet]]$relships$hyperlink]]
 
-      #   relships <- rbindlist(xml_attr(ws_rels, "Relationship"))
-      #   relships <- relships[basename(relships$Type) == "hyperlink", ]
+        relships <- rbindlist(xml_attr(ws_rels, "Relationship"))
+        relships <- relships[basename(relships$Type) == "hyperlink", ]
 
-      #   # prepare hyperlinks data frame
-      #   hlinks <- rbindlist(xml_attr(ws_hyls, "hyperlink"))
+        # prepare hyperlinks data frame
+        hlinks <- rbindlist(xml_attr(ws_hyls, "hyperlink"))
 
-      #   # merge both
-      #   hl_df <- merge(hlinks, relships, by.x = "r:id", by.y = "Id", all.x = TRUE, all.y = FALSE)
+        # merge both
+        hl_df <- merge(hlinks, relships, by.x = "r:id", by.y = "Id", all.x = TRUE, all.y = FALSE)
 
-      #   hyperlink_in_wb <- hlinks$ref
+        hyperlink_in_wb <- hlinks$ref
 
-      #   if (any(sel <- hyperlink_in_wb %in% from_dims)) {
+        if (any(sel <- hyperlink_in_wb %in% from_dims)) {
 
-      #     has_hl <- apply(from_dims_df, 2, function(x) x %in% hyperlink_in_wb)
+          has_hl <- apply(from_dims_df, 2, function(x) x %in% hyperlink_in_wb)
 
-      #     old <- from_dims_df[has_hl]
-      #     new <- to_dims_df_f[has_hl]
+          # are these always the same size?
+          old <- from_dims_df[has_hl]
+          new <- to_dims_df_f[has_hl]
 
-      #     for (hls in match(hyperlink_in_wb, old)) {
+          for (hls in match(hyperlink_in_wb, old)) {
 
-      #       # prepare the updated link
+            # prepare the updated link
+            need_clone <- hyperlink_in_wb[hls]
 
-      #       which(hls)
+            hl_df <- hlinks[hlinks$ref == need_clone, ]
+            # this assumes that old and new are the same size
+            hl_df$ref <- new[hls]
+            hl <- df_to_xml("hyperlink", hl_df)
 
-      #       # assign it
-      #       self$worksheets[[sheet]]$hyperlinks <- append(
-      #         self$worksheets[[sheet]]$hyperlinks,
-      #         hl
-      #       )
-      #     }
+            # assign it
+            self$worksheets[[sheet]]$hyperlinks <- append(
+              self$worksheets[[sheet]]$hyperlinks,
+              hl
+            )
+          }
 
-      #   }
+        }
 
-      # }
+      }
 
       invisible(self)
     },
diff --git a/tests/testthat/test-cloneWorksheet.R b/tests/testthat/test-cloneWorksheet.R
index 13a5c4dc3..e2849ea97 100644
--- a/tests/testthat/test-cloneWorksheet.R
+++ b/tests/testthat/test-cloneWorksheet.R
@@ -119,9 +119,9 @@ test_that("copy_cells works with hyperlinks and empty cells in transpose", {
   dat <- wb_data(wb_in, 1, dims = "A1:B3", col_names = FALSE)
   wb_in$copy_cells(data = dat, dims = "D1", transpose = TRUE)
 
-  # exp <- c("A1", "A3", "D1", "F1")
-  # got <- vapply(wb_in$worksheets[[1]]$hyperlinks, function(x) x$ref, "")
-  # expect_equal(exp, got)
+  exp <- c("A1", "A3", "D1", "F1")
+  got <- rbindlist(xml_attr(wb_in$worksheets[[1]]$hyperlinks, "hyperlink"))$ref
+  expect_equal(exp, got)
 
   cc <- wb_in$worksheets[[1]]$sheet_data$cc
   exp <- rep("0", 4)