Bug 5227 [3.4] Enhance the label batch edit interface

[koha.git] / koha-tmpl / intranet-tmpl / prog / en / includes / labels-batches-toolbar.inc

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/labels-batches-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/labels-batches-toolbar.inc
index b0bae2c..f3a2006 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/labels-batches-toolbar.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/labels-batches-toolbar.inc
@@ -1,6 +1,16 @@
 <div id="toolbar">
     <script type="text/JavaScript">
         //<![CDATA[
+           function DeleteConfirm() {
+                var msg = "Are you sure you want to delete batch <!-- TMPL_VAR NAME="batch_id" -->?"
+                var answer = confirm(msg);
+                if (answer) {
+                    window.location = "/cgi-bin/koha/labels/label-manage.pl?op=delete&amp;label_element=batch&amp;element_id=<!-- TMPL_VAR NAME="batch_id" -->";
+                }
+                else {
+                    return; // abort delete
+                }
+            };
             function Remove() {
                 items = new Array;
                 item_num = new Array;
@@ -36,7 +46,13 @@
                 }
             };
             function Add() {
-                window.open("/cgi-bin/koha/labels/label-item-search.pl?batch_id=<!-- TMPL_VAR NAME="batch_id" -->&amp;type=labels",'FindABibIndex','width=875,height=400,toolbar=no,scrollbars=yes');
+                var barcodes = document.getElementById("barcode");
+                if (barcodes.value == '') {
+                    window.open("/cgi-bin/koha/labels/label-item-search.pl?batch_id=<!-- TMPL_VAR NAME="batch_id" -->&amp;type=labels",'FindABibIndex','width=875,height=400,toolbar=no,scrollbars=yes');
+                }
+                else {
+                    document.forms["add_by_barcode"].submit();
+                }
             };
             function DeDuplicate() {
                 window.location = "/cgi-bin/koha/labels/label-edit-batch.pl?op=de_duplicate&amp;batch_id=<!-- TMPL_VAR NAME="batch_id" -->";
@@ -93,6 +109,7 @@
          $(document).ready(function() {
             $("#additemsc").empty();
             $("#removeitemsc").empty();
+            $("#deletebatchc").empty();
             $("#deduplicatec").empty();
             $("#exportitemsc").empty();
             $("#exportbatchc").empty();
@@ -118,6 +135,15 @@
                 onclick: {fn:function(){Remove()}}
             });
 
+            new YAHOO.widget.Button({
+                id: "deletebatch", 
+                type: "link", 
+                href: "#",
+                label: _("Delete Batch"), 
+                container: "deletebatchc",
+                onclick: {fn:function(){DeleteConfirm()}}
+            });
+
             new YAHOO.widget.Button({
                 id: "deduplicate", 
                 type: "link", 
@@ -149,12 +175,11 @@
         //]]>
     </script>
     <ul class="toolbar">
-        <li id="additemsc"><a id="additems" href="#">Add Item(s)</a></li>
+        <li id="additemsc"><a id="additems" href="#">Add Item(s)</a></li><!-- TMPL_IF NAME="table_loop" -->
         <li id="removeitemsc"><a id="removeitems" href="#">Remove Item(s)</a></li>
-        <li><a id="deletebatch" href="/cgi-bin/koha/labels/label-manage.pl?op=delete&amp;label_element=batch&amp;element_id=<!-- TMPL_VAR NAME="batch_id" -->">Delete Batch</a></li>
-                                <!-- FIXME: should use POST to change server state, not GET -->
+        <li id="deletebatchc"><a id="deletebatch" href="#">Delete Batch</a></li>
         <li id="deduplicatec"><a id="deduplicate" href="#">Remove Duplicates</a></li>
         <li id="exportitemsc"><a id="exportitems" href="#">Export Item(s)</a></li>
-        <li id="exportbatchc"><a id="exportbatch" href="#">Export Batch</a></li>
+        <li id="exportbatchc"><a id="exportbatch" href="#">Export Batch</a></li><!-- /TMPL_IF -->
     </ul>
 </div>