puppetlabs/puppet — pull request #2973 by vrthra (2014-08-14 21:51:23+00:00)
Title: (PUP-2817) Fix zone properties so that nil is not passed through

Before this patch, when `insync?` was called on the zone properties ip,
dataset and inherit with value nil (happens the first time when the properties
are not present), `true` was returned, which resulted in them not being set
until the values were actually queried, which happened after the installation
of zone. Unfortunately, the inherit property can only be set before
installation.

This patch removes the spurious check for `nil`, which always returned `true`,
and instead handles it similar to when `:absent` is used.