diff --git a/src/main/resources/META-INF/xnat/spawner/site-admin-elements.yaml b/src/main/resources/META-INF/xnat/spawner/site-admin-elements.yaml
index d9ae46d056acf18288a3bdd9b9ce3e46c41fd8d0..a36dea2ddd51796e2f1cfcfca33354f2dd334e0e 100644
--- a/src/main/resources/META-INF/xnat/spawner/site-admin-elements.yaml
+++ b/src/main/resources/META-INF/xnat/spawner/site-admin-elements.yaml
@@ -38,7 +38,7 @@ siteDescriptionFile:
label: " "
description: "Specify a velocity template file to display on the login page"
siteDescriptionText:
- kind: panel.input.text
+ kind: panel.textarea
id: siteDescriptionText
name: siteDescriptionText
label: " "
@@ -48,7 +48,7 @@ siteUrl:
id: siteUrl
name: siteUrl
label: Site Url
- validation: required id onblur
+ validation: required id
description: ""
adminEmail:
kind: panel.input.email
@@ -70,29 +70,33 @@ archiveRootPath:
id: archiveRootPath
name: archiveRootPath
label: Archive Root Path
- validation: required id onblur
+ validation: required path
description: ""
+ value: ??XNAT.data.siteConfig.archiveRootPath
cachePath:
kind: panel.input.text
id: cachePath
name: cachePath
label: Cache Path
- validation: required id onblur
+ validation: required path
description: ""
+ value: ??XNAT.data.siteConfig.cachePath
prearchivePath:
kind: panel.input.text
id: prearchivePath
name: prearchivePath
label: Prearchive Path
- validation: required id onblur
+ validation: required path
description: ""
+ value: ??XNAT.data.siteConfig.prearchivePath
ftpPath:
kind: panel.input.text
id: ftpPath
name: ftpPath
label: FTP Path
- validation: required id onblur
+ validation: required path
description: ""
+ value: ??XNAT.data.siteConfig.cachePath
buildPath:
kind: panel.input.text
id: buildPath
@@ -100,6 +104,7 @@ buildPath:
label: Build Path
validation: required id onblur
description: ""
+ value: ??XNAT.data.siteConfig.cachePath
pipelinePath:
kind: panel.input.text
id: pipelinePath
@@ -107,6 +112,7 @@ pipelinePath:
label: Pipeline Path
validation: required id onblur
description: ""
+ value: ??XNAT.data.siteConfig.cachePath
dataFolders:
kind: panel.input.text
id: dataFolders
@@ -152,13 +158,10 @@ generalSecuritySettings:
label: Security Channel
value: https
options:
- http:
- label: http
- value: http
- https:
- label: https
- value: https
- config:
+ # value: label # <- simplest setup
+ http: http
+ https: https
+ element:
id: security-channel
title: Security Channel
requireUserLogin:
@@ -172,6 +175,11 @@ userLoginsSessionControls:
kind: panel.form
name: userLoginsSessionControls
label: User Logins / Session Controls
+ method: POST
+ action: /xapi/siteConfig/batch
+ load:
+ lookup: XNAT.data.siteConfig
+ refresh: /xapi/siteConfig
contents:
sessionTimeout:
kind: panel.input.number
diff --git a/src/main/webapp/scripts/globals.js b/src/main/webapp/scripts/globals.js
index 42efbd168c7b2dde85c1fc35e946481b5cffd9fa..032b900554c3159b71372375641b0b55db78261e 100644
--- a/src/main/webapp/scripts/globals.js
+++ b/src/main/webapp/scripts/globals.js
@@ -270,7 +270,7 @@ function extendCopyDeep(){
// return a cloned copy of a single 'obj'
function cloneObject(obj){
- return extend(true, {}, obj);
+ return extend(true, {}, obj || {});
}
// add child objects to 'obj' object from string
diff --git a/src/main/webapp/scripts/xnat/ui/input.js b/src/main/webapp/scripts/xnat/ui/input.js
index 3332fa57f730068d5993d9e0c99ff363aac70cd8..737e31ad144827582630db4b56e70988b03fd4a0 100644
--- a/src/main/webapp/scripts/xnat/ui/input.js
+++ b/src/main/webapp/scripts/xnat/ui/input.js
@@ -29,16 +29,39 @@ var XNAT = getObject(XNAT);
input_ = XNAT.ui.input || {};
function lookupValue(el, lookup){
+ var val = '';
try {
- el.value = eval(lookup)
+ val = eval(lookup.trim())
}
catch (e) {
- el.value = '';
+ val = '';
console.log(e);
}
- return el.value;
+ el.value = val;
+ return val;
}
+ function lookupObjectValue(root, objStr){
+ var val = '';
+ if (!objStr) {
+ objStr = root;
+ root = window;
+ }
+ root = root || window;
+ objStr.toString().trim().split('.').forEach(function(part, i){
+ part = part.trim();
+ // start at the root object
+ if (i === 0) {
+ val = root[part] || {};
+ }
+ else {
+ val = val[part];
+ }
+ });
+ return val;
+ }
+
+
// ========================================
// MAIN FUNCTION
input = function(type, config){
@@ -53,12 +76,12 @@ var XNAT = getObject(XNAT);
// lookup a value if it starts with '??'
var doLookup = '??';
if (config.value && config.value.toString().indexOf(doLookup) === 0) {
- lookupValue(config, config.value.split(doLookup)[1])
+ config.value = lookupObjectValue(config.value.split(doLookup)[1])
}
// lookup a value from a namespaced object
// if no value is given
if (!config.value && config.data && config.data.lookup) {
- lookupValue(config, config.data.lookup)
+ config.value = lookupObjectValue(config.data.lookup)
}
var spawned = spawn('input', config);
return {
@@ -118,7 +141,7 @@ var XNAT = getObject(XNAT);
$(window).load(function(){
$(':input[data-lookup]').each(function(){
var $input = $(this);
- var val = lookupValue(this, $input.dataAttr('lookup'));
+ var val = lookupObjectValue($input.dataAttr('lookup'));
$input.changeVal(val);
});
});
diff --git a/src/main/webapp/scripts/xnat/ui/panel.js b/src/main/webapp/scripts/xnat/ui/panel.js
index 561bb6e5ac91bb333f43cc60c1c75151bbc37828..6753bcdaf4e9d8cc65c0bc94e5fb66d0b773a326 100644
--- a/src/main/webapp/scripts/xnat/ui/panel.js
+++ b/src/main/webapp/scripts/xnat/ui/panel.js
@@ -32,9 +32,48 @@ var XNAT = getObject(XNAT || {});
return obj.data;
}
+ // another way to do this without using eval()
+ // is to loop over object string using dot notation:
+ // var myVal = lookupObjectValue(XNAT, 'data.siteConfig.siteId');
+ // --> myVal == 'myXnatSiteId'
+ function lookupObjectValue(root, objStr){
+ var val = '';
+ if (!objStr) {
+ objStr = root;
+ root = window;
+ }
+ root = root || window;
+ objStr.toString().trim().split('.').forEach(function(part, i){
+ // start at the root object
+ if (i === 0) {
+ val = root[part] || {};
+ }
+ else {
+ val = val[part];
+ }
+ });
+ return val;
+ }
+
+ // string that indicates to look for a namespaced object value
+ var doLookupString = '??';
+
+ function doLookup(input){
+ if (!input) return '';
+ if (input.toString().trim().indexOf(doLookupString) === 0){
+ return lookupObjectValue(window, input.split(doLookupString)[1]);
+ }
+ return input;
+ }
+
+ /**
+ * Initialize panel.
+ * @param [opts] {Object} Config object
+ * @returns {{}}
+ */
panel.init = function panelInit(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.element = opts.element || opts.config || {};
var _target = spawn('div.panel-body', opts.element),
@@ -68,11 +107,10 @@ var XNAT = getObject(XNAT || {});
}
};
-
// creates a panel that's a form that can be submitted
panel.form = function panelForm(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.element = opts.element || opts.config || {};
var _target = spawn('div.panel-body', opts.element),
@@ -140,7 +178,7 @@ var XNAT = getObject(XNAT || {});
obj = opts.load || {};
}
- obj = extend(true, {}, obj);
+ obj = cloneObject(obj);
obj.form = obj.form || obj.target || obj.element || _formPanel;
@@ -321,7 +359,7 @@ var XNAT = getObject(XNAT || {});
// creates a panel that submits all forms contained within
panel.multiForm = function(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.element = opts.element || opts.config || {};
var inner = spawn('div.panel-body', opts.element),
@@ -407,7 +445,7 @@ var XNAT = getObject(XNAT || {});
panel.element = function(opts){
var _element, _inner = [], _target;
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.element = opts.element || opts.config || {};
if (opts.id || opts.element.id) {
opts.element.id = (opts.id || opts.element.id) + '-element';
@@ -443,7 +481,7 @@ var XNAT = getObject(XNAT || {});
};
panel.subhead = function(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.html = opts.html || opts.text || opts.label;
return XNAT.ui.template.panelSubhead(opts).spawned;
};
@@ -453,7 +491,7 @@ var XNAT = getObject(XNAT || {});
var _section, _inner = [], _body;
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.element = opts.element || opts.config || {};
opts.header = opts.header || opts.label || opts.title || '';
@@ -495,34 +533,34 @@ var XNAT = getObject(XNAT || {});
};
panel.input.number = function panelInputNumber(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.type = 'number';
return XNAT.ui.template.panelInput(opts).spawned;
};
panel.input.email = function panelInputEmail(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.type = 'text';
addClassName(opts, 'email');
return XNAT.ui.template.panelInput(opts).spawned;
};
panel.input.password = function panelInputPassword(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.type = 'password';
addClassName(opts, 'password');
return XNAT.ui.template.panelInput(opts).spawned;
};
panel.input.checkbox = function panelInputCheckbox(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.type = 'checkbox';
//addClassName(opts, 'checkbox');
return XNAT.ui.template.panelInput(opts).spawned;
};
panel.input.upload = function panelInputUpload(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.id = (opts.id||randomID('upload-', false));
opts.element = opts.element || opts.config || {};
opts.element.id = opts.id;
@@ -560,13 +598,31 @@ var XNAT = getObject(XNAT || {});
}
};
+ panel.textarea = function(opts){
+ opts = cloneObject(opts);
+ opts.element = opts.element || opts.config || {};
+ if (opts.id) opts.element.id = opts.id;
+ if (opts.name) opts.element.name = opts.name;
+ opts.element.html =
+ opts.element.html ||
+ opts.element.value ||
+ opts.value ||
+ opts.text ||
+ opts.html || '';
+
+ opts.element.html = doLookup(opts.element.html);
+
+ var textarea = spawn('textarea', opts.element);
+ return XNAT.ui.template.panelDisplay(opts, textarea).spawned;
+ };
+
panel.select = {};
panel.select.menu = function panelSelectSingle(opts, multi){
var _menu;
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.element = opts.element || opts.config || {};
opts.element.name = opts.element.name || opts.name || '';
opts.element.id = opts.element.id || opts.id || toDashed(opts.element.name);
@@ -596,7 +652,7 @@ var XNAT = getObject(XNAT || {});
};
panel.selectMenu = function panelSelectMenu(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
return XNAT.ui.template.panelSelect(opts).spawned;
};
@@ -640,94 +696,6 @@ var XNAT = getObject(XNAT || {});
// !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
- /**
- * Initialize panel and optionally render it.
- * If 'opts' argument is passed, 'setup' method will run.
- * If 'container' argument is passed, the 'render' method will run.
- * So if both args are passed, 'setup' and 'render' do NOT need to be called.
- * @param [opts] {Object} Config object
- * @param [container] {Element} Container for panel
- * @returns {{}}
- */
- function panel(opts, container){
-
- // `this` object
- var _panel = {};
-
- /**
- * Standard panel widget
- */
- function newPanel(){
-
- var sections = [
- ['div.panel-heading', [
- ['h3.panel-title', _panel.opts.title]
- ]],
- ['div.panel-body', _panel.opts.body]
- ];
-
- if (_panel.opts.footer) {
- sections.push(['div.panel-footer', _panel.opts.footer])
- }
-
- return spawn((_panel.opts.tag) + '.panel.panel-default', _panel.opts.attr, sections);
- //return $(spawn('div.panel.panel-default')).append(content);
- }
-
- /**
- * Sets up elements before rendering to the page
- * @param _opts Config object
- * @returns {{}}
- */
- _panel.setup = function _panelSetup(_opts){
-
- _panel.opts = extend(true, {}, _opts);
-
- _panel.opts.tag = _panel.opts.tag || 'div';
- _panel.opts.title = _panel.opts.title || _panel.opts.header || '';
- _panel.opts.body = _panel.opts.body || _panel.opts.content || '';
- _panel.opts.footer = _panel.opts.footer || '';
-
- _panel.opts.attr = _panel.opts.attr || {};
-
- if (_panel.opts.id) {
- _panel.opts.attr.id = _panel.opts.id
- }
-
- if (_panel.opts.name) {
- _panel.opts.attr.data = getObject(_panel.opts.attr.data);
- _panel.opts.attr.data.name = _panel.opts.name;
- }
-
- _panel.panel = _panel.element = newPanel();
-
- return _panel;
- };
-
- // if 'opts' arg is passed to .panel(), call .setup()
- if (opts) {
- _panel.setup(opts);
- }
-
- // render the panel and append to 'container'
- _panel.render = function _panelRender(container){
- $$(container).append(_panel.panel);
- return _panel;
- };
-
- // render immediately if 'container' is specified
- if (container) {
- _panel.render(container);
- }
-
- _panel.get = function _panelGet(){
- return _panel.element;
- };
-
- return _panel;
-
- }
-
/**
* Panel widget with default 'Submit' and 'Revert' buttons
diff --git a/src/main/webapp/scripts/xnat/ui/templates.js b/src/main/webapp/scripts/xnat/ui/templates.js
index ad59c311faf06359daa3ef64dfe5a923e58aade5..e61089a4ceeadf7268e1dd12355d5a4da50f3878 100644
--- a/src/main/webapp/scripts/xnat/ui/templates.js
+++ b/src/main/webapp/scripts/xnat/ui/templates.js
@@ -42,7 +42,7 @@ var XNAT = getObject(XNAT);
function lookupValue(el, lookup){
var val = '';
try {
- val = eval(lookup);
+ val = eval(lookup.trim());
}
catch (e) {
val = '';
@@ -65,10 +65,10 @@ var XNAT = getObject(XNAT);
root = window;
}
root = root || window;
- objStr.toString().split('.').forEach(function(part, i){
+ objStr.toString().trim().split('.').forEach(function(part, i){
// start at the root object
if (i === 0) {
- val = root[part];
+ val = root[part] || {};
}
else {
val = val[part];
@@ -82,7 +82,7 @@ var XNAT = getObject(XNAT);
// subhead element to segment panels
template.panelSubhead = function(opts){
var _templ, _spawn, _html;
- opts = getObject(opts);
+ opts = cloneObject(opts);
_templ = ['h4.panel-subhead', opts];
_spawn = function(){
return spawn.apply(null, _templ);
@@ -103,7 +103,7 @@ var XNAT = getObject(XNAT);
// generic panel element
template.panelElement = function(opts, content){
var _templ, _spawn, _html;
- opts = getObject(opts);
+ opts = cloneObject(opts);
addClassName(opts, 'panel-element');
_templ = [
'div|data-name='+(opts.name||''),
@@ -131,18 +131,26 @@ var XNAT = getObject(XNAT);
// ========================================
// display only element for form panels
template.panelDisplay = function(opts, element){
- opts = getObject(opts);
+
+ opts = cloneObject(opts);
opts.id = opts.id||toDashed(opts.name||'');
opts.label = opts.label||opts.title||opts.name||'';
+
+ // pass in an element or create a new 'div' element
+ element =
+ element || spawn('div', {
+ id: opts.id,
+ className: opts.className||'',
+ title: opts.title||opts.name||opts.id,
+ html: opts.value||opts.html||opts.text||opts.body||''
+ });
+
return template.panelElement(opts, [
- ['label.element-label|for='+opts.id, opts.label],
+ ['label.element-label|for='+element.id||opts.id, opts.label],
['div.element-wrapper', [
- element || ['div', {
- id: opts.id,
- className: opts.className||'',
- title: opts.title||opts.name||opts.id,
- html: opts.value||opts.html||opts.text||opts.body||''
- }],
+
+ element ,
+
['div.description', opts.description]
]]
]);
@@ -153,7 +161,7 @@ var XNAT = getObject(XNAT);
// ========================================
// input element for form panels
template.panelInput = function(opts, element){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.name = opts.name || opts.id || randomID('input-', false);
opts.id = opts.id||toDashed(opts.name||'');
opts.label = opts.label||opts.title||opts.name||'';
@@ -192,7 +200,7 @@ var XNAT = getObject(XNAT);
// look up a namespaced object value if the value starts with '??'
var doLookup = '??';
if (opts.value && opts.value.toString().indexOf(doLookup) === 0) {
- lookupValue(element, opts.value.split(doLookup)[1]);
+ element.value = lookupObjectValue(opts.value.split(doLookup)[1]);
}
if (opts.load) {
@@ -233,7 +241,7 @@ var XNAT = getObject(XNAT);
// ========================================
// select element for form panels
template.panelSelect = function(opts){
- opts = getObject(opts);
+ opts = cloneObject(opts);
opts.name = opts.name || opts.id || randomID('select-', false);
opts.id = opts.id || toDashed(opts.name||'');
opts.element = extend({
@@ -265,7 +273,7 @@ var XNAT = getObject(XNAT);
template.panelElementGroup = function(opts, elements){
- opts = getObject(opts);
+ opts = cloneObject(opts);
return template.panelElement(opts, [
['label.element-label|for='+opts.id, opts.label||opts.title||opts.name],
['div.element-wrapper', elements]