Skip to content

Commit b7d1885

Browse files
committed
fix: order and formal of callNextMethod
1 parent be480f1 commit b7d1885

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

R/qenv-class.R

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,16 +23,20 @@ setClass(
2323
contains = "environment"
2424
)
2525

26+
#' It initializes the `qenv` class
27+
#' @name qenv-class
28+
#' @keywords internal
2629
setMethod(
2730
"initialize",
2831
"qenv",
2932
function(.Object, .xData = new.env(parent = parent.env(.GlobalEnv)), ...) { # nolint: object_name.
30-
.Object <- callNextMethod(.Object, ...) # nolint: object_name.
31-
3233
checkmate::assert_environment(.xData)
3334
lockEnvironment(.xData, bindings = TRUE)
3435
.Object@.xData <- .xData # nolint: object_name.
3536

37+
# .xData needs to be unnamed as the `.environment` constructure requires 1
38+
# unnamed formal argument. See methods::findMethods("initialize")$.environment
39+
.Object <- methods::callNextMethod(.Object, .xData, ...) # nolint: object_name.
3640
.Object
3741
}
3842
)

man/qenv-class.Rd

Lines changed: 3 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)