[Freeipa-devel] [PATCH 0289] hosts: Display assigned ID view by default in host-find and show

[Tomas Babej]

On 12/02/2014 05:23 PM, Jan Cholasta wrote:
> Dne 2.12.2014 v 17:01 Tomas Babej napsal(a):
>>
>> On 12/02/2014 04:14 PM, Jan Cholasta wrote:
>>> Hi,
>>>
>>> Dne 2.12.2014 v 15:43 Tomas Babej napsal(a):
>>>> Hi,
>>>>
>>>> Makes ipaassignedidview a default attribute and takes care about the
>>>> conversion from the DN to the proper ID view name.
>>>>
>>>> https://fedorahosted.org/freeipa/ticket/4774
>>>
>>> Since you are converting the value from DN to primary key string, the
>>> type of the ipassignedview param should be changed to Str, for
>>> consistency with existing code.
>>>
>>> Honza
>>>
>>
>> I see. Actually during the development, I craved for simple
>> output_normalizer option in the Param itself, which would apply the
>> output normalization funtion after the post_callback,
>> instead of having to mangle the entry_attrs in the post callback of each
>> command (and could potentionally apply on the client). Would there a be
>> not-so-hard way to do this in the framework? My understanding is that
>> Output classes are quite decoupled from the Params they resulted from,
>> and at the point we're printing the information via the textui, we're no
>> longer aware what Param instance it originated from.
>
> This wouldn't solve the real problem in this case, which is that we do
> not support one-to-many relationships between objects in the framework
> (and our support for many-to-many relationships is clunky). I plan to
> fix this with <https://fedorahosted.org/freeipa/ticket/3932>.
>
>>
>> Updated patch attached.
>>
>
>

Updated patch with fixed WebUI bits.

-- 
Tomas Babej
Associate Software Engineer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | freeipa.org 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-tbabej-0289-3-hosts-Display-assigned-ID-view-by-default-in-host-fi.patch
Type: text/x-patch
Size: 7514 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20141204/f96a4688/attachment.bin>