Web Client: Adding bill without billing type fails silently

[Terran McCanna]

In 2.12.4 web client:

Adding a manual bill to a patron's account allows me to submit without selecting a billing type, but fails with no error message.

[Cesar V (cesardv)]

I was not able to reproduce the error as described, I tested on the latest master from the working repo.

In the webstaff client, when in a patron record's Bill section, manually adding a bill to a patron via Bill Patron button without selecting anything in the "BillingType" dropdown still created a bill for me and I noticed no errors. The billing defaulted to "misc."

[Kathy Lussier (klussier)]

Confirming that I see the same behavior described by Cesar on current master.

[Kathy Lussier (klussier)]

Having said that, it might not be a bad idea to force the user to select a billing type rather than falling back to misc.

[Cesar V (cesardv)]

I've pushed a branch that requires the user to select a billingType.

I think the reason it wasn't doing that before was because attempting validation there exposed an ugly angular wart/bug that affects value attribute conversions in ng generated <select> tag options. A work-around helper directive had to added to make it work. Kudos to gmcharlt for his help with figuring out that mess!
Now you'll see expected html5 form validation goodness.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/cesardv/lp1712646_make_patron_bill_modal_billingType_required

[Andrea Neiman (aneiman)]

Tested on MassLNC sandbox.

I have tested this code & consent to signing off on it with my name, Andrea Neiman, and my email address <email address hidden>

[Kathy Lussier (klussier)]

We discovered an issue with return parseInt(val, 10); in this patch while testing a patch for bug 1714390 on this Sandbox. See the IRC discussion at http://irc.evergreen-ils.org/evergreen/2017-09-15#i_324520

[Kathy Lussier (klussier)]

More specifically from the aforementioned blog, JBoyer noted the the branch 'either needs to change the directive name, switch over to the parseFloat function, or move that directive into the app controller that it's used in' to prevent the conflict we found with bug 1714390.

[Terran McCanna (tmccanna)]

Is someone currently working on completing this one?

[Jason Boyer (jboyer)]

Terran, I'm planning to look into it soon since there's another situation in the copy editor where a similar directive is needed I think the correct way to address this bug is to rework the patch from 1714390 and this bug into the one I'm planning to put together for bug 1713064.

[Terran McCanna (tmccanna)]

jboyer++

[Jason Boyer (jboyer)]

A patch is available here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jboyer/lp1712646-require-btype

NOTE! It requires the patch to bug 1741072 to work.

With both patches applied staff won't be able to add a billing without choosing a specific billing type.

[Bill Erickson (berick)]

Issue and fix confirmed. It now defaults to Misc (consistent w/ XUL) and complains if I try to clear the billing type value. Merged to rel_3_0 and master. Thanks, All!