Stop accepting nested attributes for attachments

Now that all attachment editing is done via the admin attachments
controller we no longer need to accept nested attributes. This removes
the nested attributes and all tests which was checking that behaviour.
Remove unused partial