Re: [Freeipa-devel] [PATCH] 1 Do lazy initializiation ipalib

2011-10-31 Thread Jan Cholasta
Dne 26.10.2011 16:50, Jan Cholasta napsal(a): Dne 26.10.2011 16:39, Martin Kosek napsal(a): On Wed, 2011-10-26 at 15:52 +0200, Jan Cholasta wrote: Dne 26.10.2011 15:41, Martin Kosek napsal(a): On Wed, 2011-10-26 at 11:39 +0200, Jan Cholasta wrote: Dne 25.10.2011 22:30, Rob Crittenden napsal(a

Re: [Freeipa-devel] [PATCH] 1 Do lazy initializiation ipalib

2011-10-31 Thread Alexander Bokovoy
On Mon, 31 Oct 2011, Jan Cholasta wrote: > Added finalization for __call__ and the check for CLI. Patch attached. ACK from my side but see below. > +def __getattribute__(self, name): > +if not name.startswith('_Plugin__') and not > name.startswith('_ReadOnly__') and name != 'finalize_

Re: [Freeipa-devel] [PATCH] 1 Do lazy initializiation ipalib

2011-10-31 Thread Simo Sorce
On Mon, 2011-10-31 at 14:19 +0200, Alexander Bokovoy wrote: > On Mon, 31 Oct 2011, Jan Cholasta wrote: > > Added finalization for __call__ and the check for CLI. Patch attached. > ACK from my side but see below. > > > +def __getattribute__(self, name): > > +if not name.startswith('_Plu

Re: [Freeipa-devel] [PATCH] 1 Do lazy initializiation ipalib

2011-10-31 Thread Alexander Bokovoy
On Mon, 31 Oct 2011, Simo Sorce wrote: > On Mon, 2011-10-31 at 14:19 +0200, Alexander Bokovoy wrote: > > On Mon, 31 Oct 2011, Jan Cholasta wrote: > > > Added finalization for __call__ and the check for CLI. Patch attached. > > ACK from my side but see below. > > > > > +def __getattribute__(sel

Re: [Freeipa-devel] Extending the IPA-API

2011-10-31 Thread Adam Young
On 10/27/2011 08:40 PM, Endi Sukma Dewata wrote: On 10/27/2011 10:59 AM, Adam Young wrote: The web UI can implement a similar mechanism. We do not want end sites modifying the .js files shipped with the IPA server RPM, other wise, they could inject columns and fields there, but they would be sus

Re: [Freeipa-devel] [PATCH] 1 Do lazy initializiation ipalib

2011-10-31 Thread Jan Cholasta
Dne 31.10.2011 13:19, Alexander Bokovoy napsal(a): On Mon, 31 Oct 2011, Jan Cholasta wrote: Added finalization for __call__ and the check for CLI. Patch attached. ACK from my side but see below. +def __getattribute__(self, name): +if not name.startswith('_Plugin__') and not name.

Re: [Freeipa-devel] [PATCH] 029 Page is cleared before it is visible

2011-10-31 Thread Endi Sukma Dewata
On 10/27/2011 4:57 AM, Petr Vobornik wrote: https://fedorahosted.org/freeipa/ticket/1459 Changes: * added clear method to widgets, section, search, details, association facets * clear method in facet is called only if key/filter was changed Some issues: 1. The new clear() method is called dur