Bug 40866: Remove unused $message variable from circulation

The $message variable was declared in circ/circulation.pl but never
actually set, making the "No patron matched" template block unreachable.

This is a remnant from old code that was incompletely cleaned when that
functionality was changed/removed.

This patch removes the unused variable, template parameter, and the
unreachable template block.

Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>

circ/circulation.pl

@@ -326,8 +326,6 @@ if ($patron) {
 # STEP 3 : ISSUING
 #
 #
-my $message;
-
 if ( @$barcodes && $op eq 'cud-checkout' ) {
     my $checkout_infos;
     for my $barcode (@$barcodes) {
@@ -784,7 +782,6 @@ $template->param(
     stickyduedate             => $stickyduedate,
     duedatespec               => $duedatespec,
     restoreduedatespec        => $restoreduedatespec,
-    message                   => $message,
     totaldue                  => sprintf( '%.2f', $balance ),                         # FIXME not used in template?
     inprocess                 => $inprocess,
     $view                     => 1,

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

@@ -787,11 +787,6 @@
         <p>Item checked out</p>
     [% END %]
 
-    [% IF ( message ) %]
-        [% INCLUDE 'patron-toolbar.inc' %]
-        <h4>No patron matched <span class="ex">[% message | html %]</span></h4>
-    [% END %]
-
     <!-- BARCODE ENTRY -->
 
     [% IF patron %]