LP1766982 - CSV downloads contain the string 'null' for null values. user/stompro/LP1766982_signoff
authorgcollum <gcollum@gmail.com>
Fri, 18 May 2018 23:57:42 +0000 (19:57 -0400)
committerJosh Stompro <stompro@stompro.org>
Mon, 11 Mar 2019 01:10:10 +0000 (20:10 -0500)
commit000c6dc69ebfd03b5ca72386caa5e9494b593a44
tree495ce2c6f8d2887ed4029d5af8ec692d69869bc0
parent1c061dd30a02a03bfe9d8043057b28e87aa910b2
LP1766982 - CSV downloads contain the string 'null' for null values.

Adds a check for null and undefined in the getItemTextContent function in grid.js which is called by both the code to download grid data as CSV and for printing grids.

To test
1. Place holds on a couple of titles, one of which has no author. In the test data TCN 226, The Crow, has no author data.
2. Pull up the Holds Pull List.
3. Download Full CSV - to download the CSV file and view it in a spreadsheet or text editor looking specifically at the Author column.
4. Print Full Grid - to view the data in a printout, since this function also affects printing.
5. Go back to the Holds Pull List and click on one of the barcodes under the Current Copy column.
6. Click on the List View.
7. Download Full CSV - open the resulting file to inspect the Alert Message data.
8. Apply patch and repeat steps 2-7.

Signed-off-by: gcollum <gcollum@gmail.com>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Open-ILS/web/js/ui/default/staff/services/grid.js