Audits

## TODO
- [X] Association between Pages / Templates
- [x]   Publish Pages  on Template Publishes
Yeeeeahh, and that construction can make sense if things are structured properly, though I admit I tend to err away from it.

My thought process here:

We've got a similar sort of statement up above, with unpublishing the published revision. Modifying that to be more similar in form, we'd have this (abbreviated) chunk of code:

```ruby
unpublish_published_revision if published_revision.present?
republish_associated_pages(options) if pages.present?
```

The question is then if it's more appropriate to put those conditions inside the helper methods, since those helper methods are the ones responsible for handling all the wiring. If we leave the conditions where they are, then we make assumptions about what's happening internally in methods elsewhere.

Speaking only about the `pages.present?` bit, removing it entirely saves us a sql query, which is why I mentioned this in the first place. :)

My suggestion is to change it, as described.