Bug 18095: Batch item modification: Better message if no item is modified
authorMarc Véron <veron@veron.ch>
Fri, 10 Feb 2017 15:07:13 +0000 (16:07 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Tue, 14 Feb 2017 14:31:44 +0000 (14:31 +0000)
If no item is modified, the result page of Batch item modification says:
"item(s) modified (with fields modified)."
The message should be: "No items modified"

To reproduce:
- Go to Tools -> Batch item modification
- Put a barcode in and click Continue
- Do not make any changes and/or deselect all item(s)
- Click "Save"
=> Result message reads: "item(s) modified (with fields modified)."

To test:
- Apply patch
- Repeat steps above
- Verify that message makes sense.

NOTE: Also tested positive case with actual field change.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-edit.tt

index 2fd838f..c04733c 100644 (file)
@@ -70,7 +70,11 @@ $(document).ready(function(){
             [% ELSE %]
               <h1>Batch item modification results</h1>
               <div class="dialog message">
-              [% modified_items %] item(s) modified (with [% modified_fields %] fields modified).
+                [% IF (modified_items) %]
+                  [% modified_items %] item(s) modified (with [% modified_fields %] field(s) modified).
+                [% ELSE %]
+                  No items modified.
+                [% END %]
               <fieldset class="action">
                   [% IF src == 'CATALOGUING' # from catalogue/detail.pl > Edit items in a batch%]
                       <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% biblionumber %]">Return to the record</a>