From 8183a9f032af097f91c60ca18c03dc0bc0919854 Mon Sep 17 00:00:00 2001 From: Ferdy Date: Mon, 10 Jul 2017 16:07:43 +0200 Subject: [PATCH] $scope.model.value can be null when used in Grid editor, causing a for loop to fail. --- src/RJP.MultiUrlPicker.Web.UI/MultiUrlPicker.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/RJP.MultiUrlPicker.Web.UI/MultiUrlPicker.js b/src/RJP.MultiUrlPicker.Web.UI/MultiUrlPicker.js index aa34f09..ed88618 100644 --- a/src/RJP.MultiUrlPicker.Web.UI/MultiUrlPicker.js +++ b/src/RJP.MultiUrlPicker.Web.UI/MultiUrlPicker.js @@ -13,10 +13,12 @@ $scope.model.value = [] } - $scope.model.value.forEach(function (link) { - link.icon = iconHelper.convertFromLegacyIcon(link.icon) - this.renderModel.push(link) - }.bind(this)) + if ($scope.model.value) { + $scope.model.value.forEach(function (link) { + link.icon = iconHelper.convertFromLegacyIcon(link.icon) + this.renderModel.push(link) + }.bind(this)) + } $scope.$on('formSubmitting', function () { $scope.model.value = this.renderModel