(zeppelin) branch branch-0.12 updated: [ZEPPELIN-6164][HOTFIX] Fix the problem that previous PR for does not …

[chanholee]

This is an automated email from the ASF dual-hosted git repository.

chanholee pushed a commit to branch branch-0.12
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/branch-0.12 by this push:
     new 84dfd6a119 [ZEPPELIN-6164][HOTFIX] Fix the problem that previous PR 
for does not …
84dfd6a119 is described below

commit 84dfd6a119b139f82d58a74e67bca2ee2e5a7d87
Author: Paul Zhang <xzhang...@126.com>
AuthorDate: Fri Aug 22 20:20:20 2025 +0800

    [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 <chanho...@apache.org>
    (cherry picked from commit 31f7d569095fc56104c75f58d94e12431ffbcef3)
    Signed-off-by: ChanHo Lee <chanho...@apache.org>
---
 .../src/app/pages/workspace/interpreter/item/item.component.ts        | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/zeppelin-web-angular/src/app/pages/workspace/interpreter/item/item.component.ts
 
b/zeppelin-web-angular/src/app/pages/workspace/interpreter/item/item.component.ts
index 5210efafb0..ad52ba6a04 100644
--- 
a/zeppelin-web-angular/src/app/pages/workspace/interpreter/item/item.component.ts
+++ 
b/zeppelin-web-angular/src/app/pages/workspace/interpreter/item/item.component.ts
@@ -82,6 +82,8 @@ export class InterpreterItemComponent extends 
DestroyHookComponent implements On
   }
 
   handleSave() {
+    this.addProperties();
+    this.addDependence();
     const formData = this.formGroup.getRawValue();
     // tslint:disable-next-line:no-any
     const properties: Record<any, any> = {};
@@ -98,9 +100,7 @@ export class InterpreterItemComponent extends 
DestroyHookComponent implements On
           name: key
         };
       });
-    this.addProperties();
     formData.properties = properties;
-    this.addDependence();
     // tslint:disable-next-line:no-any
     formData.dependencies.forEach((e: any) => {
       e.exclusions = e.exclusions.split(',').filter((s: string) => s !== '');