Please Review: Ticket 48360 - Refactor the delete agreement function

[Simon Pichugin]

Hi team,

please, review my patch for lib389 repo.

https://fedorahosted.org/389/ticket/48360

https://fedorahosted.org/389/attachment/ticket/48360/0001-Ticket-48360-Refac...

Thank you,
Simon

[Simon Pichugin]

Revision with new feature.

https://fedorahosted.org/389/ticket/48360

https://fedorahosted.org/389/attachment/ticket/48360/0001-Ticket-48360-Refac...

On Thu, Nov 26, 2015 at 05:16:59PM +0100, Simon Pichugin wrote:
> Hi team,
>
> please, review my patch for lib389 repo.
>
> https://fedorahosted.org/389/ticket/48360
>
> https://fedorahosted.org/389/attachment/ticket/48360/0001-Ticket-48360-Refac...
>
> Thank you,
> Simon
> --
> 389-devel mailing list
> 389-devel@%(host_name)s
> http://lists.fedoraproject.org/admin/lists/[e-mail]

[Simon Pichugin]

Fix the bug with bad indention. Added comment about missing test cases.

https://fedorahosted.org/389/ticket/48360

https://fedorahosted.org/389/attachment/ticket/48360/0001-Ticket-48360-Refac...

On Fri, Nov 27, 2015 at 04:42:27PM +0100, Simon Pichugin wrote:
> Revision with new feature.
>
> https://fedorahosted.org/389/ticket/48360
>
> https://fedorahosted.org/389/attachment/ticket/48360/0001-Ticket-48360-Refac...
>
> On Thu, Nov 26, 2015 at 05:16:59PM +0100, Simon Pichugin wrote:
> > Hi team,
> >
> > please, review my patch for lib389 repo.
> >
> > https://fedorahosted.org/389/ticket/48360
> >
> > https://fedorahosted.org/389/attachment/ticket/48360/0001-Ticket-48360-Refac...
> >
> > Thank you,
> > Simon
> > --
> > 389-devel mailing list
> > 389-devel@%(host_name)s
> > http://lists.fedoraproject.org/admin/lists/[e-mail]
>
> --
> 389-devel mailing list
> 389-devel@%(host_name)s
> http://lists.fedoraproject.org/admin/lists/[e-mail]