update strata var to group_var

R/tm_g_lineplot.R

@@ -5,6 +5,7 @@
 #' @inheritParams tern::g_lineplot
 #' @inheritParams tern::control_lineplot_vars
 #' @inheritParams template_arguments
+#' @param strata `r lifecycle::badge("deprecated")` Please use the `group_var` argument instead.
 #' @param group_var (`string` or `NA`)\cr group variable name.
 #' @param param (`character`)\cr parameter to filter the data by.
 #' @param incl_screen (`logical`)\cr whether the screening visit should be included.
@@ -22,6 +23,7 @@
 #'
 #' @keywords internal
 template_g_lineplot <- function(dataname = "ANL",
+                                strata = lifecycle::deprecated(),
                                 group_var = "ARM",
                                 x = "AVISIT",
                                 y = "AVAL",
@@ -40,6 +42,15 @@ template_g_lineplot <- function(dataname = "ANL",
                                 title = "Line Plot",
                                 y_lab = "",
                                 ggplot2_args = teal.widgets::ggplot2_args()) {
+  if (lifecycle::is_present(strata)) {
+    warning(
+      "The `strata` argument of `tm_g_lineplot()` is deprecated as of teal.modules.clinical 0.9.1. ",
+      "Please use the `group_var` argument instead.",
+      call. = FALSE
+    )
+    group_var <- strata
+  }
+
   checkmate::assert_string(dataname)
   checkmate::assert_string(group_var)
   checkmate::assert_string(x)
@@ -255,11 +266,8 @@ template_g_lineplot <- function(dataname = "ANL",
 #' @export
 tm_g_lineplot <- function(label,
                           dataname,
-                          parentname = ifelse(
-                            inherits(group_var, "data_extract_spec"),
-                            teal.transform::datanames_input(group_var),
-                            "ADSL"
-                          ),
+                          parentname = NULL,
+                          strata = lifecycle::deprecated(),
                           group_var = teal.transform::choices_selected(
                             teal.transform::variable_choices(parentname, c("ARM", "ARMCD", "ACTARMCD")), "ARM"
                           ),
@@ -294,6 +302,21 @@ tm_g_lineplot <- function(label,
                           pre_output = NULL,
                           post_output = NULL,
                           ggplot2_args = teal.widgets::ggplot2_args()) {
+  if (lifecycle::is_present(strata)) {
+    warning(
+      "The `strata` argument of `tm_g_lineplot()` is deprecated as of teal.modules.clinical 0.9.1. ",
+      "Please use the `group_var` argument instead.",
+      call. = FALSE
+    )
+    group_var <- strata
+  }
+
+  # Now handle 'parentname' calculation based on 'group_var'
+  parentname <- ifelse(
+    inherits(group_var, "data_extract_spec"),
+    teal.transform::datanames_input(group_var),
+    "ADSL"
+  )
   message("Initializing tm_g_lineplot")
   checkmate::assert_string(label)
   checkmate::assert_string(dataname)