adeutscher / DirectoryTools

0 stars 0 forks source link

Tidy up getProperty #22

Closed adeutscher closed 10 years ago

adeutscher commented 10 years ago

With PropertyNotFoundException being added as part of #15, getProperty()'s exitOnFail argument doesn't make a whole lot of sense.

Ideas:

adeutscher commented 10 years ago

Revamped getProperties.

Forgot to document one other detail in ticket: I also corrected an error that I had made with getProperties()' use of PropertyNotFoundException in 7e3a263 while implementing #15. It was being given the originalException argument of an ExceptionWrapper-based exception. Whoops.

adeutscher commented 10 years ago

Reference in 24033ce was a typo. #21 was the correct ticket to be reference. The comment was corrected using git commit --amend and git push --force.