From: Mike Rylander <mrylander@gmail.com>
Date: Mon, 28 Sep 2015 16:41:13 +0000 (-0400)
Subject: Grid saveConfig can generate bad data (not sure how) but we will avoid duplicates... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2af294070f2613058cb7e445536247640e2d9d55;p=working%2FEvergreen.git

Grid saveConfig can generate bad data (not sure how) but we will avoid duplicates in loadConfig

Signed-off-by: Mike Rylander <mrylander@gmail.com>
---

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 66a2bf3908..4e48f93972 100644
--- a/Open-ILS/web/js/ui/default/staff/services/grid.js
+++ b/Open-ILS/web/js/ui/default/staff/services/grid.js
@@ -450,7 +450,12 @@ angular.module('egGridMod',
                         grid_col.sort = col.sort || 0;
                         // all saved columns are assumed to be true
                         grid_col.visible = true;
-                        new_cols.push(grid_col);
+                        if (new_cols
+                                .filter(function (c) {
+                                    return c.name == grid_col.name;
+                                }).length == 0
+                        )
+                            new_cols.push(grid_col);
                     });
 
                     // columns which are not expressed within the saved