Warn when data contains an object named "all"

[gogonzo]

Because "all" is a reserved word in module(), otherwise this would be ambiguous.

[donyunardi]

We try to prevent the confusion when module(datanames = "all")

Acceptance Criteria

• teal should give a warning if there's an object in teal_data with name "all"
• the app should still continue (currently it failed when teal_data has object name "all")

[averissimo]

Teal is crashing with all. It seems related to lockfile generation

Running options("teal.lockfile.mode" = "disabled") before the code below allows the example module to work

data <- teal_data() |>
  within({
    all <- iris
  })

# data <- teal_data_module(
#   ui = function(id) {
#     ns <- NS(id)
#     actionButton(ns("submit"), label = "Load data")
#   },
#   server = function(id) {
#     moduleServer(id, function(input, output, session) {
#       eventReactive(input$submit, {
#         data <- within(
#           teal_data(),
#           {
#             all <- iris
#           }
#         )
#         data
#       })
#     })
#   }
# )


app <- init(
  data = data,
  modules = modules(
    example_module("Module with transformations 1", datanames = "all")
  ),
  title = "yada"
)

shinyApp(app$ui, app$server)

[gogonzo]

Works for me ;]

[averissimo]

Works after updating renv to v1.0.11 👍

[averissimo]

.raw_data should also be considered a reserved word as it is being overwritten in the code.

pkgload::load_all("teal")
data <- teal.data::teal_data() |>
  within({
    all <- iris
    air <- datasets::airquality
    .raw_data <- mtcars
  })

app <- teal::init(
  data = data,
  modules = teal::example_module("Module 1", datanames = "all")
)

shiny::runApp(app)