Change #6320
Category | None |
Changed by | Jane Sandberg <sandbej | @linnbenton.edu>
Changed at | Fri 19 Jul 2019 13:09:17 |
Repository | git://git.evergreen-ils.org/Evergreen.git |
Project | Evergreen |
Branch | master |
Revision | 9a4cb65a0656f5c871d8ecf254c198ea6544f79a |
Comments
LP1837067: Fixing unsubscribed Observable in Replace Barcodes dialog To test: 1) In the experimental Angular staff catalog, open up a bib record. 2) Open the Holdings View tab. 3) Select several items. 4) Use the Actions for Selected Rows menu to Replace Barcodes 5) Change the first barcode to something recognizable. Click the Replace Barcode button. 6) Note that the dialog opens again to ask you to replace the barcode you just added. 7) Apply this patch. 8) Repeat steps 1-5. 9) Note that the dialog opens again to ask you to replace the next barcode. Also removes some unused imports, and consolidates two RxJS pipes that were next to one another. Bill's Additions: Removes an unnecessary throwError which was called every time the dialog completed. NOTE: the source of the bug was that the Observable returned by pcrud.search(...) was never subscribed-to, so it never fired. Calling .toPromise() forces the observable to execute. Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu> Signed-off-by: Bill Erickson <berickxx@gmail.com>
Changed files
- Open-ILS/src/eg2/src/app/staff/share/holdings/replace-barcode-dialog.component.ts