hazza/dspace-angular
1
0
Fork 1
mirror of https://github.com/DSpace/dspace-angular.git synced 2025-10-07 10:04:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

[CSTPER-222] handle metadata reading for communities and collections

Browse Source
This commit is contained in:
Alessandro Martelli
2020-11-10 12:37:38 +01:00
parent 65516b3880
commit 5c7a111a61
4 changed files with 9 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/app/core/shared/collection.model.ts
View File

@@ -78,14 +78,13 @@ export class Collection extends DSpaceObject implements ChildHALResource {
* A string representing the unique handle of this Collection
*/
get handle(): string {
const metadataValue = this.firstMetadataValue('dc.identifier.uri');
return metadataValue ? metadataValue : this._handle;
return this.firstMetadataValue('dc.identifier.uri');
}
set handle(value: string) {
this._handle = value;
}
/**
* The introductory text of this Collection
* Corresponds to the metadata field dc.description

3
src/app/core/shared/community.model.ts
View File

@@ -67,8 +67,7 @@ export class Community extends DSpaceObject implements ChildHALResource {
* A string representing the unique handle of this Community
*/
get handle(): string {
const metadataValue = this.firstMetadataValue('dc.identifier.uri');
return metadataValue ? metadataValue : this._handle;
return this.firstMetadataValue('dc.identifier.uri');
}
set handle(value: string) {

15
src/app/shared/comcol-page-handle/comcol-page-handle.component.spec.ts
View File

@@ -39,7 +39,7 @@ describe('ComcolPageHandleComponent', () => {
});
describe('should create a link pointing the handle', () => {
it('should use the content if it includes the http protocol', () => {
component.content = handleWithProtocol;
fixture.detectChanges();
@@ -48,15 +48,6 @@ describe('ComcolPageHandleComponent', () => {
expect(link.nativeElement.getAttribute('href')).toBe(handleWithProtocol);
expect(link.nativeElement.innerHTML).toBe(handleWithProtocol);
});
it('should combine the base uri to the content if it doesnt include the http protocol', () => {
component.content = handleWithoutProtocol;
fixture.detectChanges();
const link = fixture.debugElement.query(By.css('a'));
expect(link.nativeElement.getAttribute('href')).toBe(handleWithoutProtocolUIURLCombined);
expect(link.nativeElement.innerHTML).toBe(handleWithoutProtocolUIURLCombined);
});
});
});
});

7
src/app/shared/comcol-page-handle/comcol-page-handle.component.ts
View File

@@ -21,7 +21,6 @@ export class ComcolPageHandleComponent {
@Input() content: string;
public getHandle(): string {
return this.content.includes('http') ? this.content
: new UIURLCombiner('/handle/', this.content).toString();
}}
return this.content;
}
}