Bug 10362: ensure that all messages and alerts are shown upon checkin
authorFridolyn SOMERS <fridolyn.somers@biblibre.com>
Tue, 28 May 2013 13:43:05 +0000 (15:43 +0200)
committerChris Hall <followingthepath@gmail.com>
Sun, 20 Oct 2013 06:52:20 +0000 (19:52 +1300)
If an item is returned and has mutiple error or warnings (e.g.,
it both needs to be transferred and the patron was debarred
in the course of the return), because of a bug in the template,
not all of the messages would be displayed.

This patch changes the template to show both messages and alerts.

Test plan :
- Perform an checkout that will a checkin :
  * need a transfert (return in a different branch)
  * set user debarred (depends on issuing rules)
- Perform check-in
=> You get tranfert message and debarment alert

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 5192fd81adaccf9b0f4a01dcc0ffc689edc9468f)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 8525d49e0601c85807235c007f712666c2abbceb)
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
(cherry picked from commit 8525d49e0601c85807235c007f712666c2abbceb)

koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt

index 7142cdc..6df23b9 100644 (file)
@@ -311,7 +311,7 @@ $(document).ready(function () {
         </form>
        </div>
     [% END %]
-[% ELSE %]
+[% END %]
 
 [% IF ( errmsgloop ) %]
     <div class="dialog alert">
@@ -347,7 +347,6 @@ $(document).ready(function () {
 [% END %]
         [% END %]
     </div>
-[% END %]
 
     <div id="exemptfines" class="dialog message" style="display:none;">
         <p>Fines for returned items are forgiven.</p>