HomePhorge

Don't mask NoMethodErrors
da7cd8e304e1Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Don't mask NoMethodErrors

This rescue was masking a bug in a provider (involving calling [] on
nil.NilClass) with a misleading error message. The new code asks
permission rather than begging forgiveness, but is more specific to the
problem being detected.

Details

Provenance
Dustin J. Mitchell <dustin@mozilla.com>Authored on
Andrew Parker <andy@puppetlabs.com>Committed on Aug 8 2012, 1:20 AM
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPUb840ae9a1c82: Merge remote-tracking branch 'upstream/2.7rc' into 2.7.x
Branches
Unknown
Tags
Unknown

Event Timeline

Andrew Parker <andy@puppetlabs.com> committed rPUda7cd8e304e1: Don't mask NoMethodErrors (authored by Dustin J. Mitchell <dustin@mozilla.com>).Aug 8 2012, 1:20 AM