Explicit scoping for param_meta in define.param

mbjoseph commented 3 years ago

Hey @mikejohnson51! I'm incorporating climateR into a downstream package, but running into scoping issues.

When my package calls getMACA, the following error is raised:

Error in eval(parse(text = paste0("param_meta$", service))) : 
  object 'param_meta' not found

The traceback indicates that this is raised when climateR::define.param is called by climateR::getMACA:

Proposed solution

Modifying this line to use climateR::param_meta instead of param_meta solves the issue: https://github.com/mikejohnson51/climateR/blob/master/R/utility_define_param.R#L12

I noticed that you've explicitly specified the scope like this elsewhere where eval() is used, e.g.,

and

mikejohnson51 commented 3 years ago

I think this should be taken care of, thanks again @mbjoseph!

mbjoseph commented 3 years ago

cool - thanks @mikejohnson51! :+1:

mikejohnson51 / climateR