Fixed blank krbtpolicy and config pages.

The details page compares the old and the new primary keys to determine
if the page needs to be reloaded. The Kerberos Ticket Policy and Config
pages do not use primary keys, so they are never loaded/updated with
data. A parameter has been added to force update on these pages.

Ticket #1459
This commit is contained in:
Endi S. Dewata 2011-11-03 16:02:32 -05:00 committed by Petr Vobornik
parent f7938a1773
commit 6c3e1a21d7
5 changed files with 22 additions and 10 deletions

View File

@ -1206,6 +1206,7 @@ IPA.association_facet = function (spec) {
};
that.needs_update = function() {
if (that._needs_update !== undefined) return that._needs_update;
var pkey = IPA.nav.get_state(that.entity.name+'-pkey');
return that.pkey !== pkey;
};

View File

@ -528,6 +528,7 @@ IPA.details_facet = function(spec) {
};
that.needs_update = function() {
if (that._needs_update !== undefined) return that._needs_update;
var pkey = IPA.nav.get_state(that.entity.name+'-pkey');
return pkey !== that.pkey;
};

View File

@ -44,6 +44,7 @@ IPA.facet = function (spec) {
that.header = spec.header || IPA.facet_header({ facet: that });
that.entity_name = spec.entity_name;
that._needs_update = spec.needs_update;
that.dialogs = $.ordered_map();
@ -113,6 +114,7 @@ IPA.facet = function (spec) {
};
that.needs_update = function() {
if (that._needs_update !== undefined) return that._needs_update;
return true;
};

View File

@ -76,9 +76,16 @@ IPA.entity_factories.krbtpolicy = function() {
entity('krbtpolicy').
details_facet({
title: IPA.metadata.objects.krbtpolicy.label,
sections:[{
name: 'identity',
fields:[ 'krbmaxrenewableage','krbmaxticketlife' ]
}]}).
sections: [
{
name: 'identity',
fields: [
'krbmaxrenewableage',
'krbmaxticketlife'
]
}
],
needs_update: true
}).
build();
};

View File

@ -32,12 +32,11 @@ IPA.entity_factories.config = function(){
entity('config').
details_facet({
title: IPA.metadata.objects.config.label,
sections:
[
sections: [
{
name: 'search',
label: IPA.messages.objects.config.search,
fields:[
fields: [
'ipasearchrecordslimit',
'ipasearchtimelimit'
]
@ -45,7 +44,7 @@ IPA.entity_factories.config = function(){
{
name: 'user',
label: IPA.messages.objects.config.user,
fields:[
fields: [
'ipausersearchfields',
'ipadefaultemaildomain',
{
@ -71,7 +70,7 @@ IPA.entity_factories.config = function(){
{
name: 'group',
label: IPA.messages.objects.config.group,
fields:[
fields: [
'ipagroupsearchfields',
{
factory: IPA.multivalued_text_widget,
@ -79,6 +78,8 @@ IPA.entity_factories.config = function(){
}
]
}
]}).
],
needs_update: true
}).
build();
};