Angular PO: PO status does not automatically refresh on activation

[Tiffany Little]

Breaking out Mary Llewellyn's comment #6 from the original Sprint 4 bug report: bug 1942220

"After activating a purchase order, it does not automatically refresh to change the line item statuses and color. I've had to manually refresh the browser to see the change."

[Tiffany Little (tslittle)]

Marking this as Confirmed since Mary was the original reporter, and I can confirm I see the same behavior.

[Tiffany Little (tslittle)]

Fix here: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=0aac982ccaa51d5a321fcfabf55494f6b4f98eb2

[Tiffany Little (tslittle)]

I don't know if I was just delirious on a Friday afternoon when I originally posted my branch, but upon more testing it doesn't actually do the thing. I added a second commit that appears to tie it together. So it's the top two commits here:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tlittle/LP1991801_refreshafteractivation

[Tiffany Little (tslittle)]

We applied this to our production server, and it works great at refreshing the page when the PO is activated. However the byproduct is if you attempt to go to a PO with the focus on a line item (like clicking on Line item ID in Search, or clicking the link for the PO on an invoice line item) the page attempts to load over and over again. The PO header loads, but it constantly refreshes.

So I'm removing the pullrequest for right now so I can see if I can fix it. Input's definitely welcome if anyone has ideas.

[Tiffany Little (tslittle)]

For what it's worth, this bug was more troublesome for us than whether or not links focused on a lineitem. So with the fix of bug 2003946 in 3.10.2+ and also locally changing the PO link in invoices to go to the PO instead of the focused line item, we've been using this since January and have had no complaints.

I'm happy to add a commit for the invoice link and fully close the circle if others don't care about it focusing on the LI. Otherwise I'll just say that I struck out and leave my existing branch as a starter in case it's helpful for others.