- Fix display of the bottom row in the query console by reloading table data *after* the scroll

 - Quote timestamp and connection when copying rows from the table console
 - Reduce queries when fetching database collations or encodings by removing an additional check and adapting logic to match

1 files changed, 7 insertions, 3 deletions

diff --git a/Source/SPQueryController.m b/Source/SPQueryController.m
index 6e9e561e..8a054ce3 100644
--- a/Source/SPQueryController.m
+++ b/Source/SPQueryController.m
@@ -191,14 +191,17 @@ static SPQueryController *sharedQueryController = nil;
 								
 				// If the timestamp column is not hidden we need to include them in the copy
 				if (!dateColumnIsHidden) {
+					[string appendString:@"/* "];
 					[string appendString:[dateFormatter stringFromDate:[message messageDate]]];
-					[string appendString:@" "];					
+					if (connectionColumnIsHidden) [string appendString:@" */ "];
+					else [string appendString:@" "];				
 				}
 				
 				// If the connection column is not hidden we need to include them in the copy
 				if (!connectionColumnIsHidden) {
+					if (dateColumnIsHidden) [string appendString:@"/* "];
 					[string appendString:[message messageConnection]];
-					[string appendString:@" "];
+					[string appendString:@" */ "];
 				}
 				
 				[string appendString:[message message]];
@@ -865,8 +868,9 @@ static SPQueryController *sharedQueryController = nil;
 	
 	// Reload the table and scroll to the new message if it's visible (for speed)
 	if (allowConsoleUpdate && [[self window] isVisible]) {
-		[consoleTableView reloadData];
+		[consoleTableView noteNumberOfRowsChanged];
 		[consoleTableView scrollRowToVisible:([messagesVisibleSet count] - 1)];
+		[consoleTableView reloadData];
 	}
 }