Re: Patch for column properties

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)postgresql(dot)org>
Cc: "Florian G(dot) Pflug" <fgp(at)phlo(dot)org>, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Patch for column properties
Date: 2007-03-04 10:52:13
Message-ID: 45EAA4DD.9080201@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Dave Page a écrit :
> Guillaume Lelarge wrote:
>> There's something I really like with EMS SQL Manager. When the user
>> change a property, the SQL is shown and can be changed. If the COMMIT
>> failed, they show you the SQL and it also can be modified. Perhaps we
>> just need this ? When there is a PostgreSQL related error message,
>> pgAdmin shows the SQL it tried to execute and let the user change it.
>> I think it would be a better behavior than what my patch implied.
>> Would something like this be possible ?
>
> Probably - but it seems it would be a fair amount of work. Feel free to
> look into it if you like.
>

I'll get a look at it. But I understand this could be a lot of work.

>> And one more question, I wonder why the SQL tab is not editable ? I
>> can think of one reason but I'm not sure :)
>
> Because it's dynamically generated, and any changes would either be lost
> if the user changed tab, or would have to be reverse engineered back
> into the other dialog controls.

That's what I thought.

> Also, on some dialogs there are
> placeholders included in the code for cases when we have multi-step
> queries that are tied together by a generated ID. IF the user mucked
> about with those, it could break things particularly spectaularly.
>

Can you tell me where I can find such multi-step queries ? Thanks.

Regards.

--
Guillaume.
<!-- http://abs.traduc.org/
http://lfs.traduc.org/
http://docs.postgresqlfr.org/ -->

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2007-03-04 11:11:25 Re: Patch for column properties
Previous Message Dave Page 2007-03-04 10:44:48 Re: Patch for column properties