diff --git a/R/aa_helper_functions.R b/R/aa_helper_functions.R index 7b527fdc9..0a46cc2d8 100644 --- a/R/aa_helper_functions.R +++ b/R/aa_helper_functions.R @@ -465,6 +465,11 @@ cli_to_plain <- function(msg, envir = parent.frame()) { m <- regmatches(c, regexec("^\\[(.*)\\]\\([^)]*\\)$", c, perl = TRUE))[[1L]] if (length(m) >= 2L) m[2L] else paste0("`", resolve(c), "`") }) + msg <- apply_sub(msg, "\\{\\.topic ([^}]+)\\}", function(c) { + # Handle [display text](topic) markdown link format: extract just the display text + m <- regmatches(c, regexec("^\\[(.*)\\]\\([^)]*\\)$", c, perl = TRUE))[[1L]] + if (length(m) >= 2L) m[2L] else paste0("?", resolve(c)) + }) msg <- apply_sub(msg, "\\{\\.url (\\{[^}]+\\}|[^}]+)\\}", function(c) resolve(c)) msg <- apply_sub(msg, "\\{\\.href ([^}]+)\\}", function(c) strsplit(resolve(c), " ", fixed = TRUE)[[1L]][1L]) diff --git a/R/count.R b/R/count.R index bc41cb7bc..174c6f423 100755 --- a/R/count.R +++ b/R/count.R @@ -128,7 +128,7 @@ count_resistant <- function(..., # other arguments for meet_criteria are handled by sir_calc() meet_criteria(guideline, allow_class = "character", is_in = c("EUCAST", "CLSI"), has_length = 1) if (is.null(getOption("AMR_guideline")) && missing(guideline) && message_not_thrown_before("count_resistant", "eucast_default", entire_session = TRUE)) { - message_("{.help [{.fun count_resistant}](AMR::count_resistant)} assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the {.arg guideline} argument or the {.code AMR_guideline} option to either \"CLSI\" or \"EUCAST\", see {.code ?AMR-options}.") + message_("{.help [{.fun count_resistant}](AMR::count_resistant)} assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the {.arg guideline} argument or the {.code AMR_guideline} option to either \"CLSI\" or \"EUCAST\", see {.topic AMR-options}.") message_("This message will be shown once per session.") } tryCatch( @@ -152,7 +152,7 @@ count_susceptible <- function(..., # other arguments for meet_criteria are handled by sir_calc() meet_criteria(guideline, allow_class = "character", is_in = c("EUCAST", "CLSI"), has_length = 1) if (is.null(getOption("AMR_guideline")) && missing(guideline) && message_not_thrown_before("count_susceptible", "eucast_default", entire_session = TRUE)) { - message_("{.help [{.fun count_susceptible}](AMR::count_susceptible)} assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the {.arg guideline} argument or the {.code AMR_guideline} option to either \"CLSI\" or \"EUCAST\", see {.code ?AMR-options}.") + message_("{.help [{.fun count_susceptible}](AMR::count_susceptible)} assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the {.arg guideline} argument or the {.code AMR_guideline} option to either \"CLSI\" or \"EUCAST\", see {.topic AMR-options}.") message_("This message will be shown once per session.") } tryCatch( diff --git a/R/interpretive_rules.R b/R/interpretive_rules.R index 7e498b40b..dd0359888 100755 --- a/R/interpretive_rules.R +++ b/R/interpretive_rules.R @@ -1097,7 +1097,7 @@ eucast_rules <- function(x, rules = getOption("AMR_interpretive_rules", default = c("breakpoints", "expected_phenotypes")), ...) { if (!is.null(getOption("AMR_eucastrules", default = NULL))) { - warning_("The global option {.code AMR_eucastrules} that you have set is now invalid was ignored - set {.code AMR_interpretive_rules} instead. See {.code ?AMR-options}.") + warning_("The global option {.code AMR_eucastrules} that you have set is now invalid was ignored - set {.code AMR_interpretive_rules} instead. See {.topic AMR-options}.") } interpretive_rules(x = x, col_mo = col_mo, info = info, rules = rules, guideline = "EUCAST", ...) } diff --git a/R/proportion.R b/R/proportion.R index fb151b1b6..35f0b210d 100644 --- a/R/proportion.R +++ b/R/proportion.R @@ -238,7 +238,7 @@ resistance <- function(..., # other arguments for meet_criteria are handled by sir_calc() meet_criteria(guideline, allow_class = "character", is_in = c("EUCAST", "CLSI"), has_length = 1) if (is.null(getOption("AMR_guideline")) && missing(guideline) && message_not_thrown_before("resistance", "eucast_default", entire_session = TRUE)) { - message_("`resistance()` assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the `guideline` argument or the `AMR_guideline` option to either \"CLSI\" or \"EUCAST\", see `?AMR-options`.") + message_("{.help [{.fun resistance}](AMR::resistance)} assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the {.arg guideline} argument or the {.code AMR_guideline} option to either \"CLSI\" or \"EUCAST\", see {.topic AMR-options}.") message_("This message will be shown once per session.") } tryCatch( @@ -266,7 +266,7 @@ susceptibility <- function(..., # other arguments for meet_criteria are handled by sir_calc() meet_criteria(guideline, allow_class = "character", is_in = c("EUCAST", "CLSI"), has_length = 1) if (is.null(getOption("AMR_guideline")) && missing(guideline) && message_not_thrown_before("susceptibility", "eucast_default", entire_session = TRUE)) { - message_("`susceptibility()` assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the `guideline` argument or the `AMR_guideline` option to either \"CLSI\" or \"EUCAST\", see `?AMR-options`.") + message_("{.help [{.fun susceptibility}](AMR::susceptibility)} assumes the EUCAST guideline and thus considers the 'I' category susceptible. Set the {.arg guideline} argument or the {.code AMR_guideline} option to either \"CLSI\" or \"EUCAST\", see {.topic AMR-options}.") message_("This message will be shown once per session.") } tryCatch( diff --git a/R/zz_deprecated.R b/R/zz_deprecated.R index 82fc127cb..79701a32f 100755 --- a/R/zz_deprecated.R +++ b/R/zz_deprecated.R @@ -124,7 +124,7 @@ deprecation_warning <- function(old = NULL, new = NULL, fn = NULL, extra_msg = N ". The old name will be removed in future version, so please update your code.", ifelse(type == "argument", ". While the old argument still works, it will be removed in a future version, so please update your code.", - " and will be removed in a future version, see `?AMR-deprecated`." + " and will be removed in a future version, see {.topic AMR-deprecated}." ) ), ifelse(!is.null(extra_msg),