From 4693549063259046c80fbd1a90bed6fa586d25c3 Mon Sep 17 00:00:00 2001
From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Mon, 20 Aug 2012 18:45:06 -0400
Subject: [PATCH] More UI work. Saved search selector & search scope OU
 selector & cosmetics

Still not actually functional.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---
 .../src/templates/url_verify/create_session.tt2    | 63 +++++++++++++++-------
 .../js/dojo/openils/widget/FilteringTreeSelect.js  |  1 +
 .../web/js/ui/default/url_verify/create_session.js | 47 +++++++++++++++-
 3 files changed, 91 insertions(+), 20 deletions(-)

diff --git a/Open-ILS/src/templates/url_verify/create_session.tt2 b/Open-ILS/src/templates/url_verify/create_session.tt2
index df0a61cdcd..36799684cd 100644
--- a/Open-ILS/src/templates/url_verify/create_session.tt2
+++ b/Open-ILS/src/templates/url_verify/create_session.tt2
@@ -5,12 +5,27 @@
     dojo.require("dijit.form.CheckBox");
     dojo.require("dijit.form.TextBox");
     dojo.require("openils.Util");
-    dojo.require("openils.widget.OrgUnitFilteringSelect");
     dojo.require("openils.widget.ProgressDialog");
+
+    openils.Util.addOnLoad(
+        function() {
+            progress_dialog.show(true);
+            create_session.populate_saved_searches(dojo.byId("saved-searches"));
+            create_session.prepare_org_selector(dojo.byId("org-selector"));
+            progress_dialog.hide();
+        }
+    );
 </script>
 <script type="text/javascript" src="[% ctx.media_prefix %]/js/ui/default/url_verify/create_session.js"></script>
 <style type="text/css">
     #uv-search { width: 20em; }
