lookup icon indicating copy to clipboard operation
lookup copied to clipboard

No lookup_browse/source_url for lookup::* functions

Open jdanbrown opened this issue 6 years ago • 1 comments

This error might already be what you meant in #20, but I thought I'd flag in case it's not:

> lookup(lookup_browse)
lookup::lookup_browse [closure]
function (x = the$last_lookup)
{
    stopifnot(inherits(x, "lookup"))
    browseURL(tail(source_url(x), n = 1))
}
<environment: namespace:lookup>

> lookup_browse(lookup(lookup_browse))
Error in browseURL(tail(source_url(x), n = 1)) :
  'url' must be a non-empty character string

> source_url(lookup(lookup_browse))
character(0)

(Thanks for the great package!)

jdanbrown avatar May 03 '18 19:05 jdanbrown

Yeah I think you are right, but it is good to capture the error message as well in an issue, thanks!

jimhester avatar May 03 '18 19:05 jimhester