[Freeipa-devel] Issues with ACI UI

[Adam Young]

On 12/21/2010 12:20 AM, Adam Young wrote:
> 1.  Can't add an ACI.  Before, I was able to get away with a blank 
> filter, but that doesn't seem to work anymore.
As a short term work around, I can do the object type as the default, 
and have the user switch it on edit, but that seems pretty 
counter-intuitive.  My original thought was to have something equivalent 
to a null filter that gets populated by default, without the user seeing 
it, and then, when they go to the edit page, they can either change the 
filter or change the overall type.  I need a syntactically valid null 
filter.



> 2.  Delegation-add :  the group-find for the combo boxes isn't getting 
> executed.
Note that if you clicked 'filter' and then start typing, the combo box 
does get executed, just not on initial form load. We don't currently 
have an event that signifies 'show form' which means that I have to 
populate them on initial creation.  This should be a two line fix.



> 3.  Some edits are broken for Permissions:  For certain, update dns 
> entries
The error I am getting here is that the aciattrs is missing. Since this 
permission is for the whole object, this makes sense.  I can 
shortcircuit the check there, which should be a one line fix.

> 4.  adding self service permission,  attrs is required, even if the 
> user just wants to do an 'add' permission.
> 5.  Modifying the self service permission just added gives an internal 
> error.  I removed the 'delete' and 'write' permission ( which I did 
> not set in the add dialog) as well as the 'audio' permission.  Log is 
> below:
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipaserver/rpcserver.py", line 211, 
> in wsgi_execute
> [Tue Dec 21 00:18:03 2010] [error]     result = 
> self.Command[name](*args, **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/frontend.py", line 417, in 
> __call__
> [Tue Dec 21 00:18:03 2010] [error]     ret = self.run(*args, **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/frontend.py", line 696, in run
> [Tue Dec 21 00:18:03 2010] [error]     return self.execute(*args, 
> **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/plugins/selfservice.py", line 
> 160, in execute
> [Tue Dec 21 00:18:03 2010] [error]     result = 
> api.Command['aci_mod'](aciname, **kw)['result']
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/frontend.py", line 417, in 
> __call__
> [Tue Dec 21 00:18:03 2010] [error]     ret = self.run(*args, **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/frontend.py", line 696, in run
> [Tue Dec 21 00:18:03 2010] [error]     return self.execute(*args, 
> **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/plugins/aci.py", line 550, in 
> execute
> [Tue Dec 21 00:18:03 2010] [error]     result = 
> self.api.Command['aci_add'](aciname, **newkw)['result']
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/frontend.py", line 417, in 
> __call__
> [Tue Dec 21 00:18:03 2010] [error]     ret = self.run(*args, **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/frontend.py", line 696, in run
> [Tue Dec 21 00:18:03 2010] [error]     return self.execute(*args, 
> **options)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/plugins/aci.py", line 450, in 
> execute
> [Tue Dec 21 00:18:03 2010] [error]     newaci_str = unicode(newaci)
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/aci.py", line 68, in __repr__
> [Tue Dec 21 00:18:03 2010] [error]     return self.export_to_string()
> [Tue Dec 21 00:18:03 2010] [error]   File 
> "/usr/lib/python2.6/site-packages/ipalib/aci.py", line 79, in 
> export_to_string
> [Tue Dec 21 00:18:03 2010] [error]     target = target + l + " || "
> [Tue Dec 21 00:18:03 2010] [error] TypeError: cannot concatenate 'str' 
> and 'NoneType' objects
>
>
> Some of these are on the UI side, and some are on the server side.  
> We'll need to sort out which is which.

Pretty sure these last two are server side issues.
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel