From 2c28d9c456d9da73e941bc7bd83613385da19c84 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 18 Aug 2016 11:04:38 -0400 Subject: [PATCH] [SIGNED-OFF] Bug 17147 - Streamline messages following batch record modification This patch changes the display of informational messages during and after the batch record modification process. Instead of showing a separate dialog for each record modified, messages are now grouped into one dialog. To test, apply the patch and clear your browser cache if necessary. You must have at least one MARC modification template defined. - Go to Tools -> Batch record modification. - Submit a list of biblionumbers which contains at least one number which doesn't exist in your database. - Confirm that warning and success messages are grouped instead of showing in separate dialogs. Signed-off-by: Aleisha Amohia --- koha-tmpl/intranet-tmpl/prog/css/staff-global.css | 3 +- .../en/modules/tools/batch_record_modification.tt | 85 +++++++++++++--------- 2 files changed, 52 insertions(+), 36 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css index 525cd0e..5865b1d 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css +++ b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css @@ -1067,7 +1067,8 @@ div.alert strong { border : 1px inset #999999; } -.approve i.fa { +.approve i.fa, +.success i.fa { color: green; } diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_record_modification.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_record_modification.tt index 2d7d534..57fef07 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_record_modification.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_record_modification.tt @@ -91,35 +91,42 @@ $(document).ready(function() {

Batch record modification

- [% FOREACH message IN messages %] - [% IF message.type == 'success' %] -
- [% ELSIF message.type == 'warning' %] -
- [% ELSIF message.type == 'error' %] -
- [% END %] - [% biblio.title %] - [% IF message.code == 'no_action_defined_for_the_template' %] - The selected template (id=[% message.mmtid%]) does not exist or no action is defined. - [% ELSIF message.code == 'biblio_not_exists' %] - The biblionumber [% message.biblionumber %] does not exist in the database. - [% ELSIF message.code == 'authority_not_exists' %] - The authority id [% message.authid %] does not exist in the database. - [% ELSIF message.code == 'biblio_not_modified' %] - The biblio [% message.biblionumber %] has not been modified. An error occurred on modifying it. - [% ELSIF message.code == 'authority_not_modified' %] - The authority [% message.authid %] has not been modified. An error occurred on modifying it. - [% ELSIF message.code == 'biblio_modified' %] - The biblio [% message.biblionumber %] has successfully been modified. - [% ELSIF message.code == 'authority_modified' %] - The authority [% message.authid %] has successfully been modified. - [% END %] - [% IF message.error %] - (The error was: [% message.error%], see the Koha logfile for more information). + + [% IF ( messages ) %] +
+
    + [% FOREACH message IN messages %] + [% IF message.type == 'success' %] +
  • + [% ELSIF message.type == 'warning' %] +
  • + [% ELSIF message.type == 'error' %] +
  • + [% END %] + [% IF message.code == 'no_action_defined_for_the_template' %] + The selected template (id=[% message.mmtid%]) does not exist or no action is defined. + [% ELSIF message.code == 'biblio_not_exists' %] + Bibliographic record [% message.biblionumber %] does not exist in the database. + [% ELSIF message.code == 'authority_not_exists' %] + Authority record [% message.authid %] does not exist in the database. + [% ELSIF message.code == 'biblio_not_modified' %] + Bibliographic record [% message.biblionumber %] has not been modified. An error occurred on modifying it. + [% ELSIF message.code == 'authority_not_modified' %] + Authority record [% message.authid %] has not been modified. An error occurred on modifying it. + [% ELSIF message.code == 'biblio_modified' %] + Bibliographic record [% message.biblionumber %] has successfully been modified. + [% ELSIF message.code == 'authority_modified' %] + Bibliographic record [% message.authid %] has successfully been modified. + [% END %] + [% IF message.error %] + (The error was: [% message.error%]. See the Koha logfile for more information). + [% END %] +
  • + [% END %] +
+
[% END %] -
- [% END %] + [% IF view == 'form' %]
@@ -257,24 +264,32 @@ $(document).ready(function() {
[% ELSE %] - There is no record ids defined. +
+ No records were modified. New batch record modification +
[% END %] [% ELSIF view == 'report' %] [% IF report.total_records == report.total_success %] - All records have successfully been modified! +
+ All records have successfully been modified! New batch record modification +
[% ELSE %] - [% report.total_success %] / [% report.total_records %] records have successfully been modified. - Some errors occurred. +
+ [% report.total_success %] / [% report.total_records %] records have successfully been modified. Some errors occurred. New batch record modification +
[% END %] -

New batch record modification

[% ELSIF view == 'errors' %] [% FOR error IN errors %] [% IF error == 'no_template_defined' %] - No MARC modification template is defined. You have to create at least one template for using this tool. +
+ No MARC modification template is defined. You have to create at least one template for using this tool. +
[% END %] [% END %] [% ELSE %] - No action defined for the template. +
+ No action defined for the template. +
[% END %]
-- 2.1.4