Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Crash de l'application avec selectInput_dsfr() #154

Open
ifhan opened this issue Jan 12, 2024 · 2 comments
Open

[Bug] Crash de l'application avec selectInput_dsfr() #154

ifhan opened this issue Jan 12, 2024 · 2 comments

Comments

@ifhan
Copy link

ifhan commented Jan 12, 2024

En cas d'utilisation de selectInput_dsfr() sans updateSelectInput_dsfr() correspondant l'application plante au démarrage avec les messages suivants :

  • dans l'application :
An error has occurred! 
In index: 1.
  • dans la console :
Warning: Error in purrr::map: ℹ In index: 1.
Caused by error:
! .x is not a character vector

Le code suivant génère l'erreur :
ui :

          selectInput_dsfr(
            inputId = ns("select_annee"),
            label = "Ann\u00e9e :",
            choices = c(
              year(today()):2016),
            selected = year(today())
          )

Le code suivant ne cause pas d'erreur :
ui :

          selectInput_dsfr(
            inputId = ns("select_annee"),
            label = "Ann\u00e9e :",
            choices = NULL
          )

server :

      updateSelectInput_dsfr(
        session = session,
        inputId = "select_annee",
        choices = c(year(today()):2016)
      )

Cela oblige à ajouter un updateSelectInput_dsfr() alors que l'on devrait pouvoir s'en passer dans ce cas de figure.

@jengelaere
Copy link
Contributor

@ifhan je crois comprendre que l'on force les choix à être de type character.
Le temps que l'on investigue pourquoi, tu peux utiliser

selectInput_dsfr(
              inputId = ns("select_annee"),
              label = "Ann\u00e9e :",
              choices = as.character(2016:year(today())),
              selected = as.character(year(today()))
            )

qui ne devrait pas renvoyer d'erreur

@ifhan
Copy link
Author

ifhan commented Jan 12, 2024

Merci @jengelaere j'ai testé le fix : ça fonctionne ! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants