[open-ils-commits] r17829 - in trunk/Open-ILS/web: js/ui/default/acq/common templates/default/acq/common templates/default/acq/search (senator)

[svn at svn.open-ils.org]

Author: senator
Date: 2010-09-20 01:05:00 -0400 (Mon, 20 Sep 2010)
New Revision: 17829

Modified:
   trunk/Open-ILS/web/js/ui/default/acq/common/li_table.js
   trunk/Open-ILS/web/templates/default/acq/common/li_table.tt2
   trunk/Open-ILS/web/templates/default/acq/search/unified.tt2
Log:
Acq: change a FilteringSelect dijit to a regular <select> to avoid a
manifestation  of Dojo bug #11062; add more refresh grid buttons to unified
search

"Refresh Grid" buttons are a sad little stopgap measure to address a problem
where some AutoGrids show only some of the rows they're supposed to. Calling
.refresh() on the AutoGrid consitently right the grid.  Maybe the problem
is some kind of race condition, and maybe it's related to the use of "get"
and "formatter" attributes for some columns.  Or maybe it's something else.
Attempts continue to find the root of the problem and repair it.


Modified: trunk/Open-ILS/web/js/ui/default/acq/common/li_table.js
===================================================================
--- trunk/Open-ILS/web/js/ui/default/acq/common/li_table.js	2010-09-20 04:34:49 UTC (rev 17828)
+++ trunk/Open-ILS/web/js/ui/default/acq/common/li_table.js	2010-09-20 05:05:00 UTC (rev 17829)
@@ -75,11 +75,10 @@
         }
     );
 
-    dojo.connect(acqLitLiActionsSelector, 'onChange', 
-        function() { 
-            self.applySelectedLiAction(this.attr('value')) 
-            acqLitLiActionsSelector.attr('value', '_');
-        });
+    dojo.byId("acq-lit-li-actions-selector").onchange = function() { 
+        self.applySelectedLiAction(this.options[this.selectedIndex].value);
+        this.selectedIndex = 0;
+    };
 
     acqLitCreatePoSubmit.onClick = function() {
         if (self._confirmPoPrepaySituation()) {

Modified: trunk/Open-ILS/web/templates/default/acq/common/li_table.tt2
===================================================================
--- trunk/Open-ILS/web/templates/default/acq/common/li_table.tt2	2010-09-20 04:34:49 UTC (rev 17828)
+++ trunk/Open-ILS/web/templates/default/acq/common/li_table.tt2	2010-09-20 05:05:00 UTC (rev 17829)
@@ -13,7 +13,7 @@
                             <tr>
                                 <td>
                                     <span>
-                                        <select dojoType='dijit.form.FilteringSelect' value='_' jsId='acqLitLiActionsSelector'>
+                                        <select id="acq-lit-li-actions-selector">
                                             <option mask='*'  value='_'>--Actions--</option>
                                             <option mask='sr' value='save_picklist'>Save Items To Selection List</option>
                                             <option mask='pl' value='selector_ready'>Mark Ready for Selector</option>

Modified: trunk/Open-ILS/web/templates/default/acq/search/unified.tt2
===================================================================
--- trunk/Open-ILS/web/templates/default/acq/search/unified.tt2	2010-09-20 04:34:49 UTC (rev 17828)
+++ trunk/Open-ILS/web/templates/default/acq/search/unified.tt2	2010-09-20 05:05:00 UTC (rev 17829)
@@ -89,6 +89,11 @@
         </div>
     </div>
     <div id="acq-unified-results-purchase_order" class="hidden">
+        <div class="acq-unified-result-specific-controls">
+            <button onclick="dijit.byId('acq-unified-po-grid').refresh();">
+                Refresh Grid
+            </button>
+        </div>
         <table
             id="acq-unified-po-grid"
             autoHeight="true"
@@ -111,6 +116,10 @@
     </div>
     <div id="acq-unified-results-picklist" class="hidden">
         <div class="acq-unified-result-specific-controls">
+            <button dojoType="dijit.form.Button"
+                onClick="dijit.byId('acq-unified-pl-grid').refresh();">
+                Refresh Grid
+            </button>
             <div dojoType="dijit.form.DropDownButton">
                 <span>New Selection List</span>
                 <div dojoType="dijit.TooltipDialog"