Reorder requires in wiring for clarity
Require ManualDocumentValidator in wiring
Require CmaCaseValidator in wiring
Merge pull request #180 from alphagov/bug/require-validator-in-wiring

Require MaunualDocumentValidator in wiring
Rename AttachmentsController to CmaCaseAttachmentsController
Rename AaibReportsAttachmentsController to be singular

This is the correct rails best practices
Rename ManualDocumentsAttachments to be singular

This is the rails best practices for controller naming conventions
Create and use AbstractAttachmentsController

This commit provides setup for making use of an abstract attachment controller which will provide common functionality to CmaCase and AaibReport attachments controllers
CmaCaseAttachment services to their own registry

In order to clean up the code and require different sets of services per controller, move the CmaCase related Attachment sevices to their own registry. We can now use a services method in the controller to decide which service registry to use
Correct spelling of registry
Move services method into attachment controller
Refactor servies to implement the same api
Abstract common attachment controller code

The only thing that is now not common in the controller code is the redirect path. Once we move this into a 'redirect_path' helper we can use the same code across the board for all attachments