Re: Fixed FB 20417

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Fixed FB 20417
Date: 2012-01-09 12:34:33
Message-ID: CA+OCxox2UFxiwxDiQ5K502PMXuusVDvsOzT2qyhiTVbGXRdUiA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Mon, Jan 9, 2012 at 11:23 AM, Akshay Joshi
<akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
> Hi Dave
>
> I have fixed the Fogbugz 20417 "PEM client: Unable to remove comments on
> slony Replication(edb_cluster) tab". Attached is the patch file, please
> review it.

Hi,

I think the patch is basically correct, but needs a small mod so it
only appends the comment change if the user has actually changed it
from the existing value to something else.

Thanks!

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message pgAdmin Trac 2012-01-09 20:39:13 Re: [pgAdmin III] #342: Enumerable type doesn't have all the options available on CREATE TYPE
Previous Message Akshay Joshi 2012-01-09 11:23:01 Fixed FB 20417