Merge pull request #1662 from mabashian/1555-permissions-checkboxes

Fixed permissions multi-select deselect bug
This commit is contained in:
Michael Abashian 2018-03-26 10:30:42 -04:00 committed by GitHub
commit e486b16706
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 5 deletions

View File

@ -82,6 +82,7 @@ export default ['$rootScope', '$scope', 'GetBasePath', 'Rest', '$q', 'Wait', 'Pr
scope.removeObject = function(obj){
let resourceType = scope.currentTab();
delete scope.allSelected[resourceType][obj.id];
obj.isSelected = false;
};
scope.toggleKeyPane = function() {

View File

@ -146,10 +146,6 @@ export default ['addPermissionsTeamsList', 'addPermissionsUsersList', 'TemplateL
scope[`${list.iterator}_dataset`] = scope.dataset.data;
scope[`${list.name}`] = scope[`${list.iterator}_dataset`].results;
scope.$watch(`allSelected.${list.name}`, function(){
_.forEach(scope[`${list.name}`], isSelected);
}, true);
scope.$watch(list.name, function(){
_.forEach(scope[`${list.name}`], isSelected);
optionsRequestDataProcessing();
@ -198,7 +194,6 @@ export default ['addPermissionsTeamsList', 'addPermissionsUsersList', 'TemplateL
}
function isSelected(item){
item.isSelected = false;
_.forEach(scope.allSelected[list.name], (selectedRow) => {
if(selectedRow.id === item.id) {
item.isSelected = true;