Bug 20977: Javascript vars used in confirm_deletion in catalog.js do not match string...
authorKyle M Hall <kyle@bywatetsolutions.com>
Thu, 21 Jun 2018 15:11:54 +0000 (11:11 -0400)
committerNick Clemens <nick@bywatersolutions.com>
Mon, 2 Jul 2018 11:06:59 +0000 (11:06 +0000)
The naming mismatch can cause javascript errors.

Test Plan:
1) Confirm the string vars used in catalog.js match the ones in
   catalog-strings.inc

Signed-off-by: Pierre-Luc Lapointe <pierreluc.lapointe@inLibro.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/catalog-strings.inc
koha-tmpl/intranet-tmpl/prog/js/catalog.js

index 5e92ac0..86502ef 100644 (file)
@@ -2,10 +2,10 @@
     /* Strings for translation */
     var MSG_REPLACE_RECORD = _("Please note that this external search could replace the current record.");
     var MSG_DELETE_ALL_ITEMS = _("%s item(s) are attached to this record. You must delete all items before deleting this record.");
-    var CONFIRM_USED_IN_ORDERS = _("Warning: This record is used in %s order(s). Deleting it could cause serious issues on acquisition module. Are you sure you want to delete this record?");
-    var MSG_USED_IN_ORDERS = _("%s order(s) are using this record. You need order managing permissions to delete this record.");
+    var CONFIRM_RECORD_USED_IN_ORDERS = _("Warning: This record is used in %s order(s). Deleting it could cause serious issues on acquisition module. Are you sure you want to delete this record?");
+    var MSG_RECORD_USED_IN_ORDERS = _("%s order(s) are using this record. You need order managing permissions to delete this record.");
     var CONFIRM_IN_DELETED_ORDERS = _("%s deleted order(s) are using this record. Are you sure you want to delete this record?");
-    var MSG_IN_DELTED_ORDERS = _("%s deleted order(s) are using this record. You need order managing permissions to delete this record.");
+    var MSG_IN_DELETED_ORDERS = _("%s deleted order(s) are using this record. You need order managing permissions to delete this record.");
     var CONFIRM_DELETION_HOLDS = _("%s holds(s) for this record. Are you sure you want to delete this record?");
     var CONFIRM_RECORD_DELETION = _("Are you sure you want to delete this record?");
     var MSG_DELETE_ALL_HOLDS = _("%s hold(s) on this record. You must delete all holds before deleting all items.");
index 19ccd52..2fba833 100644 (file)
@@ -35,7 +35,7 @@ function confirm_deletion() {
         if( order_manage_permission ){
             is_confirmed = confirm( CONFIRM_IN_DELETED_ORDERS.format(countdeletedorders) );
         } else {
-            is_confirmed = alert( MSG_IN_DELTED_ORDERS.format(countdeletedorders) );
+            is_confirmed = alert( MSG_IN_DELETED_ORDERS.format(countdeletedorders) );
         }
     } else if ( holdcount > 0 ) {
         is_confirmed = confirm( CONFIRM_DELETION_HOLDS.format(holdcount) );