From a8f87d6dc1ea97ecbe95a1e27c791da96090b3ca Mon Sep 17 00:00:00 2001 From: Shannon Date: Tue, 1 Oct 2013 14:07:56 +1000 Subject: [PATCH] Updates mediapicker to not use jquery each methods since they are not DOM items, use underscore instead. --- .../propertyeditors/mediapicker/mediapicker.controller.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/mediapicker/mediapicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/mediapicker/mediapicker.controller.js index c028217f71..e41961fff9 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/mediapicker/mediapicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/mediapicker/mediapicker.controller.js @@ -12,7 +12,7 @@ angular.module('umbraco').controller("Umbraco.Editors.MediaPickerController", mediaResource.getByIds($scope.ids).then(function(medias){ //img.media = media; - $(medias).each(function(i, media){ + _.each(medias, function (media, i) { //shortcuts //TODO, do something better then this for searching var img = {}; @@ -31,7 +31,7 @@ angular.module('umbraco').controller("Umbraco.Editors.MediaPickerController", $scope.add = function(){ dialogService.mediaPicker({multipicker:true, callback: function(data){ - $(data.selection).each(function(i, media){ + _.each(data.selection, function (media, i) { //shortcuts //TODO, do something better then this for searching @@ -50,5 +50,7 @@ angular.module('umbraco').controller("Umbraco.Editors.MediaPickerController", $scope.sync = function(){ $scope.model.value = $scope.ids.join(); }; + + //TODO: Need to add the onValueChanged method to detect if the server has changed the model value and re-create the ids }); \ No newline at end of file