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

has_content #91

Closed
hadley opened this issue Apr 1, 2014 · 2 comments
Closed

has_content #91

hadley opened this issue Apr 1, 2014 · 2 comments

Comments

@hadley
Copy link
Member

@hadley hadley commented Apr 1, 2014

Boolean if there's any content associated with request. as = "auto" should throw an error.

@hadley hadley closed this in bc90021 Apr 21, 2014
@jennybc
Copy link
Member

@jennybc jennybc commented Aug 20, 2015

Here's current definition of has_content():

has_content <- function(x) {
  length(x) == 0
}

I'm trying to use it over in my PR re: devtools and ... any chance you meant this to involve content(x) instead of x and/or to test if length is greater than zero? I can't find any usage of has_content() any where on github, so I can't figure out if I'm just missing something ... but this seems really weird.

@hadley
Copy link
Member Author

@hadley hadley commented Aug 21, 2015

Hmmm seems like it should be length(x$content) > 0 :/

@hadley hadley reopened this Aug 21, 2015
@hadley hadley closed this in b683afb Dec 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants