Skip to content
This repository was archived by the owner on Dec 31, 2021. It is now read-only.

Use of blank? instead of nil? to check hash value #1

Merged
merged 1 commit into from
May 21, 2012

Conversation

sciurus
Copy link
Contributor

@sciurus sciurus commented May 21, 2012

collectd_plugin.rb checks if values in the params hash are blank. I think it should check if they are nil instead.

This resolves the error "NoMethodError: undefined method `blank?' for nil:NilClass" for me with ruby 1.8.7 and chef 0.10.10.

coderanger added a commit that referenced this pull request May 21, 2012
 Use of blank? instead of nil? to check hash value
@coderanger coderanger merged commit c18eb0c into coderanger:master May 21, 2012
@coderanger
Copy link
Owner

Thanks for the patch, looks like at some point Chef stopped bundling ActiveSupport's Object#blank? hack.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants