Compare commits

...

2 Commits

Author SHA1 Message Date
Joe Cheng
9e5895da73 Add NEWS item for renderUI bookmarking fix 2018-08-06 14:50:35 -07:00
Joe Cheng
624fcfba45 hasCurrentRestoreContext returns FALSE from server side
Fixes #2138.
2018-07-30 10:03:36 -07:00
2 changed files with 9 additions and 1 deletions

View File

@@ -17,6 +17,8 @@ shiny 1.1.0.9000
* Added namespace support when freezing reactiveValue keys. [#2080](https://github.com/rstudio/shiny/pull/2080)
* Fixed [#2138](https://github.com/rstudio/shiny/issues/2138): Inputs that are part of a `renderUI` were no longer restoring correctly from bookmarked state. [#2139](https://github.com/rstudio/shiny/pull/2139)
### Documentation Updates
* Addressed [#1864](https://github.com/rstudio/shiny/issues/1864) by changing `optgroup` documentation to use `list` instead of `c`. ([#2084](https://github.com/rstudio/shiny/pull/2084))

View File

@@ -448,7 +448,13 @@ withRestoreContext <- function(ctx, expr) {
# Is there a current restore context?
hasCurrentRestoreContext <- function() {
restoreCtxStack$size() > 0
if (restoreCtxStack$size() > 0)
return(TRUE)
domain <- getDefaultReactiveDomain()
if (!is.null(domain) && !is.null(domain$restoreContext))
return(TRUE)
return(FALSE)
}
# Call to access the current restore context. First look on the restore