Not hugely important, but if you made this state_attrs this part isn't needed as new_resource.state will load it.

Though it seems like older versions of chef don't have state_attrs, so I guess it is half a dozen or the other.