浏览代码

[security] Fix privileges refresh and save from browse section

Enrico Berti 11 年之前
父节点
当前提交
9e5f125
共有 2 个文件被更改,包括 4 次插入3 次删除
  1. 3 2
      apps/security/src/security/templates/hive.mako
  2. 1 1
      apps/security/static/js/hive.ko.js

+ 3 - 2
apps/security/src/security/templates/hive.mako

@@ -35,8 +35,8 @@ ${ layout.menubar(section='hive') }
   <div class="acl-block acl-actions">
     <span class="pointer" data-bind="visible: privilegesForViewTo() < privileges().length, click: function(){ privilegesForViewTo(privilegesForViewTo() + 50) }" title="${ _('Show 50 more...') }"><i class="fa fa-ellipsis-h"></i></span>
     <span class="pointer" data-bind="click: addPrivilege" title="${ _('Add privilege') }"><i class="fa fa-plus"></i></span>
-    <span class="pointer" data-bind="click: $root.list_sentry_privileges_by_authorizable, visible: privilegesChanged().length > 0" title="${ _('Undo') }"> &nbsp; <i class="fa fa-undo"></i></span>
-    <span class="pointer" data-bind="click: $root.role.savePrivileges, visible: privilegesChanged().length > 0" title="${ _('Save') }"> &nbsp; <i class="fa fa-save"></i></span>
+    <span class="pointer" data-bind="click: function() { $root.list_sentry_privileges_by_authorizable() }, visible: privilegesChanged().length > 0" title="${ _('Undo') }"> &nbsp; <i class="fa fa-undo"></i></span>
+    <span class="pointer" data-bind="click: $root.role().savePrivileges, visible: privilegesChanged().length > 0" title="${ _('Save') }"> &nbsp; <i class="fa fa-save"></i></span>
   </div>
 </script>
 
@@ -259,6 +259,7 @@ ${ layout.menubar(section='hive') }
                   </a>
                   <div data-bind="visible: showEditGroups() || groupsChanged()">
                     <select data-bind="options: $root.selectableHadoopGroups, selectedOptions: groups, select2: { update: groups, type: 'group'}" size="5" multiple="true" style="width: 400px"></select>
+                    &nbsp;
                     <a class="pointer" data-bind="visible: groupsChanged, click: resetGroups">
                       <i class="fa fa-undo"></i>
                     </a>

+ 1 - 1
apps/security/static/js/hive.ko.js

@@ -227,7 +227,7 @@ var Role = function (vm, role) {
         var role = new Role(vm, data.role);
         vm.roles.unshift(role);
         vm.assist.refreshTree();
-        vm.list_sentry_privileges_by_role(role); // Show privileges        
+        vm.list_sentry_privileges_by_role(role); // Show privileges
       } else {
         $(document).trigger("error", data.message);
       }