From 0d7661a77915f1184d0def0a0e81e4c4084f6a49 Mon Sep 17 00:00:00 2001 From: Mads Rasmussen Date: Fri, 7 Apr 2017 13:59:26 +0200 Subject: [PATCH] fixes: U4-9753 When uploading, moving or deleting folders in the media section you have to reload to see the changes --- .../listview/listview.controller.js | 16 ++++++++-------- .../views/propertyeditors/listview/listview.html | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.controller.js index a9eab75a04..f4068c0f02 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.controller.js @@ -54,8 +54,6 @@ function listViewController($rootScope, $scope, $routeParams, $injector, $cookie $scope.actionInProgress = false; $scope.selection = []; $scope.folders = []; - //tracks if we've already loaded the folders for the current node - var foldersLoaded = false; $scope.listViewResultSet = { totalPages: 0, items: [] @@ -217,7 +215,7 @@ function listViewController($rootScope, $scope, $routeParams, $injector, $cookie 500); if (reload === true) { - $scope.reloadView($scope.contentId); + $scope.reloadView($scope.contentId, true); } if (err.data && angular.isArray(err.data.notifications)) { @@ -252,7 +250,11 @@ function listViewController($rootScope, $scope, $routeParams, $injector, $cookie /*Pagination is done by an array of objects, due angularJS's funky way of monitoring state with simple values */ - $scope.reloadView = function (id) { + $scope.getContent = function() { + $scope.reloadView($scope.contentId, true); + } + + $scope.reloadView = function (id, reloadFolders) { $scope.viewLoaded = false; @@ -270,13 +272,12 @@ function listViewController($rootScope, $scope, $routeParams, $injector, $cookie }); } - if (!foldersLoaded && $scope.entityType === 'media') { + if (reloadFolders && $scope.entityType === 'media') { //The folders aren't loaded - we only need to do this once since we're never changing node ids mediaResource.getChildFolders($scope.contentId) .then(function (folders) { $scope.folders = folders; $scope.viewLoaded = true; - foldersLoaded = true; }); } else { @@ -319,7 +320,6 @@ function listViewController($rootScope, $scope, $routeParams, $injector, $cookie function makeSearch() { if ($scope.options.filter !== null && $scope.options.filter !== undefined) { $scope.options.pageNumber = 1; - //$scope.actionInProgress = true; $scope.reloadView($scope.contentId); } } @@ -621,7 +621,7 @@ function listViewController($rootScope, $scope, $routeParams, $injector, $cookie $scope.options.allowBulkMove || $scope.options.allowBulkDelete; - $scope.reloadView($scope.contentId); + $scope.reloadView($scope.contentId, true); } function getLocalizedKey(alias) { diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.html b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.html index 92436ee218..97500fd7c1 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.html +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/listview/listview.html @@ -157,7 +157,7 @@ selection="selection" options="options" entity-type="{{entityType}}" - on-get-content="reloadView"> + on-get-content="getContent">