Angular patron editor post-2.10 misc. bugs

Bug #1564685 reported by Bill Erickson
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fix Released

Bug Description

Evergreen 2.10.1

Omnibus branch en-route to resolve the following angular patron editor issues:

1. Avoid warning on duplicate username when the username is owned by the patron being edited.

2. When a username regex is present, consider both regex-compliant usernames and an equal match on the barcode as valid usernames, even if the barcode is not regex-compliant. (It usually isn't).

3. Instead of disabling the Save buttons, allow the user to click Save or Save/Clone when invalid fields are present, but display an alert (and prevent save) instead. This is consistent w/ the XUL client and better clarifies why the action is not allowed.

4. Allow all addresses to be deleted on a user unless the org unit setting for requiring an address is true.

5. Misc. repairs to enforcing required fields. Of note, county and state specifically are required now if the matching org setting is true.

6. Consistent (small) sizes of checkboxes for boolean fields -- they done got real big.

7. Removing the last address should not hide the New Address button. It should always be visible.

8. Linked addresses should not be editable from the cloned account. Only the lead account.

9. Only attempt to save stat cat values for stat cats that are within scope.

Tags: pullrequest
Revision history for this message
Bill Erickson (berick) wrote :
tags: added: pullrequest
Changed in evergreen:
milestone: none → 2.10.2
Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
importance: Undecided → Medium
Revision history for this message
Bill Erickson (berick) wrote :

Pushed a fix to prevent linked/cloned address editing when loading a cloned user after cloning is done.

Revision history for this message
Kathy Lussier (klussier) wrote :

I know gmcharlt is looking at this branch, but I'm adding a quick note that the bug fixes work for me. I added my sign-off at:;a=shortlog;h=refs/heads/user/kmlussier/lp1564685-ang-patron-reg-post-2.10-misc-bugs

Thanks for your work on this Bill!


Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master and rel_2_10. Thanks Bill and Kathy!

Changed in evergreen:
status: New → Fix Committed
assignee: Galen Charlton (gmc) → nobody
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.