Skip to content

Commit d3da730

Browse files
rootMykhaylo Boychuk
root
authored and
Mykhaylo Boychuk
committed
[DSC-2782] Rename Sherpa Romeo to new name Open Policy Finder
1 parent e4e961e commit d3da730

62 files changed

Lines changed: 388 additions & 266 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

src/app/core/submission/models/sherpa-policies-details.model.ts renamed to src/app/core/submission/models/opf-policies-details.model.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,25 @@
11
/**
22
* An interface to represent an access condition.
33
*/
4-
export class SherpaPoliciesDetailsObject {
4+
export class JiscOpfPoliciesDetailsObject {
55

66
/**
7-
* The sherpa policies error
7+
* The Jisc Open Policy Finder policies error
88
*/
99
error: boolean;
1010

1111
/**
12-
* The sherpa policies journal details
12+
* The Jisc Open Policy Finder policies journal details
1313
*/
1414
journals: Journal[];
1515

1616
/**
17-
* The sherpa policies message
17+
* The Jisc Open Policy Finder policies message
1818
*/
1919
message: string;
2020

2121
/**
22-
* The sherpa policies metadata
22+
* The Jisc Open Policy Finder policies metadata
2323
*/
2424
metadata: Metadata;
2525

‎src/app/core/submission/models/workspaceitem-section-opf-policies.model.ts‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
import { JiscOpfPoliciesDetailsObject } from './opf-policies-details.model';
2+
3+
/**
4+
* An interface to represent the submission's item accesses condition.
5+
*/
6+
export interface WorkspaceitemSectionJiscOpfPoliciesObject {
7+
8+
/**
9+
* The access condition id
10+
*/
11+
id: string;
12+
13+
/**
14+
* The Jisc Open Policy Finder policies retrievalTime
15+
*/
16+
retrievalTime: string;
17+
18+
/**
19+
* The Jisc Open Policy Finder policies details
20+
*/
21+
opfResponse: JiscOpfPoliciesDetailsObject;
22+
}

‎src/app/core/submission/models/workspaceitem-section-sherpa-policies.model.ts‎

Lines changed: 0 additions & 22 deletions
This file was deleted.

‎src/app/core/submission/models/workspaceitem-sections.model.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { WorkspaceitemSectionDuplicatesObject } from './workspaceitem-section-du
66
import { WorkspaceitemSectionFormObject } from './workspaceitem-section-form.model';
77
import { WorkspaceitemSectionIdentifiersObject } from './workspaceitem-section-identifiers.model';
88
import { WorkspaceitemSectionLicenseObject } from './workspaceitem-section-license.model';
9-
import { WorkspaceitemSectionSherpaPoliciesObject } from './workspaceitem-section-sherpa-policies.model';
9+
import { WorkspaceitemSectionJiscOpfPoliciesObject } from './workspaceitem-section-opf-policies.model';
1010
import { WorkspaceitemSectionUploadObject } from './workspaceitem-section-upload.model';
1111

1212
/**
@@ -26,7 +26,7 @@ export type WorkspaceitemSectionDataType
2626
| WorkspaceitemSectionLicenseObject
2727
| WorkspaceitemSectionCcLicenseObject
2828
| WorkspaceitemSectionAccessesObject
29-
| WorkspaceitemSectionSherpaPoliciesObject
29+
| WorkspaceitemSectionJiscOpfPoliciesObject
3030
| WorkspaceitemSectionIdentifiersObject
3131
| WorkspaceitemSectionDuplicatesObject
3232
| WorkspaceitemSectionDetectDuplicateObject

‎src/app/shared/form/builder/ds-dynamic-form-ui/relation-lookup-modal/dynamic-lookup-relation-modal.component.spec.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,8 @@ describe('DsDynamicLookupRelationModalComponent', () => {
7979
hierarchical: false,
8080
}),
8181
Object.assign(new ExternalSource(), {
82-
id: 'sherpaPublisher',
83-
name: 'sherpaPublisher',
82+
id: 'opfPublisher',
83+
name: 'opfPublisher',
8484
hierarchical: false,
8585
}),
8686
];
@@ -106,7 +106,7 @@ describe('DsDynamicLookupRelationModalComponent', () => {
106106
relationshipType: 'isAuthorOfPublication',
107107
nameVariants: true,
108108
searchConfiguration: 'personConfig',
109-
externalSources: ['orcidV2', 'sherpaPublisher'],
109+
externalSources: ['orcidV2', 'opfPublisher'],
110110
});
111111
nameVariant = 'Doe, J.';
112112
metadataField = 'dc.contributor.author';

src/app/shared/mocks/section-sherpa-policies.service.mock.ts renamed to src/app/shared/mocks/section-opf-policies.service.mock.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
import { WorkspaceitemSectionSherpaPoliciesObject } from '../../core/submission/models/workspaceitem-section-sherpa-policies.model';
1+
import { WorkspaceitemSectionJiscOpfPoliciesObject } from '../../core/submission/models/workspaceitem-section-opf-policies.model';
22

3-
export const SherpaDataResponse = {
4-
'id': 'sherpaPolicies',
3+
export const OpfDataResponse = {
4+
'id': 'opfPolicies',
55
'retrievalTime': '2022-04-20T09:44:39.870+00:00',
6-
'sherpaResponse':
6+
'opfResponse':
77
{
88
'error': false,
99
'message': null,
@@ -96,4 +96,4 @@ export const SherpaDataResponse = {
9696
'inDOAJ': false,
9797
}],
9898
},
99-
} as WorkspaceitemSectionSherpaPoliciesObject;
99+
} as WorkspaceitemSectionJiscOpfPoliciesObject;

‎src/app/shared/testing/section-opf-policies.service.mock.ts‎

Lines changed: 99 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,99 @@
1+
import { WorkspaceitemSectionJiscOpfPoliciesObject } from '../../core/submission/models/workspaceitem-section-opf-policies.model';
2+
3+
export const OpfDataResponse = {
4+
'id': 'opfPolicies',
5+
'retrievalTime': '2022-04-20T09:44:39.870+00:00',
6+
'opfResponse':
7+
{
8+
'error': false,
9+
'message': null,
10+
'metadata': {
11+
'id': 23803,
12+
'uri': 'http://v2.sherpa.ac.uk/id/publication/23803',
13+
'dateCreated': '2012-11-20 14:51:52',
14+
'dateModified': '2020-03-06 11:25:54',
15+
'inDOAJ': false,
16+
'publiclyVisible': true,
17+
},
18+
'journals': [{
19+
'titles': ['The Lancet', 'Lancet'],
20+
'url': 'http://www.thelancet.com/journals/lancet/issue/current',
21+
'issns': ['0140-6736', '1474-547X'],
22+
'romeoPub': 'Elsevier: The Lancet',
23+
'zetoPub': 'Elsevier: The Lancet',
24+
'publisher': {
25+
'name': 'Elsevier',
26+
'relationshipType': null,
27+
'country': null,
28+
'uri': 'http://www.elsevier.com/',
29+
'identifier': null,
30+
'publicationCount': 0,
31+
'paidAccessDescription': 'Open access',
32+
'paidAccessUrl': 'https://www.elsevier.com/about/open-science/open-access',
33+
},
34+
'publishers': [{
35+
'name': 'Elsevier',
36+
'relationshipType': null,
37+
'country': null,
38+
'uri': 'http://www.elsevier.com/',
39+
'identifier': null,
40+
'publicationCount': 0,
41+
'paidAccessDescription': 'Open access',
42+
'paidAccessUrl': 'https://www.elsevier.com/about/open-science/open-access',
43+
}],
44+
'policies': [{
45+
'id': 0,
46+
'openAccessPermitted': false,
47+
'uri': null,
48+
'internalMoniker': 'Lancet',
49+
'permittedVersions': [{
50+
'articleVersion': 'submitted',
51+
'option': 1,
52+
'conditions': ['Upon publication publisher copyright and source must be acknowledged', 'Upon publication must link to publisher version'],
53+
'prerequisites': [],
54+
'locations': ['Author\'s Homepage', 'Preprint Repository'],
55+
'licenses': [],
56+
'embargo': null,
57+
}, {
58+
'articleVersion': 'accepted',
59+
'option': 1,
60+
'conditions': ['Publisher copyright and source must be acknowledged', 'Must link to publisher version'],
61+
'prerequisites': [],
62+
'locations': ['Author\'s Homepage', 'Institutional Website'],
63+
'licenses': ['CC BY-NC-ND'],
64+
'embargo': null,
65+
}, {
66+
'articleVersion': 'accepted',
67+
'option': 2,
68+
'conditions': ['Publisher copyright and source must be acknowledged', 'Must link to publisher version'],
69+
'prerequisites': ['If Required by Funder'],
70+
'locations': ['Non-Commercial Repository'],
71+
'licenses': ['CC BY-NC-ND'],
72+
'embargo': { amount: 6, units: 'Months' },
73+
}, {
74+
'articleVersion': 'accepted',
75+
'option': 3,
76+
'conditions': ['Publisher copyright and source must be acknowledged', 'Must link to publisher version'],
77+
'prerequisites': [],
78+
'locations': ['Non-Commercial Repository'],
79+
'licenses': [],
80+
'embargo': null,
81+
}],
82+
'urls': {
83+
'http://download.thelancet.com/flatcontentassets/authors/lancet-information-for-authors.pdf': 'Guidelines for Authors',
84+
'http://www.thelancet.com/journals/lancet/article/PIIS0140-6736%2813%2960720-5/fulltext': 'The Lancet journals welcome a new open access policy',
85+
'http://www.thelancet.com/lancet-information-for-authors/after-publication': 'What happens after publication?',
86+
'http://www.thelancet.com/lancet/information-for-authors/disclosure-of-results': 'Disclosure of results before publication',
87+
'https://www.elsevier.com/__data/assets/pdf_file/0005/78476/external-embargo-list.pdf': 'Journal Embargo Period List',
88+
'https://www.elsevier.com/__data/assets/pdf_file/0011/78473/UK-Embargo-Periods.pdf': 'Journal Embargo List for UK Authors',
89+
},
90+
'openAccessProhibited': false,
91+
'publicationCount': 0,
92+
'preArchiving': 'can',
93+
'postArchiving': 'can',
94+
'pubArchiving': 'cannot',
95+
}],
96+
'inDOAJ': false,
97+
}],
98+
},
99+
} as WorkspaceitemSectionJiscOpfPoliciesObject;

‎src/app/submission/objects/submission-objects.effects.ts‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -562,11 +562,11 @@ export class SubmissionObjectEffects {
562562
mappedActions.push(new UpdateSectionDataAction(submissionId, sectionId, sectionData, filteredErrors, sectionErrors));
563563
}
564564

565-
// Sherpa Policies section needs to be updated when the rest response section is empty
566-
const sherpaPoliciesSectionId = findKey(currentState.sections, (section) => section.sectionType === SectionsType.SherpaPolicies);
567-
if (isNotUndefined(sherpaPoliciesSectionId) && isNotEmpty(currentState.sections[sherpaPoliciesSectionId]?.data)
568-
&& isEmpty(sections[sherpaPoliciesSectionId])) {
569-
mappedActions.push(new UpdateSectionDataAction(submissionId, sherpaPoliciesSectionId, null, [], []));
565+
// Opf Policies section needs to be updated when the rest response section is empty
566+
const opfPoliciesSectionId = findKey(currentState.sections, (section) => section.sectionType === SectionsType.OpfPolicies);
567+
if (isNotUndefined(opfPoliciesSectionId) && isNotEmpty(currentState.sections[opfPoliciesSectionId]?.data)
568+
&& isEmpty(sections[opfPoliciesSectionId])) {
569+
mappedActions.push(new UpdateSectionDataAction(submissionId, opfPoliciesSectionId, null, [], []));
570570
}
571571

572572
// When Duplicate Detection step is enabled, add it only if there are duplicates in the response section data

‎src/app/submission/sections/container/section-container.component.html‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
(panelChange)="sectionRef.sectionChange($event)" activeIds="{{ sectionRef.isOpened() ? [sectionRef.sectionId] : [] }}" [destroyOnHide]="false">
88
<ngb-panel id="{{ sectionData.id }}" [type]="sectionRef.isInfo() ? 'info' : ''">
99
<ng-template ngbPanelTitle class="d-flex">
10-
<span [ngClass]="{ 'text-white' : sectionRef.isInfo()}" class="flex-grow-1 section-title" tabindex="0">{{
10+
<span [ngClass]="{ 'text-black' : sectionRef.isInfo()}" class="flex-grow-1 section-title" tabindex="0">{{
1111
'submission.sections.'+sectionData.header | translate
1212
}}</span>
1313
<div class="d-inline-block">

src/app/submission/sections/sherpa-policies/content-accordion/content-accordion.component.html renamed to src/app/submission/sections/opf-policies/content-accordion/content-accordion.component.html

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<div class="d-flex">
44
<button type="button" class="btn btn-link p-0 me-4" (click)="$event.preventDefault()"
55
[attr.aria-expanded]="!collapse.collapsed" aria-controls="collapseExample">
6-
{{version.articleVersion | titlecase}} {{ 'submission.sections.sherpa.publisher.policy.version' |
6+
{{version.articleVersion | titlecase}} {{ 'submission.sections.jisc.publisher.policy.version' |
77
translate
88
}}
99
</button>
@@ -14,7 +14,7 @@
1414
{{version.embargo?.units[0]}}</span>
1515
} @else {
1616
<span><i class="fas fa-hourglass-half"></i> {{
17-
'submission.sections.sherpa.publisher.policy.noembargo' | translate }}</span>
17+
'submission.sections.jisc.publisher.policy.noembargo' | translate }}</span>
1818
}
1919

2020
<span class="m-1 ms-4">
@@ -28,7 +28,7 @@
2828
</span>
2929
} @else {
3030
<span>{{
31-
'submission.sections.sherpa.publisher.policy.nolocation' | translate }}</span>
31+
'submission.sections.jisc.publisher.policy.nolocation' | translate }}</span>
3232
}
3333
</span>
3434
</div>
@@ -49,14 +49,14 @@
4949
<div class="row">
5050
<div class="col-4">
5151
<p class="m-1"><i class="fas fa-hourglass-half"></i> {{
52-
'submission.sections.sherpa.publisher.policy.embargo' | translate }}</p>
52+
'submission.sections.jisc.publisher.policy.embargo' | translate }}</p>
5353
</div>
5454
<div class="col-6">
5555
@if (!!version.embargo) {
5656
<p class="m-1">{{version.embargo.amount}}
5757
{{version.embargo.units}}</p>
5858
} @else {
59-
<p class="m-1">{{ 'submission.sections.sherpa.publisher.policy.noembargo' | translate }}</p>
59+
<p class="m-1">{{ 'submission.sections.jisc.publisher.policy.noembargo' | translate }}</p>
6060
}
6161
</div>
6262
</div>
@@ -65,7 +65,7 @@
6565
<div class="row">
6666
<div class="col-4">
6767
<p class="m-1"><i class="fas fa-certificate"></i> {{
68-
'submission.sections.sherpa.publisher.policy.license' | translate }}</p>
68+
'submission.sections.jisc.publisher.policy.license' | translate }}</p>
6969
</div>
7070
<div class="col-6">
7171
@for (license of version.licenses; track license) {
@@ -78,7 +78,7 @@
7878
<div class="row">
7979
<div class="col-4">
8080
<p class="m-1"><i class="fas fa-exclamation-circle"></i> {{
81-
'submission.sections.sherpa.publisher.policy.prerequisites' | translate }}</p>
81+
'submission.sections.jisc.publisher.policy.prerequisites' | translate }}</p>
8282
</div>
8383
<div class="col-6">
8484
@for (prerequisite of version.prerequisites; track prerequisite) {
@@ -91,7 +91,7 @@
9191
<div class="row">
9292
<div class="col-4">
9393
<p class="m-1"><i class="far fa-folder-open"></i> {{
94-
'submission.sections.sherpa.publisher.policy.location' | translate }}</p>
94+
'submission.sections.jisc.publisher.policy.location' | translate }}</p>
9595
</div>
9696
<div class="col-6">
9797
@for (location of version.locations; track location) {
@@ -104,7 +104,7 @@
104104
<div class="row">
105105
<div class="col-4">
106106
<p class="m-1"><i class="fas fa-tasks"></i> {{
107-
'submission.sections.sherpa.publisher.policy.conditions' | translate }}</p>
107+
'submission.sections.jisc.publisher.policy.conditions' | translate }}</p>
108108
</div>
109109
<div class="col-6">
110110
@for (condition of version.conditions; track condition) {

0 commit comments

Comments
 (0)