[Kimchi-devel] [PATCH 2/4] Authorization: remove host/template tabs for non-root users

[Wen Wang]

On 07/19/2014 01:40 AM, Aline Manera wrote:
>
> On 07/18/2014 08:18 AM, wenwang at linux.vnet.ibm.com wrote:
>> From: Wen Wang <wenwang at linux.vnet.ibm.com>
>>
>> This patch removed host/template tabs from non-root users
>>
>> Signed-off-by: Wen Wang <wenwang at linux.vnet.ibm.com>
>> ---
>>   ui/js/src/kimchi.main.js |   21 +++++++++++++--------
>>   1 files changed, 13 insertions(+), 8 deletions(-)
>>
>> diff --git a/ui/js/src/kimchi.main.js b/ui/js/src/kimchi.main.js
>> index 8eb4d73..54a0043 100644
>> --- a/ui/js/src/kimchi.main.js
>> +++ b/ui/js/src/kimchi.main.js
>> @@ -23,13 +23,16 @@ kimchi.main = function() {
>>           $(tabs).each(function(i, tab) {
>>               var title = tab['title'];
>>               var path = tab['path'];
>> -            tabsHtml.push(
>> -                '<li>',
>> -                    '<a class="item" href="', path, '">',
>> -                        title,
>> -                    '</a>',
>> -                '</li>'
>> -            );
>> +            var mode = tab['mode'];
>> +            if (mode != 'none') {
>> +                tabsHtml.push(
>> +                    '<li>',
>> +                        '<a class="item" href="', path, '">',
>> +                            title,
>> +                        '</a>',
>> +                    '</li>'
>> +                );
>> +            }
>>           });
>>           return tabsHtml.join('');
>>       };
>> @@ -41,9 +44,11 @@ kimchi.main = function() {
>>               var titleKey = $tab.find('title').text();
>>               var title = i18n[titleKey] ? i18n[titleKey] : titleKey;
>>               var path = $tab.find('path').text();
>
>> +            var mode = kimchi.cookie.get(titleKey + 'Mode');
>
> While loading the tabs.xml and tab-ext.xml you need to get the mode 
> attribute according to role access element
> For example:
>
> var role = kimchi.cookie.get(titlekey + 'Role')
>
> var mode = $tab.find(["role='" + role + "']').attrib("mode")
ACK
>
>
>>               tabs.push({
>>                   title: title,
>> -                path: path
>> +                path: path,
>> +                mode: mode
>>               });
>>           });
>>
>