[rspace-os#39] Addresses suggestions re iRODS file system setup UX

ll4strw · Jul 5, 2024 · 6a31a5b · 6a31a5b
1 parent 93d7186
commit 6a31a5b
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 8 deletions.
diff --git a/src/main/java/com/researchspace/netfiles/NfsFactory.java b/src/main/java/com/researchspace/netfiles/NfsFactory.java
@@ -89,10 +89,16 @@ public NfsClient getNfsClient(String nfsusername, String nfspassword, NfsFileSys
           fileSystem.getClientOption(NfsFileSystemOption.SFTP_SERVER_PUBLIC_KEY));
     }
     if (NfsClientType.IRODS.equals(clientType)) {
+      int irodsPort;
+      if (StringUtils.isEmpty(fileSystem.getClientOption(NfsFileSystemOption.IRODS_PORT))) {
+        irodsPort = 1247;
+      } else {
+        irodsPort = Integer.parseInt(fileSystem.getClientOption(NfsFileSystemOption.IRODS_PORT));
+      }
       return new IRODSClient(
           new IRODSAccount(
               fileSystem.getUrl(),
-              Integer.parseInt(fileSystem.getClientOption(NfsFileSystemOption.IRODS_PORT)),
+              irodsPort,
               nfsusername,
               nfspassword,
               fileSystem.getClientOption(NfsFileSystemOption.IRODS_HOME_DIR),

diff --git a/src/main/resources/bundles/system/system.properties b/src/main/resources/bundles/system/system.properties
@@ -277,7 +277,7 @@ system.netfilesystem.details.client.sftp.server.public.key=SFTP server public ke
 system.netfilesystem.details.client.sftp.server.dir.choice=User subdirectory required
 system.netfilesystem.details.client.irods=iRODS
 system.netfilesystem.details.client.irods.zone=iRODS Zone
-system.netfilesystem.details.client.irods.homedir=Home Directory
+system.netfilesystem.details.client.irods.homedir=iRODS Search Path
 system.netfilesystem.details.client.irods.port= iRODS Port
 system.netfilesystem.details.auth=Authentication Type
 system.netfilesystem.details.auth.password=Username/Password

diff --git a/src/main/webapp/WEB-INF/pages/system/netfilesystem_ajax.jsp b/src/main/webapp/WEB-INF/pages/system/netfilesystem_ajax.jsp
@@ -123,12 +123,12 @@
             <tr class="fileSystemDetailsIrodsZoneRow">
                <td><label for="fileSystemIrodsZone">
                    <spring:message code="system.netfilesystem.details.client.irods.zone" /></label></td>
-               <td><input id="fileSystemIrodsZone" type="text" style="width: 20em" /></td>
+               <td><input id="fileSystemIrodsZone" type="text" style="width: 20em" required/></td>
             </tr>
             <tr class="fileSystemDetailsIrodsHomeDirRow">
                <td><label for="fileSystemIrodsHomeDir">
                    <spring:message code="system.netfilesystem.details.client.irods.homedir" /></label></td>
-               <td><input id="fileSystemIrodsHomeDir" type="text" style="width: 20em" /></td>
+               <td><input id="fileSystemIrodsHomeDir" type="text" style="width: 20em" required/></td>
             </tr>
             <tr class="fileSystemDetailsIrodsPortRow">
                 <td><label for="fileSystemIrodsPort">
@@ -152,8 +152,8 @@
                 <td><label><spring:message code="system.netfilesystem.details.auth" /></label></td>
                 <td>
                     <label><input type="radio" id="fileSystemAuthTypePassword" name="fileSystemAuthType" value="PASSWORD" required>
-                        <spring:message code="system.netfilesystem.details.auth.password" /></label>
-                    <label><input type="radio" id="fileSystemAuthTypePubKey" name="fileSystemAuthType" value="PUBKEY">
+                        <span id="fileSystemAuthTypePasswordSpan"><spring:message code="system.netfilesystem.details.auth.password" /></span></label>
+                    <label for="fileSystemAuthTypePubKey"><input type="radio" id="fileSystemAuthTypePubKey" name="fileSystemAuthType" value="PUBKEY">
                         <spring:message code="system.netfilesystem.details.auth.pubkey" /></label>
                 </td>
             </tr>

diff --git a/src/main/webapp/scripts/pages/system/netfilesystem_mod.js b/src/main/webapp/scripts/pages/system/netfilesystem_mod.js
@@ -301,8 +301,18 @@ define(function() {
             $('#fileSystemUrl')
                 .attr('title', 'Samba server URL should start with smb://')
                 .attr('pattern', '^smb://.*');
+	    $("label[for='fileSystemAuthTypePubKey']").show();
+        } else if (isIrodsClient) {
+	    $('#fileSystemAuthTypePassword').click();
+	    $('#fileSystemUrl')
+		.removeAttr('pattern')
+		.attr('title', 'iRODS hostname or IP without protocol');
+	    $("label[for='fileSystemUrl']").text('iRODS Host');
+	    $("label[for='fileSystemAuthTypePubKey']").hide();
+	    $('#fileSystemAuthTypePasswordSpan').text('Native');
         } else {
-            $('#fileSystemUrl').removeAttr('title').removeAttr('pattern')
+            $('#fileSystemUrl').removeAttr('title').removeAttr('pattern');
+	    $("label[for='fileSystemAuthTypePubKey']").show();
         }
     }
 
@@ -324,4 +334,4 @@ define(function() {
         $(document).on('submit', '#fileSystemDetailsForm', saveFileSystem);
     });
 
-});
+});