adium 2290:7ad20dd9aac0: Avoid being way, way too overzealous in...

commits at adium.im commits at adium.im
Sat May 23 23:47:46 UTC 2009


details:	http://hg.adium.im/adium/rev/7ad20dd9aac0
revision:	2290:7ad20dd9aac0
author:		Zachary West <zacw at adium.im>
date:		Sat May 23 19:47:33 2009 -0400

Avoid being way, way too overzealous in replacing parenthesis with HTML. /facepalm

diffstat:

 Plugins/Twitter Plugin/AITwitterActionsHTMLFilter.m |  8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)

diffs (27 lines):

diff -r ce81a0ced825 -r 7ad20dd9aac0 Plugins/Twitter Plugin/AITwitterActionsHTMLFilter.m
--- a/Plugins/Twitter Plugin/AITwitterActionsHTMLFilter.m	Sat May 23 17:43:12 2009 -0400
+++ b/Plugins/Twitter Plugin/AITwitterActionsHTMLFilter.m	Sat May 23 19:47:33 2009 -0400
@@ -40,17 +40,21 @@
 			
 			NSMutableString *mutableHTML = [inHTMLString mutableCopy];
 			
+			NSRange startRange = [mutableHTML rangeOfString:@" (" options:NSBackwardsSearch];
+			
 			// Replace the start with <span class="twitter_actions"><span class="twitter_actionStart>(</span>
 			// This overall span will be ended next.
 			[mutableHTML replaceOccurrencesOfString:@"("
 										 withString:@"<span class=\"twitter_actions\"><span class=\"twitter_actionStart\">(</span>"
-											options:NSBackwardsSearch];
+											options:NSBackwardsSearch
+											  range:startRange];
 			
 			// Replace the end with <span class="twitter_actionEnd">)</span></span>
 			// The second </span> finishes off the overall actions links span.
 			[mutableHTML replaceOccurrencesOfString:@")"
 										 withString:@"<span class=\"twitter_actionEnd\">)</span></span>"
-											options:NSBackwardsSearch];
+											options:NSBackwardsSearch
+											  range:NSMakeRange(mutableHTML.length - 1, 1)];
 			
 			return [mutableHTML autorelease];
 		}




More information about the commits mailing list