Re: Lightspeed for frmQuery and other issues. - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Lightspeed for frmQuery and other issues.
Date
Msg-id 007e01c66c5e$fb04c666$6a01a8c0@valehousing.co.uk
Whole thread Raw
In response to Lightspeed for frmQuery and other issues.  (Andreas Pflug <pgadmin@pse-consulting.de>)
Responses Re: Lightspeed for frmQuery and other issues.
List pgadmin-hackers

-----Original Message-----
From: "Andreas Pflug"<pgadmin@pse-consulting.de>
Sent: 30/04/06 14:48:08
To: "Dave Page"<dpage@vale-housing.co.uk>
Cc: "pgadmin-hackers@postgresql.org"<pgadmin-hackers@postgresql.org>
Subject: Re: Lightspeed for frmQuery and other issues.

> EditGrid is untouched.
> copying of arbitrary groups of cells did *never* work correct. The
> current implementation will omit cells if it doesn't know how to handle
> it. This is not acceptable: The ctl needs to be rewritten to not accept
> selections that can't be executed.

I've never run into such a bug, nor has it been reported. How can it be reproduced? Had it been reported, it would have
beenlooked at of course. 

> As I already may have mentioned earlier, I don't intend to spent time on
> the crappy wxGrid. So if anybody else likes to do so, he/she should pay
> attention not to change the ctlSQLResult interface for that.

With or without a bug, removing useful functionality is completely unnacceptable. You would be understandably pissed
offif a  feature you used was removed, and I would ensure the author reimplemented it, or rollback the change. Bug
aside,please fix the copy to clipboard to match what was there. 

/D

-----Unmodified Original Message-----
Dave Page wrote:
>
> -----Original Message----- From: "Andreas
> Pflug"<pgadmin@pse-consulting.de> Sent: 30/04/06 12:24:08 To: "Dave
> Page"<dpage@vale-housing.co.uk> Cc:
> "pgadmin-hackers@postgresql.org"<pgadmin-hackers@postgresql.org>
> Subject: Re: Lightspeed for frmQuery and other issues.
>
>
>>> Cool, nice work. I assume all the clipboard stuff still works?
>>
>> Depends on what you call "the clipboard stuff". Everything that
>> worked in 1.4 still works, i.e. single/multiple row copy. To select
>> columns, there's a special syntax to the SELECT command....
>
>
> Then you have some work to finish - both the edit and query grid
> could copy individual cells, rows, columns or arbitrary groups of
> cells to the clipboard - functionality that I for one am already
> using on a regular basis.

EditGrid is untouched.
copying of arbitrary groups of cells did *never* work correct. The
current implementation will omit cells if it doesn't know how to handle
it. This is not acceptable: The ctl needs to be rewritten to not accept
selections that can't be executed.
As I already may have mentioned earlier, I don't intend to spent time on
the crappy wxGrid. So if anybody else likes to do so, he/she should pay
attention not to change the ctlSQLResult interface for that.

Regards,
Andreas

pgadmin-hackers by date:

Previous
From: Andreas Pflug
Date:
Subject: Re: Lightspeed for frmQuery and other issues.
Next
From: svn@pgadmin.org
Date:
Subject: SVN Commit by andreas: r5103 - trunk/pgadmin3