Re: [Freeipa-devel] [PATCH] jderose 027 Extensible return values

2009-12-10 Thread Jason Gerard DeRose
On Wed, 2009-12-09 at 23:08 -0500, Rob Crittenden wrote: > Jason Gerard DeRose wrote: > > Okay, here's a revised patch. > > > > Significant additions/changes from the previous version are: > > > > 1. The return value dict now includes a 'summary' value, something like > > 'Added user "jdoe"'. Th

Re: [Freeipa-devel] [PATCH] jderose 027 Extensible return values

2009-12-09 Thread Rob Crittenden
Jason Gerard DeRose wrote: Okay, here's a revised patch. Significant additions/changes from the previous version are: 1. The return value dict now includes a 'summary' value, something like 'Added user "jdoe"'. This summary is used by the CLI and webUI. Previously I was generating the summary

Re: [Freeipa-devel] [PATCH] jderose 027 Extensible return values

2009-12-01 Thread Pavel Zůna
Rob Crittenden wrote: Jason Gerard DeRose wrote: On Wed, 2009-11-25 at 12:05 -0500, Rob Crittenden wrote: This is purely from reading the patch, I haven't applied and tested it yet. ipalib/output.py: +primary_key = Output('primary_key', unicode, +'The primary key of the deleted entry' +)

Re: [Freeipa-devel] [PATCH] jderose 027 Extensible return values

2009-11-25 Thread Rob Crittenden
Jason Gerard DeRose wrote: On Wed, 2009-11-25 at 12:05 -0500, Rob Crittenden wrote: This is purely from reading the patch, I haven't applied and tested it yet. ipalib/output.py: +primary_key = Output('primary_key', unicode, +'The primary key of the deleted entry' +) This isn't only for del

Re: [Freeipa-devel] [PATCH] jderose 027 Extensible return values

2009-11-25 Thread Jason Gerard DeRose
On Wed, 2009-11-25 at 12:05 -0500, Rob Crittenden wrote: > > This is purely from reading the patch, I haven't applied and tested it yet. > > ipalib/output.py: > +primary_key = Output('primary_key', unicode, > +'The primary key of the deleted entry' > +) > > This isn't only for deleted entrie