Skip to content

Commit 31f7d56

Browse files
authored
[ZEPPELIN-6164][HOTFIX] Fix the problem that previous PR for does not …
…work ### What is this PR for? Fix the problem that previous PR for ZEPPELIN-6164 does not work. ### What type of PR is it? Bug Fix ### Todos ### What is the Jira issue? [ZEPPELIN-6164](https://issues.apache.org/jira/browse/ZEPPELIN-6164) ### How should this be tested? Compiled and tested manually. ### Screenshots (if appropriate) ### Questions: * Does the license files need to update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Closes #5031 from paul8263/ZEPPELIN-6164. Signed-off-by: ChanHo Lee <[email protected]>
1 parent f32b50f commit 31f7d56

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

zeppelin-web-angular/src/app/pages/workspace/interpreter/item/item.component.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,8 @@ export class InterpreterItemComponent extends DestroyHookComponent implements On
8282
}
8383

8484
handleSave() {
85+
this.addProperties();
86+
this.addDependence();
8587
const formData = this.formGroup.getRawValue();
8688
// tslint:disable-next-line:no-any
8789
const properties: Record<any, any> = {};
@@ -98,9 +100,7 @@ export class InterpreterItemComponent extends DestroyHookComponent implements On
98100
name: key
99101
};
100102
});
101-
this.addProperties();
102103
formData.properties = properties;
103-
this.addDependence();
104104
// tslint:disable-next-line:no-any
105105
formData.dependencies.forEach((e: any) => {
106106
e.exclusions = e.exclusions.split(',').filter((s: string) => s !== '');

0 commit comments

Comments
 (0)