Bug 16097: Making messages for subscription fields more user friendly
authorAleisha <aleishaamohia@hotmail.com>
Thu, 17 Mar 2016 23:15:40 +0000 (23:15 +0000)
committerBrendan Gallagher <brendan@bywatersolutions.com>
Wed, 23 Mar 2016 00:45:42 +0000 (00:45 +0000)
To test:
1) Go to Serials -> Add Subscription fields
2) Create a field, confirm message.
3) Attempt to create a new field that has the same name as an existing
   field, confirm message.
4) Create another field with a different name. Attempt to edit this
   field to have the same name as another field. Confirm message
5) Edit field (change name to something different)
6) Delete field, confirm message.
6) Turn off permissions to delete subcriptions ONLY. Attempt to delete a
   field, confirm message.

Sponsored-by: Catalyst IT
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com
koha-tmpl/intranet-tmpl/prog/en/modules/serials/add_fields.tt

index 18c424d..b5fdad4 100644 (file)
     [% FOR message IN messages %]
       [% IF message.code == 'insert' %]
         [% IF message.number > 0 %]
-          <div class="dialog message">The field has been inserted</div>
+          <div class="dialog message">Field created.</div>
         [% ELSE %]
-          <div class="dialog alert">The field has not been inserted (name still exist?)</div>
+          <div class="dialog alert">The field could not be created. Perhaps the name already exists?</div>
         [% END %]
       [% ELSIF message.code == 'update' %]
         [% IF message.number > 0 %]
-          <div class="dialog message">The field has been updated</div>
+          <div class="dialog message">Field updated.</div>
         [% ELSE %]
-          <div class="dialog alert">The field has not been updated (name still exist?)</div>
+          <div class="dialog alert">The field could not be updated. Perhaps the name already exists?</div>
         [% END %]
       [% ELSIF message.code == 'delete' %]
         [% IF message.number > 0 %]
-          <div class="dialog message">The field has been deleted</div>
+          <div class="dialog message">Field deleted.</div>
         [% ELSE %]
-          <div class="dialog alert">The field has not been deleted</div>
+          <div class="dialog alert">The field could not be deleted. Check the log for errors.</div>
         [% END %]
       [% END %]
     [% END %]