There is an issue in patch #701, new version attached.
On 10.7.2014 14:29, Petr Vobornik wrote:
ticket: https://fedorahosted.org/freeipa/ticket/4253 == [PATCH] 697 webui: improve usability of attributes widget == Attributes widget layour was changed from tiny table which allowed to display only few options to a checkbox list with multiple columns (depends on container). Check all attributes option was removed to force the user to read through the attributes which he selects. Initial version authored by: Adam Misnyovszki == [PATCH] 698 webui: add filter to attributes widget == Adds filter field to attribute box in permissions for better user experience. User can then quickly find the desired attribute. Initial version of the patch authored by: Adam Misnyovszki == [PATCH] 699 webui: optimize (re)creation of option widget == There is a case where attributes widget can contain > 1000 items. It's about 3000 nodes. It's slow in jQuery. Simple move to dojo speeds it up (is closer to native calls) while maintaining developer friendliness. Now the biggest lag is in browser's render. It's probably not worth developer time to optimize that. == [PATCH] 700 webui: custom attr in attributes widget == Web UI doesn't always know what are the possible attributes for target object. This will allow to add custom attributes if necessary. == [PATCH] 701 webui: attr widget: get list of possible attrs from ipapermdefaultattr == Very useful for managed permissions since the list of attrs in metadata might be smaller that default attributes. This smooths behavior if one removes an attr from effective attrs which is not in metadata. Without this it will disappear from the list and one has to add it manually through 'Add'. [PATCH] 702 webui: option_widget_base: sort options
-- Petr Vobornik
From 8f2adbe422a24639f7d6bee83f7b198c15bf9497 Mon Sep 17 00:00:00 2001 From: Petr Vobornik <pvobo...@redhat.com> Date: Wed, 9 Jul 2014 16:49:46 +0200 Subject: [PATCH] webui: attr widget: get list of possible attrs from ipapermdefaultattr Very useful for managed permissions since the list of attrs in metadata might be smaller that default attributes. This smooths behavior if one removes an attr from effective attrs which is not in metadata. Without this it will disappear from the list and one has to add it manually through 'Add'. https://fedorahosted.org/freeipa/ticket/4253 --- install/ui/src/freeipa/aci.js | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/install/ui/src/freeipa/aci.js b/install/ui/src/freeipa/aci.js index 4cf93c9b15c5361c8162e17a4b8cb7b28c604490..d4eb0febef59c41e66f833f8b0d8464cbc97c3fc 100644 --- a/install/ui/src/freeipa/aci.js +++ b/install/ui/src/freeipa/aci.js @@ -898,6 +898,16 @@ aci.permission_target_policy = function (spec) { var attribute_table = that.permission_target.widgets.get_widget('attrs'); var skip_unmatched_org = attribute_table.skip_unmatched; attribute_table.object_type = type; + + // UI doesn't always know what are the possible attributes. + // In case of managed permissions, one of the possible lists is in ipapermdefaultattr. + var default_attrs = that.container.fields.get_field('ipapermdefaultattr'); + if (default_attrs && default_attrs.enabled) { // if managed permission + attribute_table.custom_options = default_attrs.get_value(); + } else { + attribute_table.custom_options = []; + } + // skip values which don't belong to new type. Bug #2617 attribute_table.skip_unmatched = skip_unmatched || skip_unmatched_org; attribute_field.reset(); -- 1.9.3
_______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
