From b955a2d7af5906979363ba930868cf69fed8810d Mon Sep 17 00:00:00 2001 From: valepu Date: Wed, 6 May 2015 14:16:53 +0200 Subject: [PATCH] fix(clearThisRowInvisible) not using parameter reason Not sure if intended but clearThisRowInvisible is not using the parameter "reason" when deleting the "invisible reason" but it always deletes the hardcoded parameter "user". This shouldn't change any other behavior since ng-grid internal functions call this function only with the parameter "user" but it will fix any error if someone was directly using this function with a custom invisible reason --- src/js/core/factories/GridRow.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/js/core/factories/GridRow.js b/src/js/core/factories/GridRow.js index 3e14e0c80e..579bcef58c 100644 --- a/src/js/core/factories/GridRow.js +++ b/src/js/core/factories/GridRow.js @@ -176,7 +176,7 @@ angular.module('ui.grid') */ GridRow.prototype.clearThisRowInvisible = function ( reason, fromRowsProcessor ) { if (typeof(this.invisibleReason) !== 'undefined' ) { - delete this.invisibleReason.user; + delete this.invisibleReason[reason]; } this.evaluateRowVisibility( fromRowsProcessor ); }; @@ -216,4 +216,4 @@ angular.module('ui.grid') return GridRow; }]); -})(); \ No newline at end of file +})();