Thread: [Squirrel-sql-develop] Another Re-Arranged Columns Bug
A Java SQL client for any JDBC compliant database
Brought to you by:
colbell,
gerdwagner
From: Joseph B. <jb...@ww...> - 2003-08-20 06:58:28
Attachments:
CellDataPopup.java.patch
|
Squirrel (1.2b3) has a bug related to the title of the window that pops up when you double-click a cell in the results table. If you have re-arranged the table columns by dragging them around, the title displayed in the pop-up window does not reflect the new order of the columns. This is essentially the same bug I posted about previously, where copy/paste of re-arranged columns was broken, it's just in a different place. It's essentially the same fix, too. After I noticed the first bug, I was sort of grep-ing through the source, looking for similar bugs. I found one. Attached is a patch to CellDataPopup.java that corrects the problem. Let me know if there are any problems. Enjoy. ...Joseph Boyd Lead Application Developer accessIndiana..."linking hoosiers to government" http://www.IN.gov 10 West Market St., Suite 600 Indianapolis, IN 46204 |
From: Colin B. <col...@bi...> - 2003-08-22 23:09:32
|
On Wed, 20 Aug 2003 08:07 am, Joseph Boyd wrote: > Squirrel (1.2b3) has a bug related to the title of the window that pops > up when you double-click a cell in the results table. > > If you have re-arranged the table columns by dragging them around, the > title displayed in the pop-up window does not reflect the new order of > the columns. > > This is essentially the same bug I posted about previously, where > copy/paste of re-arranged columns was broken, it's just in a different > place. It's essentially the same fix, too. After I noticed the first > bug, I was sort of grep-ing through the source, looking for similar > bugs. I found one. > > > Attached is a patch to CellDataPopup.java that corrects the problem. > Let me know if there are any problems. > > > Enjoy. > > > ...Joseph Boyd > > Lead Application Developer > accessIndiana..."linking hoosiers to government" > http://www.IN.gov > 10 West Market St., Suite 600 > Indianapolis, IN 46204 Thanks Joseph, this has been applied to CVS. -- Colin Bell http://squirrel-sql.sf.net |