From 862a431ebf1571f09890f43702e50fdc43b13415 Mon Sep 17 00:00:00 2001
From: Bibiko <bibiko@eva.mpg.de>
Date: Thu, 28 Jan 2010 17:04:04 +0000
Subject: =?UTF-8?q?=E2=80=A2=20for=20[CMTextView=20selectLineNumber:?=
 =?UTF-8?q?=E2=80=A6]=20added=20safety-check=20for=20lineNumber=20<=201=20?=
 =?UTF-8?q?=E2=80=A2=20error=20message=20highlighting=20in=20CustomQuery?=
 =?UTF-8?q?=20-=20check=20for=20error=20ID=201064=20before=20trying=20to?=
 =?UTF-8?q?=20select=20the=20erroneous=20line=20given=20as=20'=E2=80=A6=20?=
 =?UTF-8?q?at=20line=20x'=20(this=20solves=20the=20issue=20that=20other=20?=
 =?UTF-8?q?error=20messages=20could=20end=20by=20a=20number)=20-=20improve?=
 =?UTF-8?q?d=20regexp=20to=20get=20rid=20of=20localized=20error=20messages?=
 =?UTF-8?q?=20for=20parsing=20the=20erroneous=20line=20number=20(thanks=20?=
 =?UTF-8?q?to=20Jakob)=20-=20look=20the=20last=20number=20in=20a=20string?=
 =?UTF-8?q?=20-=20improved=20regexp=20to=20parse=20for=20the=20'near=20mes?=
 =?UTF-8?q?sage'=20for=20localized=20error=20messages?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Source/CMTextView.m  |  3 ++-
 Source/CustomQuery.m | 19 ++++++++++++-------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/Source/CMTextView.m b/Source/CMTextView.m
index 1d75eb93..6bd6f315 100644
--- a/Source/CMTextView.m
+++ b/Source/CMTextView.m
@@ -760,9 +760,10 @@ NSInteger alphabeticSort(id string1, id string2, void *reverse)
 			lineNumber++;
 		}
 	}
-	
+
 	// Safety-check the line number
 	if (lineNumber > [lineRanges count]) lineNumber = [lineRanges count];
+	if (lineNumber < 1) lineNumber = 1;
 
 	// Grab the range to select
 	selRange = NSRangeFromString([lineRanges objectAtIndex:lineNumber-1]);
diff --git a/Source/CustomQuery.m b/Source/CustomQuery.m
index 6f8afecf..45f177a3 100644
--- a/Source/CustomQuery.m
+++ b/Source/CustomQuery.m
@@ -691,9 +691,13 @@
 	if ( [mySQLConnection queryCancelled] || ([errors length] && !queryIsTableSorter)) {
 		// set the error text
 		[errorText setStringValue:errors];
-		// select the line x of the first error if error message contains "at line x"
-		NSRange errorLineNumberRange = [errors rangeOfRegex:@"(?<!key )([0-9]+)$" options:RKLNoOptions inRange:NSMakeRange(0, [errors length]) capture:1 error:nil];
-		if(errorLineNumberRange.length) // if a line number was found
+
+		// try to select the line x of the first error if error message with ID 1064 contains "at line x"
+		// by capturing the last number of the error string
+		NSRange errorLineNumberRange = [errors rangeOfRegex:@"([0-9]+)[^0-9]*$" options:RKLNoOptions inRange:NSMakeRange(0, [errors length]) capture:1L error:nil];
+
+		// if error ID 1064 and a line number was found
+		if([mySQLConnection getLastErrorID] == 1064 && errorLineNumberRange.length)
 		{
 			// Get the line number
 			NSUInteger errorAtLine = [[errors substringWithRange:errorLineNumberRange] integerValue];
@@ -701,7 +705,7 @@
 			[textView selectLineNumber:errorAtLine+lineOffset ignoreLeadingNewLines:YES];
 
 			// Check for near message
-			NSRange errorNearMessageRange = [errors rangeOfRegex:@" '(+*?)' " options:(RKLMultiline|RKLDotAll) inRange:NSMakeRange(0, [errors length]) capture:1 error:nil];
+			NSRange errorNearMessageRange = [errors rangeOfRegex:@"[( ]'(.+)'[ -]" options:(RKLMultiline|RKLDotAll) inRange:NSMakeRange(0, [errors length]) capture:1L error:nil];
 			if(errorNearMessageRange.length) // if a "near message" was found
 			{
 				// Build the range to search for nearMessage (beginning from queryStartPosition to try to avoid mismatching)
@@ -712,7 +716,6 @@
 				textNearMessageRange = NSMakeRange(textNearMessageRange.location+queryStartPosition, textNearMessageRange.length);
 				// Select the near message and scroll to it
 				[textView setSelectedRange:textNearMessageRange];
-				[textView scrollRangeToVisible:textNearMessageRange];
 			}
 		} else { // Select first erroneous query entirely
 			
@@ -727,8 +730,8 @@
 				// select the query for which the first error was detected
 				queryRange = [self queryTextRangeForQuery:firstErrorOccuredInQuery startPosition:queryStartPosition];
 				[textView setSelectedRange:queryRange];
+				[textView scrollRangeToVisible:queryRange];
 			}
-
 		}
 		
 	} else if ( [errors length] && queryIsTableSorter ) {
@@ -1071,10 +1074,12 @@
 	
 	[queries release];
 	
-	// Remove all leading white spaces
+	// Remove all leading and trailing white spaces
 	NSInteger offset = [theQueryString rangeOfRegex:@"^(\\s*)"].length;
 	theQueryRange.location += offset;
 	theQueryRange.length -= offset;
+	offset = [theQueryString rangeOfRegex:@"(\\s*)$"].length;
+	theQueryRange.length -= offset;
 	return theQueryRange;
 }
 
-- 
cgit v1.2.3