Bug 16125

Summary: Regression: Can't add messages at checkout if no other messages are present
Product: Koha Reporter: Owen Leonard <oleonard>
Component: CirculationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: brendan, gmcharlt, jonathan.druart, julian.maurice, kyle.m.hall, veron
Version: Main   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/circ/circulation.pl?borrowernumber=X
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 15632    
Bug Blocks:    
Attachments: Bug 16125: Remove the flagged boolean in circ/circulation.tt
Bug 16125: Remove the flagged boolean in circ/circulation.tt
[PASSED QA] Bug 16125: Remove the flagged boolean in circ/circulation.tt

Description Owen Leonard 2016-03-22 16:07:30 UTC
The "Add a new message" link is no longer visible on the checkout screen if no other messages are present.

A sidebar on the right shows patron messages, if any, including hold waiting notices, outstanding fines, messages on their account, etc.

Before Bug 15632 that sidebar displayed all the time, even if no messages were present. This wasn't ideal, but at least it created a space for the "Add new message" link to appear.

Now that the sidebar is gone if there are no messages we need a place for the "Add new message" link. I think coming up with a new place for it is better than making it work like it did before.
Comment 1 Jonathan Druart 2016-03-23 16:08:33 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-03-23 16:29:20 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2016-03-23 21:23:12 UTC
Created attachment 49463 [details] [review]
[PASSED QA] Bug 16125: Remove the flagged boolean in circ/circulation.tt

Before 15632, flagged was passed in any cases: GetMessages returned an
arrayref, so:
  my $bor_messages_loop = GetMessages( $borrowernumber, 'B', $branch );
  if($bor_messages_loop){ $template->param(flagged => 1 ); }
set 'flagged'.

It seems that this variable is not needed as it's always set.

Test plan:
Confirm that the "Add a new message" link is always displayed (even if
no message exist).
Note that a "Attention" is displayed but should not, that already
existed before bug 15632.

Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Brendan Gallagher 2016-03-24 15:39:34 UTC
Pushed to Master - Should be in the May 2016 release.  Thanks!
Comment 5 Julian Maurice 2016-04-07 07:39:47 UTC
Not needed in 3.22.x, status changed to RESOLVED