Attachment 55438 Details for Bug 16949 – [SIGNED-OFF] Bug 16949: Batch record deletion says success when no records have been passed in

[patch] [SIGNED-OFF] Bug 16949: Batch record deletion says success when no records have been passed in

SIGNED-OFF-Bug-16949-Batch-record-deletion-says-su.patch (text/plain), 3.11 KB, created by Owen Leonard on 2016-09-09 14:24:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2016-09-09 14:24:52 UTC

Size: 3.11 KB

patch

obsolete

>From 8ae5406b096b5eea6b4bbd3e1523b44eb3b574e4 Mon Sep 17 00:00:00 2001
>From: Aleisha <aleishaamohia@hotmail.com>
>Date: Wed, 20 Jul 2016 22:14:05 +0000
>Subject: [PATCH] [SIGNED-OFF] Bug 16949: Batch record deletion says success
> when no records have been passed in
>Content-Type: text/plain; charset="utf-8"
>
>The reason this happens is because the page will say success when the
>total number of records given equals the total number of successful
>deletions. If you pass in no records, there are no successful deletions
>--> 0 = 0 --> it thinks it has been successful. This patch adds a check
>that validates if any checkboxes were selected before submitting the
>final form.
>
>I have removed the check for if any records were selected AFTER the form
>has been submitted because it seemed unnecessary if the form can't be
>submitted without selection of records anyway.
>
>To test:
>1) Go to Tools -> Batch record deletion
>2) Put in a record number and click Continue
>3) Deselect the record so that it doesn't actually delete and click
>   Delete selected records
>4) Page says 'All records have been deleted successfully!'
>5) Apply patch. Go back and repeat step 3
>6) Form should not submit and you should receive an alert saying that no
>   records have been selected.
>7) If you try selecting and deleting a record after this alert, it
>   should still work
>
>Note: Have also changed the wording of error in Step 1 when you are
>entering record numbers to delete.
>
>Sponsored-by: Catalyst IT
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>---
> .../prog/en/modules/tools/batch_delete_records.tt  | 22 +++++++++++++++++++++-
> 1 file changed, 21 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_delete_records.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_delete_records.tt
>index 6f6f58a..ceb5bd4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_delete_records.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_delete_records.tt
>@@ -60,6 +60,26 @@ $(document).ready(function() {
>     "aaSorting": [],
>     "bPaginate": false
>   }));
>+
>+  function Checkbox(){
>+    var form = document.getElementById('selectrecords');
>+    var inputs = form.getElementsByTagName('input');
>+    var checked = false;
>+    for (var i=0; i<inputs.length; i++) {
>+      if (inputs[i].type == 'checkbox' && inputs[i].name == 'record_id') {
>+        checked = inputs[i].checked;
>+        if (checked) return true;
>+      }
>+    }
>+    return false;
>+  }
>+
>+  $("#selectrecords").on("submit",function(){
>+    if (Checkbox() == false){
>+      alert(_("No records have been selected."));
>+      return false;
>+    }
>+  });
> });
> //]]>
> </script>
>@@ -150,7 +170,7 @@ $(document).ready(function() {
>           | <a id="selectwithoutitems" href="#">Select without items</a>
>           | <a id="selectnotreserved" href="#">Select without holds</a>
>         </div>
>-        <form action="/cgi-bin/koha/tools/batch_delete_records.pl" method="post">
>+        <form action="/cgi-bin/koha/tools/batch_delete_records.pl" method="post" id="selectrecords">
>           <table id="biblios" class="records">
>             <thead>
>               <tr>
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16949: 53540 | 53792 | 53870 | 54127 | 54901 | 55438 | 55514 | 55515