http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6739

--- Comment #49 from Kyle M Hall <[email protected]> ---
Comment on attachment 17909
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17909
Bug 6739 - expired patrons not blocked from opac actions - Followup 2

Review of attachment 17909:
 --> 
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=6739&attachment=17909)
-----------------------------------------------------------------

Most of the changes in opac-user.tt are whitespace cleanup. I've marked out the
actual changes.

::: koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt
@@ +70,3 @@
>              });
> +
> +            [% IF ( canrenew && !userdebarred && OpacRenewalAllowed && !( 
> borrower.is_expired && borrower.BlockExpiredPatronOpacActions ) ) %]

This line fixes the renewal links bug.

@@ +132,4 @@
>                  <string>Please note:</string>
>                  <span>
>                      Your account renewal failed because of the following:
> +                    [% FOREACH error IN RENEW_ERROR.split('\|') %]

The pipe character needed escaping, otherwise it was split into an array of
single characters.

@@ +205,5 @@
> +                        [% END %]
> +
> +                        <th>Call No.</th>
> +
> +                        [% IF OpacRenewalAllowed && !( borrower.is_expired 
> && borrower.BlockExpiredPatronOpacActions  ) %]

Here is another actual change.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to