copy_attachments does some pretty terrible things

First of all, `unique_slugify` has the potential to do `n` queries where `n` is the number of attachments in the DB. Second of all, in `copy` stop doing `copy.attached_by = self.attached_by` do `copy.attached_by_id = self.attached_by_id` instead.