Author: dpage
Date: 2006-01-09 09:22:29 +0000 (Mon, 09 Jan 2006)
New Revision: 4882

Modified:
   branches/REL-1_4_0_PATCHES/pgadmin3/CHANGELOG.txt
   branches/REL-1_4_0_PATCHES/pgadmin3/src/ctl/ctlSQLResult.cpp
Log:
Honour the copy quoting setting properly in the SQL results pane [Magnus 
Hagander]

Modified: branches/REL-1_4_0_PATCHES/pgadmin3/CHANGELOG.txt
===================================================================
--- branches/REL-1_4_0_PATCHES/pgadmin3/CHANGELOG.txt   2006-01-09 09:11:33 UTC 
(rev 4881)
+++ branches/REL-1_4_0_PATCHES/pgadmin3/CHANGELOG.txt   2006-01-09 09:22:29 UTC 
(rev 4882)
@@ -18,6 +18,7 @@
 </ul>
 <br>
 <ul>
+    <li>2006-01-09 DP  1.4.2  Honour the copy quoting setting properly in the 
SQL results pane [Magnus Hagander]
     <li>2006-01-09 DP  1.4.2  Set the initial Unicode/Local charset options 
correctly in the Export dialogue [Magnus Hagander]
     <li>2005-12-26 AP  1.4.2  Fix PK detection in EditGrid per Andrus Moor
     <li>2005-12-26 AP  1.4.2  Fix Bitmap Index explain per Alexander Kirpa

Modified: branches/REL-1_4_0_PATCHES/pgadmin3/src/ctl/ctlSQLResult.cpp
===================================================================
--- branches/REL-1_4_0_PATCHES/pgadmin3/src/ctl/ctlSQLResult.cpp        
2006-01-09 09:11:33 UTC (rev 4881)
+++ branches/REL-1_4_0_PATCHES/pgadmin3/src/ctl/ctlSQLResult.cpp        
2006-01-09 09:22:29 UTC (rev 4882)
@@ -72,19 +72,25 @@
 
         wxString text=GetItemText(row, col);
 
-        bool needQuote=(settings->GetExportQuoting() > 1);
+               bool needQuote  = false;
+               if (settings->GetExportQuoting() == 1)
+               {
+                       /* Quote strings only */
+                       switch (colTypClasses.Item(col))
+                       {
+                       case PGTYPCLASS_NUMERIC:
+                       case PGTYPCLASS_BOOL:
+                               break;
+                       default:
+                               needQuote=true;
+                               break;
+                       }
+               }
+               else if (settings->GetExportQuoting() == 2)
+                       /* Quote everything */
+                       needQuote = true;
 
-    
-        switch (colTypClasses.Item(col))
-        {
-            case PGTYPCLASS_NUMERIC:
-            case PGTYPCLASS_BOOL:
-                break;
-            default:
-                needQuote=true;
-                break;
-        }
-        if (needQuote)
+               if (needQuote)
             str.Append(settings->GetExportQuoteChar());
         str.Append(text);
         if (needQuote)


---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Reply via email to