diff --git a/actions/shownotice.php b/actions/shownotice.php index b189fdb87f..4bf4238cd4 100644 --- a/actions/shownotice.php +++ b/actions/shownotice.php @@ -69,7 +69,7 @@ class ShownoticeAction extends Action { $profile->nickname); # FIXME: URL, image, video, audio common_element_start('span', array('class' => 'content')); - common_raw(common_render_content($notice->content)); + common_raw(common_render_content($notice->content, $notice)); common_element_end('span'); common_element('span', array('class' => 'date'), common_date_string($notice->created)); diff --git a/actions/showstream.php b/actions/showstream.php index d4af631327..fdf3d33bd4 100644 --- a/actions/showstream.php +++ b/actions/showstream.php @@ -315,7 +315,7 @@ class ShowstreamAction extends StreamAction { if ($notice->find(true)) { # FIXME: URL, image, video, audio common_element_start('span', array('class' => 'content')); - common_raw(common_render_content($notice->content)); + common_raw(common_render_content($notice->content, $notice)); common_element_end('span'); } diff --git a/lib/stream.php b/lib/stream.php index b6834eb45a..ac683367f1 100644 --- a/lib/stream.php +++ b/lib/stream.php @@ -48,7 +48,7 @@ class StreamAction extends Action { $profile->nickname); # FIXME: URL, image, video, audio common_element_start('span', array('class' => 'content')); - common_raw(common_render_content($notice->content)); + common_raw(common_render_content($notice->content, $notice)); common_element_end('span'); $noticeurl = common_local_url('shownotice', array('notice' => $notice->id)); common_element('a', array('class' => 'notice', diff --git a/lib/util.php b/lib/util.php index 1b308d14ea..4ed1a4afb8 100644 --- a/lib/util.php +++ b/lib/util.php @@ -349,11 +349,47 @@ function common_canonical_email($email) { return $email; } -function common_render_content($text) { - # XXX: @ messages +function common_render_content($text, $notice=NULL) { + $r = htmlspecialchars($text); + if ($notice) { + $id = $notice->profile_id; + $r = preg_replace('/\b@([\w-]{1-64})\b/e', "@common_at_link($id, '\\1')", $r); + } # XXX: # tags # XXX: machine tags - return htmlspecialchars($text); + return $r; +} + +function common_at_link($profile_id, $nickname) { + # Try to find profiles this profile is subscribed to that have this nickname + $profile = new Profile(); + # XXX: chokety and bad + $profile->whereAdd('EXISTS (SELECT subscribed from subscription where subscriber = '.$profile_id.' and subscribed = id)', 'AND'); + $profile->whereAdd('nickname = "' . trim($nickname) . '"', 'AND'); + if ($profile->find(TRUE)) { + return ''.$nickname.''; + } + # Try to find profiles that listen to this profile and that have this nickname + $profile = new Profile(); + # XXX: chokety and bad + $profile->whereAdd('EXISTS (SELECT subscriber from subscription where subscribed = '.$profile_id.' and subscriber = id)', 'AND'); + $profile->whereAdd('nickname = "' . trim($nickname) . '"', 'AND'); + if ($profile->find(TRUE)) { + return ''.$nickname.''; + } + # If this is a local user, try to find a local user with that nickname. + $sender = User::staticGet($profile_id); + if ($sender) { + $recipient = User::staticGet('nickname', $nickname); + if ($recipient) { + $profile = $recipient->getProfile(); + return ''.$nickname.''; + } + } + # Otherwise, no links. @messages from local users to remote users, + # or from remote users to other remote users, are just + # outside our ability to make intelligent guesses about + return $nickname; } // where should the avatar go for this user?