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

Possible regression in support of setMethodS3() and setConstructorS3() of R.methodsS3 and R.oo #631

Closed
Lucaweihs opened this issue May 22, 2017 · 1 comment
Labels
bug an unexpected problem or unintended behavior wip work in progress

Comments

@Lucaweihs
Copy link
Contributor

Lucaweihs commented May 22, 2017

#525 added support for R.methodsS3::setMethodS3(name, class, definition) and R.oo::setConstructorS3(name, definition). With the newest version of Roxygen, attempting to run roxygen using these methods results in an

Error in methods::is(x, "refObjectGenerator") : object 'value' not found

error for me. I've created a minimal package demonstrating the error here. The error goes away when prepending the package namespace to setConstructorS3 and setMethodS3 (that is, always using the fully namespaced R.oo::setConstructorS3 and R.methodsS3::setMethodS3) but, when doing so, the s3 methods are not appropriately registered by the namespace (they are simply exported).

@Lucaweihs
Copy link
Contributor Author

@HenrikBengtsson

@hadley hadley added bug an unexpected problem or unintended behavior wip work in progress labels Aug 16, 2017
@hadley hadley closed this as completed Aug 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug an unexpected problem or unintended behavior wip work in progress
Projects
None yet
Development

No branches or pull requests

2 participants