Fix/remove data location url in responses

[CelineTrammi]

Fjern datalocationUrl fra respons slik at det ikke eksponeres for brukeren

Description

Fjernet fra de som var direkte i Response objekter og fleste som var i External mappers.

Datalocationurl er fjernet fra:

• GetAttachmentDetailsResponse
• GetAttachmentOverviewResponse
• AttachmentDetailsMapper
• AttachmentOverviewMapper
• CorrespondenceAttachmentMapper
• AttachmentOverviewExt
• InitializeCorrespondenceAttachmentExt

Den siste er kanskje mest interessant siden dette gjør at CorrespondenceAttachmentEntity ikke får med seg DataLocationUrl, men den er fremdeles DataLocationUrl for AttachmentEntity i databasen.

I tillegg til disse endringene, er det laget en klasse BaseAttachmentExt som InitializeAttachmentExt og InitializeCorrespoondenceAttachmentExt arver fra.

Related Issue(s)

• DataLocationUrl skal ikke være en verdi som brukere kan se #222

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)
• All tests run green

Documentation

• User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

[Ceredron]

LGTM!
We should update swagger in https://github.com/Altinn/altinn-studio-docs/tree/master/static/swagger with removed field.

@Andreass2 Can we remove DataLocationType from external API models too? (Can keep it in db).

[Andreass2]

LGTM! We should update swagger in https://github.com/Altinn/altinn-studio-docs/tree/master/static/swagger with removed field.

@Andreass2 Can we remove DataLocationType from external API models too? (Can keep it in db).

yeah, I think so. If you would use an externalStorage. You should use the ExternalReferences values. So I Actually think it can be removed everywhere(Except DB of course). Good catch, missed it when I created the issue.