mirror of
https://github.com/DSpace/dspace-angular.git
synced 2025-10-08 10:34:15 +00:00
[CST-3782] Fix issue with dispatchOperationsFromRemoveEvent method
This commit is contained in:
@@ -1,4 +1,4 @@
|
|||||||
import { waitForAsync, TestBed } from '@angular/core/testing';
|
import { TestBed, waitForAsync } from '@angular/core/testing';
|
||||||
|
|
||||||
import { TranslateLoader, TranslateModule } from '@ngx-translate/core';
|
import { TranslateLoader, TranslateModule } from '@ngx-translate/core';
|
||||||
import {
|
import {
|
||||||
@@ -35,10 +35,10 @@ describe('SectionFormOperationsService test suite', () => {
|
|||||||
let serviceAsAny: any;
|
let serviceAsAny: any;
|
||||||
|
|
||||||
const jsonPatchOpBuilder: any = jasmine.createSpyObj('jsonPatchOpBuilder', {
|
const jsonPatchOpBuilder: any = jasmine.createSpyObj('jsonPatchOpBuilder', {
|
||||||
add: jasmine.createSpy('add'),
|
add: jasmine.createSpy('add'),
|
||||||
replace: jasmine.createSpy('replace'),
|
replace: jasmine.createSpy('replace'),
|
||||||
remove: jasmine.createSpy('remove'),
|
remove: jasmine.createSpy('remove'),
|
||||||
});
|
});
|
||||||
const pathCombiner = new JsonPatchOperationPathCombiner('sections', 'test');
|
const pathCombiner = new JsonPatchOperationPathCombiner('sections', 'test');
|
||||||
|
|
||||||
const dynamicFormControlChangeEvent: DynamicFormControlEvent = {
|
const dynamicFormControlChangeEvent: DynamicFormControlEvent = {
|
||||||
|
@@ -285,9 +285,10 @@ export class SectionFormOperationsService {
|
|||||||
previousValue: FormFieldPreviousValueObject): void {
|
previousValue: FormFieldPreviousValueObject): void {
|
||||||
const path = this.getFieldPathFromEvent(event);
|
const path = this.getFieldPathFromEvent(event);
|
||||||
const value = this.getFieldValueFromChangeEvent(event);
|
const value = this.getFieldValueFromChangeEvent(event);
|
||||||
|
console.log(value);
|
||||||
if (this.formBuilder.isQualdropGroup(event.model as DynamicFormControlModel)) {
|
if (this.formBuilder.isQualdropGroup(event.model as DynamicFormControlModel)) {
|
||||||
this.dispatchOperationsFromMap(this.getQualdropValueMap(event), pathCombiner, event, previousValue);
|
this.dispatchOperationsFromMap(this.getQualdropValueMap(event), pathCombiner, event, previousValue);
|
||||||
} else if (isNotEmpty(value) && (value instanceof FormFieldMetadataValueObject && value.hasValue())) {
|
} else if ((isNotEmpty(value) && typeof value === 'string') || (isNotEmpty(value) && value instanceof FormFieldMetadataValueObject && value.hasValue())) {
|
||||||
this.operationsBuilder.remove(pathCombiner.getPath(path));
|
this.operationsBuilder.remove(pathCombiner.getPath(path));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user