[BACKLOG-39677] Added code changes for local Vfs connection changes

abryant-hv · May 17, 2024 · fddfbc6 · fddfbc6
1 parent 1dc5216
commit fddfbc6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/core/src/main/java/org/pentaho/di/connections/vfs/BaseVFSConnectionDetails.java b/core/src/main/java/org/pentaho/di/connections/vfs/BaseVFSConnectionDetails.java
@@ -71,7 +71,6 @@ public String getRootPath() {
     return rootPath;
   }
 
-  @Override
   public void setRootPath( String rootPath ) {
     this.rootPath = StringUtils.isEmpty( rootPath ) ? null : rootPath;
   }
@@ -83,5 +82,6 @@ public void setRootPath( String rootPath ) {
    */
   protected void fillProperties( Map<String, String> props ) {
     props.put( "baRoles", getBaRoles().toString() );
+    props.put( "rootPath", getRootPath() );
   }
 }
diff --git a/core/src/main/java/org/pentaho/di/connections/vfs/VFSConnectionDetails.java b/core/src/main/java/org/pentaho/di/connections/vfs/VFSConnectionDetails.java
@@ -125,18 +125,6 @@ default String getRootPath() {
     return null;
   }
 
-  /**
-   * Sets the root folder path, given as a string.
-   * <p>
-   * An empty root folder path value should be converted to {@code null}.
-   * Further syntax validation is performed only after variable substitution.
-   * <p>
-   * The default interface implementation exists to ensure backward compatibility and does nothing.
-   *
-   * @param rootPath The root path.
-   */
-  default void setRootPath( String rootPath ) { }
-
   /**
    * Gets the list of roles with access to the connection from the Business Analytics product.
    * <p>