dotCMS/core#14999 UI feedback when delete a field without permission (#…

…772)
dotCMS · Aug 17, 2018 · cf17237 · cf17237
1 parent 5837b8f
commit cf17237
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/app/portlets/content-types/edit/content-types-edit.component.spec.ts b/src/app/portlets/content-types/edit/content-types-edit.component.spec.ts
@@ -465,6 +465,26 @@ describe('ContentTypesEditComponent edit mode', () => {
         expect(comp.fields).toEqual(fieldsReturnByServer);
     });
 
+    it('should handle remove field error', () => {
+        spyOn(dotHttpErrorManagerService, 'handle').and.callThrough();
+        const fieldService = fixture.debugElement.injector.get(FieldService);
+        spyOn(fieldService, 'deleteFields').and.returnValue(Observable.throw(mockResponseView(403)));
+
+        const contentTypeFieldsDropZone = de.query(By.css('dot-content-type-fields-drop-zone'));
+
+        const fieldToRemove = {
+            name: 'field 3',
+            id: '3',
+            clazz: 'com.dotcms.contenttype.model.field.ImmutableColumnField',
+            sortOrder: 3
+        };
+
+        // when: the saveFields event is tiggered in content-type-fields-drop-zone
+        contentTypeFieldsDropZone.componentInstance.removeFields.emit(fieldToRemove);
+
+        expect(dotHttpErrorManagerService.handle).toHaveBeenCalledTimes(1);
+    });
+
     describe('update', () => {
         let contentTypeForm: DebugElement;
 

diff --git a/src/app/portlets/content-types/edit/content-types-edit.component.ts b/src/app/portlets/content-types/edit/content-types-edit.component.ts
@@ -151,6 +151,8 @@ export class ContentTypesEditComponent implements OnInit {
             .pluck('fields')
             .subscribe((fields: ContentTypeField[]) => {
                 this.fields = fields;
+            }, (err: ResponseView) => {
+                this.dotHttpErrorManagerService.handle(err).subscribe((() => {}));
             });
     }