Various TPac fixes

Bug #859823 reported by Thomas Berezansky
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned

Bug Description

Over the weekend I wrote up some Template Toolkit OPAC fixes:

I found some missing quotes for fine payment code

Made it so that you aren't prompted to pay fines in the OPAC if it is disabled

Added a default password strength rule that should match the others already in various places in the system

Fixed bolding of bad email/username on change attempts

And made OU dropdowns obey the opac_visible flag. Unless you are in the staff client.

And due to how I was working on them, they are all in one branch:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/tpac_fixes

Tags: pullrequest
Revision history for this message
Dan Scott (denials) wrote :

For "prevent pay fines in OPAC if disabled", it's good as far as it goes, but perhaps we should go further and make https://localhost/eg/opac/myopac/main_payment_form? return a blank page or 404 or something other than a payment form?

Revision history for this message
Dan Scott (denials) wrote :

I've tested and pushed the following commits to master:

  * missing quotes for fine payment code
  * do not prompt to pay fines in the OPAC if CC payments are disabled
  * fix bolding of bad email/username on change attempts
  * make OU dropdowns obey the opac_visible flag

I've left the "default password strength" out of this set of commits to master, as I'm concerned about propagating the default rule in hard-to-customize Perl module code rather than having the default rule come from the OU settings.

Revision history for this message
Dan Scott (denials) wrote :

I'm going to mark this as "Fix committed" as the bulk of the changes were indeed committed; please open a new bug specific to the "default password strength" change to focus on that issue.

Changed in evergreen:
status: New → Fix Committed
Changed in evergreen:
milestone: none → 2.2.0
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.