OSRA-300 Split Orphan `father_name`: factory

OSRA-306 Update orphans factory & seeds

 - remove `father_name`
 - add `father_given_name { Faker::Name.first_name }` &
   `family_name { Faker::Name.last_name }`
 - change from `Faker::Name.name` to `Faker::Name.first_name` for `name` &
   `mother_name`
 - no change necessary to dev seeds
OSRA-300 split orphan `father_name`: full_name

OSRA-304 update implementation of Orphan#full_name

 - change the `#full_name` method to use `family_name` instead of `father_name`
 - change orphan.feature; to be changed further when views updated
Merge pull request #229 from NikitaAvvakumov/osra_306_update_orphan_factory_for_father_name

OSRA-300 Split Orphan `father_name`: factory
Merge pull request #230 from NikitaAvvakumov/osra-304-change-orphan-full-name

OSRA-300 split orphan `father_name`: full_name method
OSRA-300 - refine Orphan#full_name

OSRA-304 - change implementation of Orphan#full_name

 - based on clarification from client, change the method to compose
   an orphan's `full_name` out of `name`, `father_given_name` and `family_name`
 - revert orphans.feature to original
Merge pull request #231 from NikitaAvvakumov/osra-304-pt2-refine-orphan-full-name

OSRA-300 - refine Orphan#full_name
Refactor OrphanImporter#extract_orphans

+ Remove unnecessary return
+ alter control flow - don't check for duplicates if no spreadsheet 
+ remove unused variable assignment
Swap return responsibilities in OrphanImporter

Returning false seems like the concern of the exception logging method, not of the add import errors method.
OrphanImporter#import_orphan refactor

+ Use #inject 
+ Use existing add_import_errors method where possible
+ Remove unnecessary variable assignment
+ Improve the check_for_duplicates loop
OrphanImporter - prefer string interp
OrphanImporter - concat strings with <<, not +=

This can be up to 25% faster, as += creates intermediary strings.
Fix typo in OrphanImporter spec
OrphanImporter: decrease public API size
update views
OSRA-300 Split `father_name`: orphan uniqueness

OSRA-308 Update orphan uniqueness validation

 - scope of uniqueness validation updated to include `father_given_name`,
   `mother_name` & `family_name`
Merge pull request #234 from theB264/osra305_update_orphan_views

OSRA-305 #Done
Merge pull request #235 from NikitaAvvakumov/osra-308-orphan-uniqueness-validation

OSRA-300 Split `father_name`: orphan uniqueness
prevent rake sending status to std out when being tested

this implementation used in preference to wrapping in a quietly block
becauase the quietly block silences std err as well.
Merge pull request #236 from PurityControl/rake-tidy

prevent rake sending status to std out when being tested
spreadsheet change splitting father name into given_name and family_name

- pending orphan migration converting father name to given name and family name
- amending tests to suit
- amending columns in config/settings.yml
- fix arabic in province after discussion with client
- use of new excel format provided by client
- client wants to change father_martyr settings to Y / N values, have added
  these as allowed values but have kept the old ones too. These could be
  removed if this was preferred.