+    .note { font-style: italic; background-color: #eee; }
+    #uv-tags-and-subfields { background-color: #ddd; }
+    table.create-session-form th { text-align: right; }
+    table.create-session-form {
+        border-collapse: separate;
+        border-spacing: 0.5ex;
+    }
 </style>
 <div dojoType="dijit.layout.ContentPane" layoutAlign="client">
     <div dojoType="dijit.layout.ContentPane"
@@ -18,19 +33,11 @@
         <div> [% ctx.page_title %] </div>
         <div>
             <button dojoType="dijit.form.Button"
-                onClick="alert('XXX do something');">[% l('XXX Clone Existing') %]</button>
+                onClick="alert('XXX TODO');">[% l('Clone Existing') %]</button>
         </div>
     </div>
-<!--    <div class="oils-acq-basic-roomy">
-        <label for="org_selector">[% l('Show events at and below') %]:</label>
-        <select
-            id="org_selector" jsId="org_selector"
-            dojoType="openils.widget.OrgUnitFilteringSelect"
-            searchAttr="name" labelAttr="name">
-        </select>
-    </div> -->
     <div>
-        <table>
+        <table class="create-session-form">
             <tr>
                 <th>
                     <label for="uv-session-name">[% l("Sesssion name:") %]</label>
@@ -39,6 +46,8 @@
                     <input dojoType="dijit.form.TextBox"
                         id="uv-session-name" jsId="uv_session_name" />
                 </td>
+                <td class="note">
+                </td>
             </tr>
 
             <tr>
@@ -46,10 +55,10 @@
                     <label for="org-selector">[% l('Search scope:') %]</label>
                 </th>
                 <td>
-                    <select id="org-selector" jsId="org_selector"
-                        dojoType="openils.widget.OrgUnitFilteringSelect"
-                        searchAttr="name" labelAttr="name"> </select>
-                    <span class="note">[% l("This will only be used if your search doesn't contain a hand-entered filter such as site(BR1)") %]</span>
+                    <div id="org-selector"></div>
+                </td>
+                <td class="note">
+                    [% l("This will only be used if your search doesn't contain a hand-entered filter such as site(BR1)") %]
                 </td>
             </tr>
 
@@ -63,9 +72,23 @@
                     <input dojoType="dijit.form.TextBox" id="uv-search"
                         jsId="uv_search" />
                 </td>
+                <td class="note">
+                </td>
             </tr>
 
-            <!-- XXX TODO multiselect from saved searches here -->
+            <tr>
+                <th>
+                    <label for="saved-searches">[% l("Saved searches:") %]</label>
+                </th>
+                <td><!-- XXX we're just assuming this list won't grow so
+                    large as to be unrepresentable in a multiselect?  We
+                    could switch to a PCrudAutocompleteBox if needed for
+                    constant load time regardless of dataset size. -->
+                    <select id="saved-searches" multiple="true" size="6"></select>
+                </td>
+                <td class="note">[% l("Optionally select one or more to combine with 'Search' field above.") %]
+                </td>
+            </tr>
 
             <tr>
                 <th>
@@ -75,11 +98,13 @@
                     <input dojoType="dijit.form.CheckBox" id="no-url-selection"
                         jsId="no_url_selection" />
                 </td>
+                <td class="note">
+                </td>
             </tr>
 
             <tr>
                 <th>
-                    [% l('Tags and subfields with URLs:') %]
+                    [% l('Tags and subfields possibly containing URLs:') %]
                 </th>
                 <td>
                     <div id="uv-tags-and-subfields">
@@ -88,10 +113,12 @@
                         [% l("Tag") %]
                         <input type="text" size="4" maxlength="3" />
                         [% l("Subfield(s)") %]
-                        <input type="text" size="20" />
+                        <input type="text" size="15" />
                         <a href="javascript:create_session.tag_and_subfields.add();">[% l('Add') %]
                     </div>
                 </td>
+                <td class="note">
+                </td>
             </tr>
         </table>
 
diff --git a/Open-ILS/web/js/dojo/openils/widget/FilteringTreeSelect.js b/Open-ILS/web/js/dojo/openils/widget/FilteringTreeSelect.js
index 6443f89e0a..7791369020 100644
--- a/Open-ILS/web/js/dojo/openils/widget/FilteringTreeSelect.js
+++ b/Open-ILS/web/js/dojo/openils/widget/FilteringTreeSelect.js
@@ -16,6 +16,7 @@ tree1.startup();
 if(!dojo._hasResource["openils.widget.FilteringTreeSelect"]){
     dojo.provide("openils.widget.FilteringTreeSelect");
     dojo.require("dijit.form.FilteringSelect");
+    dojo.require("dojo.data.ItemFileWriteStore");
 
     dojo.declare(
         "openils.widget.FilteringTreeSelect", [dijit.form.FilteringSelect], {
diff --git a/Open-ILS/web/js/ui/default/url_verify/create_session.js b/Open-ILS/web/js/ui/default/url_verify/create_session.js
index 450963dd1e..2ee7284ec3 100644
--- a/Open-ILS/web/js/ui/default/url_verify/create_session.js
+++ b/Open-ILS/web/js/ui/default/url_verify/create_session.js
@@ -1,10 +1,53 @@
+dojo.require("dojo.data.ItemFileWriteStore");
+dojo.require("dojox.jsonPath");
+
+dojo.require("fieldmapper.OrgUtils");
+dojo.require("openils.PermaCrud");
+dojo.require("openils.widget.FilteringTreeSelect");
+
 var create_session = {};
 
 (function() {
     create_session.begin = function() {
         alert("XXX TODO implement");
-    }
+    };
+
+    /* At least in Dojo 1.3.3 (I know, I know), dijit.form.MultiSelect does
+     * not behave like FilteringSelect, like you might thing, and work from a
+     * data store.  So we'll use a native <select> control, which will have
+     * fewer moving parts to go haywire anyway.
+     */
+    create_session.populate_saved_searches = function(node) {
+        var list = (new openils.PermaCrud()).retrieveAll(
+            "asq", {"order_by": {"asq": "label"}}
+        );
+
+        dojo.forEach(
+            list,
+            function(o) {
+                dojo.create(
+                    "option", {
+                        "innerHTML": o.label(),
+                        "value": o.id(),
+                        "title": o.query_text()
+                    }, node, "last"
+                );
+            }
+        );
+    };
+
+    /* set up an all-org-units-in-the-tree selector */
+    create_session.prepare_org_selector = function(node) {
+        var widget = new openils.widget.FilteringTreeSelect(null, node);
+        widget.searchAttr = "name";
+        widget.labelAttr = "name";
+        widget.tree = fieldmapper.aou.globalOrgTree;
+        widget.parentField = 'parent_ou';
+        widget.startup();
+        widget.attr("value", openils.User.user.ws_ou());
+    };
 
+    /* This is the thing that lets you add/remove rows of tab/subfield pairs */
     function TagAndSubfieldsMgr(container_id) {
         var self = this;
 
@@ -31,7 +74,7 @@ var create_session = {};
                     "innerHTML": "<span class='t-and-s-tag'>" +
                         newdata.tag +
                         "</span> \u2021<span class='t-and-s-subfields'>" +
-                        newdata.subfields + "</span>"
+                        newdata.subfields + "</span> "
                 }, this.container_id, "last"
             );
             dojo.create(
-- 
2.11.0