From: Tim Düsterhus <duesterhus@woltlab.com>
Date: Tue, 2 Dec 2014 02:06:59 +0000 (+0100)
Subject: Match usernames with up to one space in PreParserAtUserListener
X-Git-Tag: 2.1.0_Beta_1~106
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=a024d6a4030f726738e36cfe5d1cf7d0c198f03b;p=GitHub%2FWoltLab%2FWCF.git

Match usernames with up to one space in PreParserAtUserListener
---

diff --git a/wcfsetup/install/files/lib/system/event/listener/PreParserAtUserListener.class.php b/wcfsetup/install/files/lib/system/event/listener/PreParserAtUserListener.class.php
index bb0f72bb36..eed04584dc 100644
--- a/wcfsetup/install/files/lib/system/event/listener/PreParserAtUserListener.class.php
+++ b/wcfsetup/install/files/lib/system/event/listener/PreParserAtUserListener.class.php
@@ -31,7 +31,17 @@ class PreParserAtUserListener implements IParameterizedEventListener {
 		
 		static $userRegex = null;
 		if ($userRegex === null) {
-			$userRegex = new Regex("(?<=^|\s)@([^',\s][^,\s]{2,}|'(?:''|[^'])*')");
+			$userRegex = new Regex("
+				(?<=^|\s)					# either at start of string, or after whitespace
+				@
+				(
+					([^',\s][^,\s]{2,})(?:\s[^,\s]+)?	# either at most two strings, not containing
+										# whitespace or the comma, not starting with a single quote
+										# separated by a single whitespace character
+				|
+					'(?:''|[^'])*'				# or a string delimited by single quotes
+				)
+			", Regex::IGNORE_WHITESPACE);
 		}
 		
 		// cache quotes
@@ -70,14 +80,19 @@ class PreParserAtUserListener implements IParameterizedEventListener {
 			$text .= $quote;
 		}
 		
-		$userRegex->match($text, true);
+		$userRegex->match($text, true, Regex::ORDER_MATCH_BY_SET);
 		$matches = $userRegex->getMatches();
 		
-		if (!empty($matches[1])) {
+		if (!empty($matches)) {
 			$usernames = array();
-			foreach ($matches[1] as $match) {
-				$username = self::getUsername($match);
-				if (!in_array($username, $usernames)) $usernames[] = $username;
+			foreach ($matches as $match) {
+				// we don't care about the full match
+				array_shift($match);
+				
+				foreach ($match as $username) {
+					$username = self::getUsername($username);
+					if (!in_array($username, $usernames)) $usernames[] = $username;
+				}
 			}
 			
 			if (!empty($usernames)) {
@@ -91,9 +106,15 @@ class PreParserAtUserListener implements IParameterizedEventListener {
 				}
 				
 				$text = $userRegex->replace($text, new Callback(function ($matches) use ($users) {
-					$username = PreParserAtUserListener::getUsername($matches[1]);
+					// containing the full match
+					$usernames = array($matches[1]);
+					// containing only the part before the first space
+					if (isset($matches[2])) $usernames[] = $matches[2];
+					
+					$usernames = array_map(array('self', 'getUsername'), $usernames);
 					
-					if (isset($users[$username])) {
+					foreach ($usernames as $username) {
+						if (!isset($users[$username])) continue;
 						$link = LinkHandler::getInstance()->getLink('User', array(
 							'appendSession' => false,
 							'object' => $users[$username]