I actually [made the update](https://github.com/rtkwlf/cookbook-simple-iptables/commit/247073b23b26318f4b2ea2fcb5ba07d41bc0a385) to use persistent attributes for rules so the Chef notification mechanism would work correctly, so I don't really agree with this approach. I think a better approach would to continue to use normal attributes in the providers but provide a `:reset` action that basically zero's down the rules.

That being said, this code basically brute forces what you could do by using default attributes. If you are going to take this approach (but I really hope you reconsider) I'd suggest getting rid of this code block and just use default attributes throughout the providers. I'd also strongly recommend removing the code in the providers that triggers Chef notifications (`new_resource.updated_by_last_action(true)`).

I'd be willing to help with a `:reset` LWRP action if you'd consider taking that approach.