[open-ils-commits] r11141 - in trunk/Open-ILS: web/js/dojo/openils xul/staff_client/server/admin

[svn at svn.open-ils.org]

Author: erickson
Date: 2008-11-11 15:10:22 -0500 (Tue, 11 Nov 2008)
New Revision: 11141

Modified:
   trunk/Open-ILS/web/js/dojo/openils/User.js
   trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.js
   trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml
Log:
fixed a description dtd.  when we draw the edit dialog, load the existing setting from the grid.  allow for a selected org in the filtering select perm drawer

Modified: trunk/Open-ILS/web/js/dojo/openils/User.js
===================================================================
--- trunk/Open-ILS/web/js/dojo/openils/User.js	2008-11-11 19:47:57 UTC (rev 11140)
+++ trunk/Open-ILS/web/js/dojo/openils/User.js	2008-11-11 20:10:22 UTC (rev 11141)
@@ -233,7 +233,7 @@
          * @param perm The permission to check
          * @param selector The pre-created dijit.form.FilteringSelect object.  
          */
-        buildPermOrgSelector : function(perm, selector) {
+        buildPermOrgSelector : function(perm, selector, selectedOrg) {
             var _u = this;
     
             dojo.require('dojo.data.ItemFileReadStore');
@@ -241,7 +241,10 @@
             function hookupStore(store) {
                 selector.store = store;
                 selector.startup();
-                selector.setValue(_u.user.ws_ou());
+                if(selectedOrg != null)
+                    selector.setValue(selectedOrg);
+                else
+                    selector.setValue(_u.user.ws_ou());
             }
 
             function buildTreePicker(orgList) {

Modified: trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.js
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.js	2008-11-11 19:47:57 UTC (rev 11140)
+++ trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.js	2008-11-11 20:10:22 UTC (rev 11141)
@@ -155,7 +155,7 @@
 function osLaunchEditor(name) {
     osEditDialog._osattr = name;
     osEditDialog.show();
-    user.buildPermOrgSelector('UPDATE_ORG_UNIT_SETTING.' + name, osEditContextSelector);
+    user.buildPermOrgSelector('UPDATE_ORG_UNIT_SETTING.' + name, osEditContextSelector, osSettings[name].context);
     dojo.byId('os-edit-name').innerHTML = osSettings[name].label;
     dojo.byId('os-edit-desc').innerHTML = osSettings[name].desc || '';
 
@@ -164,19 +164,23 @@
     dojo.style(osEditNumberTextBox.domNode, 'display', 'none');
     dojo.style(osEditBoolSelect.domNode, 'display', 'none');
 
+    var widget;
     switch(osSettings[name].type) {
         case 'number':
-            dojo.style(osEditNumberTextBox.domNode, 'display', 'block');
+            widget = osEditNumberTextBox; 
             break;
         case 'currency':
-            dojo.style(osEditCurrencyTextBox.domNode, 'display', 'block');
+            widget = osEditCurrencyTextBox; 
             break;
         case 'bool':
-            dojo.style(osEditBoolSelect.domNode, 'display', 'block');
+            widget = osEditBoolSelect; 
             break;
         default:
-            dojo.style(osEditTextBox.domNode, 'display', 'block');
+            widget = osEditTextBox;
     }
+
+    dojo.style(widget.domNode, 'display', 'block');
+    widget.setValue(osSettings[name].value);
 }
 
 function osEditSetting(deleteMe) {

Modified: trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml	2008-11-11 19:47:57 UTC (rev 11140)
+++ trunk/Open-ILS/xul/staff_client/server/admin/org_unit_settings.xhtml	2008-11-11 20:10:22 UTC (rev 11141)
@@ -95,7 +95,7 @@
                 },
                 'circ.hold_estimate_wait_interval' : {
                     label : '&staff.server.admin.org_settings.circ.hold_estimate_wait_interval;',
-                    desc : '&staff.server.admin.org_settings.circ.circ.hold_estimate_wait_interval;',
+                    desc : '&staff.server.admin.org_settings.circ.hold_estimate_wait_interval.desc;',
                     type : 'number'
                 }
             };