Skip to content

Commit

Permalink
Merge pull request #13 from SimplGy/personal/bcherny/fix-asyncPostRen…
Browse files Browse the repository at this point in the history
…derers

catch errors thrown in asyncPostRenderers, so we can continue applying other asyncPostRenderers
  • Loading branch information
bcherny committed May 4, 2015
2 parents f86e02b + 91e93ad commit 69a8a75
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bower.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "slickgrid",
"version": "2.3.9",
"version": "2.3.10",
"homepage": "https://github.com/SimpleAsCouldBe/SlickGrid",
"authors": [
"Michael Leibman",
Expand Down
6 changes: 5 additions & 1 deletion slick.grid.js
Original file line number Diff line number Diff line change
Expand Up @@ -2653,7 +2653,11 @@ if (typeof Slick === "undefined") {
if (m.asyncPostRender && !postProcessedRows[row][columnIdx]) {
var node = cacheEntry.cellNodesByColumnIdx[columnIdx];
if (node) {
m.asyncPostRender(node, row, getDataItem(row), m);
try {
m.asyncPostRender(node, row, getDataItem(row), m);
} catch (error) {
console.error('Error in asyncPostRenderer:', error, [node, row, getDataItem(row), m]);
}
}
postProcessedRows[row][columnIdx] = true;
}
Expand Down

0 comments on commit 69a8a75

Please sign in to comment.