From 8dc83f3003b010f5ef0e30f48f876c41cb4baac3 Mon Sep 17 00:00:00 2001 From: gcollum Date: Thu, 10 May 2018 21:33:36 -0400 Subject: [PATCH] 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 --- Open-ILS/web/js/ui/default/staff/services/grid.js | 1 + 1 file changed, 1 insertion(+) diff --git a/Open-ILS/web/js/ui/default/staff/services/grid.js b/Open-ILS/web/js/ui/default/staff/services/grid.js index fc3a05b..b6cbf9c 100644 --- a/Open-ILS/web/js/ui/default/staff/services/grid.js +++ b/Open-ILS/web/js/ui/default/staff/services/grid.js @@ -1035,6 +1035,7 @@ angular.module('egGridMod', } } else { val = grid.dataProvider.itemFieldValue(item, col); + if (val === null || val === undefined || val === '') return ''; val = $filter('egGridValueFilter')(val, col, item); } return val; -- 2.7.4