The following list pages were modified to show these columns only:
 * HBAC rules: name, type, enabled, description
 * Sudo rules: name, enabled, description

Ticket #1796

--
Endi S. Dewata
From de81aeff5c86471d7173dda98bc12e5f0d4efd05 Mon Sep 17 00:00:00 2001
From: Endi S. Dewata <edew...@redhat.com>
Date: Fri, 16 Sep 2011 10:32:28 -0500
Subject: [PATCH] Fixed columns in HBAC/sudo rules list pages.

The following list pages were modified to show these columns only:
 * HBAC rules: name, type, enabled, description
 * Sudo rules: name, enabled, description

Ticket #1796
---
 install/ui/hbac.js |   53 +++++++++++++++++++++++++++++----------------------
 install/ui/sudo.js |   26 +++++++++++++++++-------
 2 files changed, 48 insertions(+), 31 deletions(-)

diff --git a/install/ui/hbac.js b/install/ui/hbac.js
index e2873a78789dbc66405398a2164bc6142dc4cf8f..bcc7c135d3ebec77b6d4644b98355940e5bf6303 100644
--- a/install/ui/hbac.js
+++ b/install/ui/hbac.js
@@ -27,29 +27,31 @@ IPA.entity_factories.hbacrule = function() {
     return IPA.entity_builder().
         entity('hbacrule').
         search_facet({
-            search_all:true,
-            columns:['cn',
-                     {
-                         factory: IPA.column,
-                         name:'accessruletype',
-                         setup : function(container,record){
-                             container.empty();
-                             var value = record[this.name];
-                             value = value ? value.toString() : '';
-                             if (value === 'deny'){
-                                 container.addClass('hbac-deny-rule');
-                             }
-                             container.append(value);
-                         }
-                     },
-                     'usercategory','hostcategory','ipaenabledflag',
-                     'servicecategory','sourcehostcategory']
+            search_all: true,
+            columns: [
+                'cn',
+                {
+                    factory: IPA.column,
+                    name: 'accessruletype',
+                    setup: function(container, record) {
+                        container.empty();
+                        var value = record[this.name];
+                        value = value ? value.toString() : '';
+                        if (value === 'deny') {
+                            container.addClass('hbac-deny-rule');
+                        }
+                        container.append(value);
+                    }
+                },
+                'ipaenabledflag',
+                'description'
+            ]
         }).
         details_facet({
             factory: IPA.hbacrule_details_facet
         }).
         adder_dialog({
-            fields:['cn']
+            fields: [ 'cn' ]
         }).
         build();
 };
@@ -143,19 +145,24 @@ IPA.hbacrule_details_facet = function(spec) {
     function general_section(){
         var section = IPA.details_table_section({
             name: 'general',
-            entity:that.entity,
+            entity: that.entity,
             label: IPA.messages.details.general
         });
 
-        section.text({name: 'cn', read_only: true});
+        section.text({
+            name: 'cn',
+            read_only: true
+        });
         section.text({
             name: 'accessruletype',
-            read_only:true
+            read_only: true
+        });
+        section.textarea({
+            name: 'description'
         });
-        section.textarea({name: 'description'});
         section.radio({
             name: 'ipaenabledflag',
-            options:[
+            options: [
                 { value: 'TRUE', label: IPA.get_message('true') },
                 { value: 'FALSE', label: IPA.get_message('false') }
             ]
diff --git a/install/ui/sudo.js b/install/ui/sudo.js
index 6b17e89add4b4546dd27ed8ffb8ce4704beeeab8..c9e7cdaf1104541584612bd38bc1fa2cad27d5fa 100644
--- a/install/ui/sudo.js
+++ b/install/ui/sudo.js
@@ -28,13 +28,17 @@ IPA.entity_factories.sudorule = function() {
     return IPA.entity_builder().
         entity('sudorule').
         search_facet({
-            columns:['cn','description','cmdcategory']
+            columns: [
+                'cn',
+                'ipaenabledflag',
+                'description'
+            ]
         }).
         details_facet({
             factory: IPA.sudorule_details_facet
         }).
         adder_dialog({
-            fields:['cn']
+            fields: [ 'cn' ]
         }).
         build();
 };
@@ -140,14 +144,20 @@ IPA.sudorule_details_facet = function(spec) {
             facet: that
         });
 
-        section.text({name: 'cn', read_only: true});
-        section.textarea({name: 'description'});
+        section.text({
+            name: 'cn',
+            read_only: true
+        });
+        section.textarea({
+            name: 'description'
+        });
         section.radio({
             name: 'ipaenabledflag',
-            options:[
-                {'value': 'TRUE',label: IPA.messages['true']},
-                {'value': 'FALSE',label:IPA.messages['false']}
-            ]});
+            options: [
+                { value: 'TRUE', label: IPA.get_message('true') },
+                { value: 'FALSE', label: IPA.get_message('false') }
+            ]
+        });
         return section;
     }
 
-- 
1.7.5.1

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to