hiera_hash lookup introduced in d0abd007 fails if not supplied hash

[mattyindustries]

I've just upgraded my CUPS module to master to bring in Pull Request #52 to resolve #50 as I was having this problem.

Because of this I have also included commit d0abd00 which introduces hiera support for the variable
cups::printers.

If you don't set a hiera variable of cups::printers now an error is logged

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Could not find data item cups::printers in any Hiera data file and no default supplied at /app/puppet/sapv2_cups/puppet/modules/cups/manifests/init.pp:21 on node toyecq01.tmca.com.au
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

I have my own hiera variable that I use to store the cups printer definitions and I could swap this over to use cups::printers but this is probably not the behaviour you want especially if someone isn't using hiera.

Adding a blank default to the hiera_hash might resolve this issue but haven't tested it yet. I'll hopefully get some time to play around with a couple of options and submit a pull request in the next few days.

[mattyindustries]

I've just raised Pull Request #54 to cover this issue

[mosen]

Thanks a bunch. Will review and merge in a few hours.