Update includedeaccessioned to GetFileById

[ChengShi-1]

What this PR does / why we need it:

Related to IQSS/dataverse-client-javascript#322

In order to correctly get the file which is deaccessioned, we need include the includeDeaccessioned, so even though the dataset is deaccessioned, by applying includeDeaccessioned = true to the GETFile endpoint, the owner could get right response to access the deaccessioned file.

• add includeDeaccessioned to getById and getCitationById, the parameter is affected by EditPermission so I change the order of the promises, and get permission first.
• File version tab, deaccessioned version links only access to who canEditDataset

Which issue(s) this PR closes:

• Closes Update includeDeaccessioned to GetFile() #764

Special notes for your reviewer:

Suggestions on how to test this:

It's better to wait #741 to be merged to this PR and then test

• As a superuser,

• create dataset with a file and published dataset

• deaccessioned dataset

• click to the file table's file, in dataset page, so navigate to file page

• see if the file page of deaccessioned version is ok, also should be shown with deaccessioned format
  

• Log out (As a guest user

• go the deaccessioned dataset(superuser created), the file deaccessioned version should not be shown(404 page expected

• If there are other versions not deaccessioned, guest users should be accessed to other released/ non-deaccessioned versions, but the file > version tab should not attach links to deaccessioned version

[coveralls]

coverage: 97.527% (-0.5%) from 98.062%
when pulling 32916dd on 764-update-includedeaccessioned-to-getfile
into e7769b4 on develop.

[g-saracca]

LGTM

[ofahimIQSS]

Hey! this one has branch conflicts - can i get an update please :)

The merge-base changed after approval.

[g-saracca]

@ChengShi-1 I think you didn't solve merge conflicts correctly, I see lot of unrelated changes and things removed we shouldn't remove.

[ChengShi-1]

@g-saracca Yes, I need revert and merge again. solving

[ChengShi-1]

@g-saracca Reset the branch and merge again, should be ok now. Thanks!

(I falsely merged branch with discarding all changes from dev....sorry for that

[g-saracca]

@g-saracca Reset the branch and merge again, should be ok now. Thanks!

(I falsely merged branch with discarding all changes from dev....sorry for that

That's ok, no need to be sorry 👍🏼 , waiting for tests to pass and will approve 🙂

[g-saracca]

@ofahimIQSS all good now 👍🏼

[ofahimIQSS]

tested in local - no issues found. Merging!