Bug fixes to 0.9.0

[WoodsBagotAndreMarquesLee]

Hey, I just fixed up a couple of the little issues I had when I tried to use your gem out-of-the-box. For what it's worth, it's entirely possible that I only encountered the nil-checking bug because one of my co-workers was messing around with our LDAP server while I was composing and testing my search query, it might not be a problem when working with an operational server.

Implemented a couple of bug fixes:

• Connection#search now does more graceful nil-checking in its ensure clause, reducing cryptic crash messages
• Filter class methods should now be able to gracefully accept numeric literals as filter rhs values

[mtodd]

I'd be comfortable with always calling right.to_s.gsub instead of introducing conditional branching.

[mtodd]

Also, add a simple test to make sure this works! :)

[jch]

Fixed in b133b31

[mtodd]

Thanks for the bugfixes @WoodsBagotAndreMarquesLee! Really appreciate you taking the time and expending the effort to help out.

I notice that this PR is open against the release-0.9.0 branch. Since 0.9.0 is already released, changes should be targeting master which can be individually backported/cherry-picked as necessary. Do you mind addressing the comments I've included (minimal tests and style remarks) and reopening this PR against master? Thanks! And let me know how I can help.

[mtodd]

@WoodsBagotAndreMarquesLee hey there, any chance you could rebase this off of master and submit as a new PR? I can't accept this against the release-0.9.0 branch. Thanks!

[jch]

Superseded by #184