Skip to content

Commit

Permalink
[rspace-os#39] Addresses suggestions re iRODS file system setup UX.
Browse files Browse the repository at this point in the history
Retrieve and use default labels from system.properties.
  • Loading branch information
ll4strw committed Jul 5, 2024
1 parent 6a31a5b commit 62471d4
Showing 1 changed file with 21 additions and 3 deletions.
24 changes: 21 additions & 3 deletions src/main/webapp/scripts/pages/system/netfilesystem_mod.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
define(function() {

var sysNetFileSysDetUrl;
var sysNetfileSysDetAuthPasswd;

var fileSystemsArray;

function loadNetFileSystemsList() {
Expand Down Expand Up @@ -33,6 +36,7 @@ define(function() {
}

function setFileSystemClientTypeLabels() {

$.each(fileSystemsArray, function(i, fs) {
if (fs.clientType === 'SAMBA') {
fs.clientTypeLabel = 'SMBv1';
Expand Down Expand Up @@ -137,7 +141,7 @@ define(function() {
$('#fileSystemDetailsSftpDirChoiceNo').prop('checked', isSftpClient && !fileSystemRequiresUserDirs(fileSystem));

refreshClientTypeRows();

$('#fileSystemName').val(fileSystem.name || "");
$('#fileSystemUrl').val(fileSystem.url || "");

Expand Down Expand Up @@ -268,6 +272,15 @@ define(function() {
//therefore we hide the choice from non SFTP clients but we also
//have to give it a value in the UI else the UI framework throws an error on save
function refreshClientTypeRows() {

// retrieve default label values from system.properties
if (sysNetFileSysDetUrl === undefined) {
sysNetFileSysDetUrl = $("label[for='fileSystemUrl']").text();
}
if (sysNetfileSysDetAuthPasswd === undefined) {
sysNetfileSysDetAuthPasswd = $('#fileSystemAuthTypePasswordSpan').text();
}

const isSambaClient = $('#fileSystemClientTypeSamba').prop('checked');
const isSambaSmbjClient = isSambaClient && $('#fileSystemClientTypeSambaSmbj').prop('checked');
const isSftpClient = $('#fileSystemClientTypeSftp').prop('checked');
Expand Down Expand Up @@ -296,12 +309,14 @@ define(function() {
if (isSambaClient) {
$('#fileSystemAuthTypePassword').click();
}

if (isSambaClient || isSambaSmbjClient) {
$('#fileSystemUrl')
.attr('title', 'Samba server URL should start with smb://')
.attr('pattern', '^smb://.*');
$("label[for='fileSystemAuthTypePubKey']").show();
$('#fileSystemAuthTypePasswordSpan').text(sysNetfileSysDetAuthPasswd);
$("label[for='fileSystemUrl']").text(sysNetFileSysDetUrl);
} else if (isIrodsClient) {
$('#fileSystemAuthTypePassword').click();
$('#fileSystemUrl')
Expand All @@ -313,6 +328,8 @@ define(function() {
} else {
$('#fileSystemUrl').removeAttr('title').removeAttr('pattern');
$("label[for='fileSystemAuthTypePubKey']").show();
$('#fileSystemAuthTypePasswordSpan').text(sysNetfileSysDetAuthPasswd);
$("label[for='fileSystemUrl']").text(sysNetFileSysDetUrl);
}
}

Expand All @@ -323,7 +340,8 @@ define(function() {
$('#fileSystemPubKeyRegistrationUrl').prop('required', isPubKeyAuth);
}

$(document).ready(function() {
$(document).ready(function() {

$(document).on('click', '#netFileSystemLink', loadNetFileSystemsList);
$(document).on('click', '.fileSystemDetailsButton', showFileSystemDetails);
$(document).on('click', '.fileSystemDeleteButton', deleteFileSystem);
Expand Down

0 comments on commit 62471d4

Please sign in to comment.