escape database name

[agerlic]

escape database name with backquotes to handle database name with dashes eg: "test-1"

[branan]

does this issue also potentially affect the database_grant type?

[agerlic]

I guess only db parameter in the destroy method from database_grant type must also be escaped.

[branan]

I'm pretty happy with the idea of merging this. It's a small change and fixes a real problem with the module.

If you're comfortable with rspec-puppet, it would be really fantastic if you could write some tests for the things you've changed. I realize there are currently no tests for the database provider, and I don't expect you to write anything with full test coverage - basically just verify that it tries to call the correct command lines. Because there aren't yet any tests for that provider this is totally optional.

Let me know if you're interested in doing that. Even if you choose not to take that on, this will almost certainly get merged as it stands.

[agerlic]

Sorry for the delay.

The absence of database provider is a real problem, If I can free some time I will try to contribute.

Thanks for the merge.