From ca3f5f267dd9c16191133f5bd785460aea16b8c8 Mon Sep 17 00:00:00 2001 From: gkolokythas Date: Tue, 15 Oct 2019 18:29:34 +0300 Subject: [PATCH] Refactors "RDA Common Standards" property moving it from FieldSet to Field. --- .../components/datasetprofile/Field.java | 22 ++++++++---------- .../components/datasetprofile/FieldSet.java | 12 +--------- .../datasetprofiledefinition/Field.java | 23 ++++++++++++------- .../datasetprofiledefinition/FieldSet.java | 14 ----------- .../admin/dataset-profile/dataset-profile.ts | 2 +- .../dataset-profile.service.ts | 21 ++++++++++++++--- .../admin/field-editor-model.ts | 5 +++- .../admin/field-set-editor-model.ts | 5 +--- ...file-editor-composite-field.component.html | 8 ------- ...rofile-editor-composite-field.component.ts | 2 -- ...ataset-profile-editor-field.component.html | 9 ++++++++ .../dataset-profile-editor-field.component.ts | 4 +++- ...aset-profile-editor-section.component.html | 2 +- ...ataset-profile-editor-section.component.ts | 1 - .../dataset-profile-editor.component.html | 2 +- .../dataset-profile-editor.component.ts | 7 ------ 16 files changed, 64 insertions(+), 75 deletions(-) diff --git a/dmp-backend/web/src/main/java/eu/eudat/models/data/admin/components/datasetprofile/Field.java b/dmp-backend/web/src/main/java/eu/eudat/models/data/admin/components/datasetprofile/Field.java index 5e96bf6f6..7903a4ddc 100644 --- a/dmp-backend/web/src/main/java/eu/eudat/models/data/admin/components/datasetprofile/Field.java +++ b/dmp-backend/web/src/main/java/eu/eudat/models/data/admin/components/datasetprofile/Field.java @@ -43,6 +43,7 @@ public class Field implements ViewStyleDefinition getValidations() { return this.validations.stream().map(item -> (int) item.getValue()).collect(Collectors.toList()); } - public void setValidations(List validations) { this.validations = ValidationType.fromIntegers(validations); } @@ -145,6 +141,7 @@ public class Field implements ViewStyleDefinition { private String id; private Integer ordinal; - private String rdaCommonStandard; private Multiplicity multiplicity; private String title; private String description; @@ -44,13 +43,6 @@ public class FieldSet implements Comparable, ViewStyleDefinition fieldToShort = this.fields; Collections.sort(fieldToShort); diff --git a/dmp-backend/web/src/main/java/eu/eudat/models/data/entities/xmlmodels/datasetprofiledefinition/Field.java b/dmp-backend/web/src/main/java/eu/eudat/models/data/entities/xmlmodels/datasetprofiledefinition/Field.java index ce530fbc9..b83bcc65e 100644 --- a/dmp-backend/web/src/main/java/eu/eudat/models/data/entities/xmlmodels/datasetprofiledefinition/Field.java +++ b/dmp-backend/web/src/main/java/eu/eudat/models/data/entities/xmlmodels/datasetprofiledefinition/Field.java @@ -18,6 +18,7 @@ import java.util.List; public class Field implements DatabaseViewStyleDefinition, XmlSerializable { private String id; private int ordinal; + private String rdaCommonStandard; private String numbering; private ViewStyle viewStyle; private DefaultValue defaultValue; @@ -28,7 +29,6 @@ public class Field implements DatabaseViewStyleDefinition, XmlSerializable getValidations() { return validations; } - public void setValidations(List validations) { this.validations = validations; } @@ -84,7 +85,6 @@ public class Field implements DatabaseViewStyleDefinition, XmlSerializable { private String id; private int ordinal; - private String rdaCommonStandard; private List fields; private String numbering; private String title; @@ -46,13 +45,6 @@ public class FieldSet implements DatabaseViewStyleDefinition, XmlSerializable { - return this.http.get(this.actionUrl + "getRDACommonStandards"); + getRDACommonStandards(): String[] { + if (!this.rdaCommonStandards && !this.rdaCommonStandardsLoading) { this.getRDACommonStandardsInternal(); } + return this.rdaCommonStandards; + } + + private getRDACommonStandardsInternal() { + this.rdaCommonStandardsLoading = true; + return this.http.get(this.actionUrl + "getRDACommonStandards").pipe(takeUntil(this._destroyed)).subscribe(x => { + this.rdaCommonStandards = x; + this.rdaCommonStandardsLoading = false; + }); } } diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-editor-model.ts b/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-editor-model.ts index ff076d543..d80d1ebbf 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-editor-model.ts +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-editor-model.ts @@ -30,6 +30,7 @@ export class FieldEditorModel extends BaseFormModel { public visible: VisibilityEditorModel = new VisibilityEditorModel(); public data: FieldDataEditorModel; public validations: ValidationType[] = []; + public rdaCommonStandard: string; fromModel(item: Field): FieldEditorModel { this.id = item.id; @@ -39,6 +40,7 @@ export class FieldEditorModel extends BaseFormModel { this.validations = item.validations; this.viewStyle = new ViewStyleEditorModel().fromModel(item.viewStyle); this.visible = new VisibilityEditorModel().fromModel(item.visible); + this.rdaCommonStandard = item.rdaCommonStandard; if (item.data) { if (this.viewStyle.renderStyle === 'combobox') { @@ -66,7 +68,8 @@ export class FieldEditorModel extends BaseFormModel { // title: [this.title], page: [{ value: this.page, disabled: (disabled && !skipDisable.includes('FieldEditorModel.page')) }], ordinal: [{ value: this.ordinal, disabled: (disabled && !skipDisable.includes('FieldEditorModel.ordinal')) }], - validations: [{ value: this.validations, disabled: (disabled && !skipDisable.includes('FieldEditorModel.validations')) }] + validations: [{ value: this.validations, disabled: (disabled && !skipDisable.includes('FieldEditorModel.validations')) }], + rdaCommonStandard: [{value: this.rdaCommonStandard, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.rdaCommonStandard')) }] }); formGroup.addControl('defaultValue', this.defaultValue.buildForm(disabled, skipDisable)); diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-set-editor-model.ts b/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-set-editor-model.ts index f381d0537..37e9f1fe4 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-set-editor-model.ts +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/admin/field-set-editor-model.ts @@ -16,7 +16,6 @@ export class FieldSetEditorModel extends BaseFormModel { public extendedDescription: string; public additionalInformation: string; public hasCommentField: boolean; - public rdaCommonStandard: string; fromModel(item: FieldSet): FieldSetEditorModel { @@ -29,7 +28,6 @@ export class FieldSetEditorModel extends BaseFormModel { this.extendedDescription = item.extendedDescription; this.additionalInformation = item.additionalInformation; this.hasCommentField = item.hasCommentField; - this.rdaCommonStandard = item.rdaCommonStandard; return this; } @@ -41,8 +39,7 @@ export class FieldSetEditorModel extends BaseFormModel { description: [{ value: this.description, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.description')) }], extendedDescription: [{ value: this.extendedDescription, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.extendedDescription')) }], additionalInformation: [{ value: this.additionalInformation, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.additionalInformation')) }], - hasCommentField: [{ value: this.hasCommentField, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.hasCommentField')) }], - rdaCommonStandard: [{value: this.rdaCommonStandard, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.rdaCommonStandard')) }] + hasCommentField: [{ value: this.hasCommentField, disabled: (disabled && !skipDisable.includes('FieldSetEditorModel.hasCommentField')) }] }); const fieldsFormArray = new Array(); this.fields.forEach(item => { diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.html b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.html index 2dfe882f7..5e3d64c3e 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.html +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.html @@ -36,14 +36,6 @@ - - {{'DATASET-PROFILE-EDITOR.STEPS.FORM.COMPOSITE-FIELD.FIELDS.RDA-COMMON-STANDARDS' | translate}} - - - {{property}} - - -
diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.ts b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.ts index f5c1d29c5..d8d433ec1 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.ts +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.ts @@ -1,7 +1,6 @@ import { Component, Input, OnInit } from '@angular/core'; import { FormArray, FormControl, FormGroup } from '@angular/forms'; import { FieldEditorModel } from '../../../admin/field-editor-model'; -import { FieldSetEditorModel } from '../../../admin/field-set-editor-model'; @Component({ selector: 'app-dataset-profile-editor-composite-field-component', @@ -13,7 +12,6 @@ export class DatasetProfileEditorCompositeFieldComponent implements OnInit { @Input() form: FormGroup; @Input() indexPath: string; @Input() viewOnly: boolean; - @Input() rdaCommonStandards: String[]; isComposite = false; isMultiplicityEnabled = false; diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.html b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.html index 0e0547147..fa4f0968c 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.html +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.html @@ -36,6 +36,15 @@ {{enumUtils.toDatasetProfileFieldValidationTypeString(validationTypeEnum.Required)}} + + {{'DATASET-PROFILE-EDITOR.STEPS.FORM.COMPOSITE-FIELD.FIELDS.RDA-COMMON-STANDARDS' | translate}} + + -- + + {{property}} + + +
diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.ts b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.ts index 1d109c26e..bdc243b73 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.ts +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/field/dataset-profile-editor-field.component.ts @@ -16,6 +16,7 @@ import { RuleEditorModel } from '../../../admin/rule-editor-model'; import { DatePickerDataEditorModel } from '../../../admin/field-data/date-picker-data-editor-models'; import { ResearchersAutoCompleteFieldDataEditorModel } from '../../../admin/field-data/researchers-auto-complete-field-data-editor-model'; import { DatasetProfileInternalDmpEntitiesType } from '../../../../../../core/common/enum/dataset-profile-internal-dmp-entities-type'; +import { DatasetProfileService } from "../../../../../../core/services/dataset-profile/dataset-profile.service"; @Component({ selector: 'app-dataset-profile-editor-field-component', @@ -33,7 +34,8 @@ export class DatasetProfileEditorFieldComponent extends BaseComponent implements constructor( - public enumUtils: EnumUtils + public enumUtils: EnumUtils, + public datasetProfileService: DatasetProfileService ) { super(); } ngOnInit() { diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.html b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.html index 99dbc473b..f34dc1eee 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.html +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.html @@ -42,7 +42,7 @@ - +
diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.ts b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.ts index c4e937bc8..98b885051 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.ts +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/section/dataset-profile-editor-section.component.ts @@ -18,7 +18,6 @@ export class DatasetProfileEditorSectionComponent extends BaseComponent implemen @Input() dataModel: SectionEditorModel; @Input() indexPath: string; @Input() viewOnly: boolean; - @Input() rdaCommonStandards: String[]; constructor() { super(); } diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.html b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.html index 56283b3cc..65945f77d 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.html +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.html @@ -46,7 +46,7 @@
- +
diff --git a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.ts b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.ts index 16094a2f2..afbba1dec 100644 --- a/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.ts +++ b/dmp-frontend/src/app/ui/admin/dataset-profile/editor/dataset-profile-editor.component.ts @@ -46,7 +46,6 @@ export class DatasetProfileEditorComponent extends BaseComponent implements OnIn breadCrumbs: Observable; @ViewChild('stepper', { static: false }) stepper: MatHorizontalStepper; viewOnly = false; - rdaCommonStandards: String[]; constructor( private datasetProfileService: DatasetProfileService, @@ -146,12 +145,6 @@ export class DatasetProfileEditorComponent extends BaseComponent implements OnIn this.addPage(); } }); - this.datasetProfileService.getRDACommonStandards() - .subscribe( - data => { - this.rdaCommonStandards = data; - } - ); } prepareForm() {