Merge pull request #69 from rpact-com/dev/4.1.1

NEWS.md

@@ -15,7 +15,8 @@
   For more information, please visit: [rpact.org/iq](www.rpact.org/iq)
 * Usage of `maxInformation` improved (see enhancement [#65](https://github.com/rpact-com/rpact/issues/65))
 * `testPackage()`: additional warning details will be added to the test report if warnings exist
-* Issue [#61](https://github.com/rpact-com/rpact/issues/61)) fixed
+* Issue [#61](https://github.com/rpact-com/rpact/issues/61) fixed
+* Issue [#68](https://github.com/rpact-com/rpact/issues/68) fixed
 * Flexibility of function `getPiecewiseSurvivalTime()` improved
 * Test coverage improved
 * Minor improvements

R/class_summary.R

@@ -140,9 +140,7 @@ plot.SummaryFactory <- function(x, y, ..., showSummary = FALSE) {
 }
 
 .knitPrintQueue <- function(x, ..., sep = NA_character_, prefix = "") {
-    on.exit({
-        attr(x, "queue") <- NULL
-    })
+    on.exit(.resetPipeOperatorQueue(x))
     result <- character()
     if (inherits(x, "SummaryFactory")) {
         queue <- attr(x$object, "queue")
@@ -234,7 +232,6 @@ print.SummaryFactory <- function(x, ...,
         }
         queue[[length(queue) + 1]] <- x$object
         attr(x$object, "queue") <- queue
-
         return(.knitPrintQueue(x, sep = sep))
     }
 

R/f_core_utilities.R

@@ -1598,7 +1598,17 @@ getParameterName <- function(obj, parameterCaption) {
 }
 
 .resetPipeOperatorQueue <- function(x) {
-    attr(x, "queue") <- NULL
+    for (attrName in c("queue", "printObject", "printObjectSeparator")) {
+        tryCatch({
+            if (inherits(x, "SummaryFactory")) {
+                attr(x$object, attrName) <- NULL
+            } else {
+                attr(x, attrName) <- NULL
+            }
+        }, error = function(e) {
+            message("Failed to reset pipe operator queue attribute ", sQuote(attrName), ": ", e$message)
+        })
+    }
     return(x)
 }