Web staff: Patron bill payment annotation dialog/modal doesn't work

[Cesar V]

In the web staff Patron view, if you go to Bills for a patron with some outstanding debt, you can submit a payment on the right. The "Annotate" checkbox, when active, will pop up a prompt/dialog/modal that asks the user to enter the note text. However, this modal is not wired up to actually work and add a note to the submitted payment.

Further, the OK/Cancel buttons don't seem to work on this modal, and if the modals get changed to backdrop:static (i.e clicking outside them does not dismiss them) per bug 1689325, the only way to get out of the modal is hitting the ESC key, otherwise you're stuck in the modal... Forever...

Tested to occur on 3.0 and likely 2.12.

[Cesar V (cesardv)]

I have a potential fix for this here:

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

I am not sure how the payment annotation is working on webby, given tat this seems broken on master.
Regardless, I think the change to egPromptDialog are probably good to have.

[Terran McCanna (tmccanna)]

Confirmed that this is still broken on 3.0.1 as well. Will try to get Cesar's fix tested soon.

[Terran McCanna (tmccanna)]

We applied this to our test server running 3.0.1 and annotated payments worked, but then non-annotated payments didn't work.

[Cesar V (cesardv)]

Hi Terran, thanks for testing this.

I've pushed new commit to the same branch that adds a small change that checks to prevent the error I think you're seeing... (i.e undefined pay_note.value blowing up, when a payment is non-annotated)

Let me know if there's any further issues.

[Terran McCanna (tmccanna)]

Thanks Cesar, that does the trick!

I have tested this code and consent to signing off on it with my name, Terran McCanna and my email address, <email address hidden>.

[Galen Charlton (gmc)]

Pushed to master and rel_3_0. Thanks, Cesar and Terran!