Skip to content

Commit

Permalink
got rid of warning that popped up when renderFunc took no arguments; …
Browse files Browse the repository at this point in the history
…there really isn't a good reason to require this (not at this point at least)
  • Loading branch information
bborgesr committed Apr 15, 2016
1 parent 0bdc8f0 commit ad1e52b
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions R/shinywrappers.R
Original file line number Diff line number Diff line change
Expand Up @@ -42,13 +42,8 @@ markRenderFunction <- function(uiFunc, renderFunc, outputArgs = list()) {
# stop warning from happening again for the same object
hasExecuted$set(TRUE)
}
if (is.null(formals(origRenderFunc))) {
warning("The render function should take at least two arguments (`name` ",
"and `shinysession`.")
origRenderFunc()
} else {
origRenderFunc(...)
}
if (is.null(formals(origRenderFunc))) origRenderFunc()
else origRenderFunc(...)
}

structure(renderFunc,
Expand Down

0 comments on commit ad1e52b

Please sign in to comment.