From fbfbe7fbc76e115503525233ed5f523d0f07ac61 Mon Sep 17 00:00:00 2001
From: Zach Copley <zach@controlyourself.ca>
Date: Sat, 22 Nov 2008 21:29:02 -0500
Subject: [PATCH] Don't display avatars when posting to personal streams via
 ajax

darcs-hash:20081123022902-7b5ce-fbdd4e9a3df3e206caec0ca37c10a2e0a58cef85.gz
---
 actions/newnotice.php | 39 +++++++++++++++++++++++++--------------
 1 file changed, 25 insertions(+), 14 deletions(-)

diff --git a/actions/newnotice.php b/actions/newnotice.php
index 0c1ad9523b..4eb6616f18 100644
--- a/actions/newnotice.php
+++ b/actions/newnotice.php
@@ -94,7 +94,7 @@ class NewnoticeAction extends Action {
 			common_element_end('html');
 		} else {
 			$returnto = $this->trimmed('returnto');
-			
+
 			if ($returnto) {
 				$url = common_local_url($returnto,
 										array('nickname' => $user->nickname));
@@ -157,21 +157,30 @@ class NewnoticeAction extends Action {
 				common_favor_form($notice);
 			}
 		}
-		$avatar = $profile->getAvatar(AVATAR_STREAM_SIZE);
-		common_element_start('a', array('href' => $profile->profileurl));
-		common_element('img', array('src' => ($avatar) ? common_avatar_display_url($avatar) : common_default_avatar(AVATAR_STREAM_SIZE),
-									'class' => 'avatar stream',
-									'width' => AVATAR_STREAM_SIZE,
-									'height' => AVATAR_STREAM_SIZE,
-									'alt' =>
-									($profile->fullname) ? $profile->fullname :
-									$profile->nickname));
-		common_element_end('a');
-		common_element('a', array('href' => $profile->profileurl,
-								  'class' => 'nickname'),
-					   $profile->nickname);
+
+		$returnto = $this->trimmed('returnto');
+
+		# If this is the personal stream, we don't want avatars
+		if ($returnto != 'showstream') {
+
+			$avatar = $profile->getAvatar(AVATAR_STREAM_SIZE);
+			common_element_start('a', array('href' => $profile->profileurl));
+			common_element('img', array('src' => ($avatar) ? common_avatar_display_url($avatar) : common_default_avatar(AVATAR_STREAM_SIZE),
+										'class' => 'avatar stream',
+										'width' => AVATAR_STREAM_SIZE,
+										'height' => AVATAR_STREAM_SIZE,
+										'alt' =>
+										($profile->fullname) ? $profile->fullname :
+										$profile->nickname));
+			common_element_end('a');
+			common_element('a', array('href' => $profile->profileurl,
+									  'class' => 'nickname'),
+						   $profile->nickname);
+		}
+
 		# FIXME: URL, image, video, audio
 		common_element_start('p', array('class' => 'content'));
+
 		if ($notice->rendered) {
 			common_raw($notice->rendered);
 		} else {
@@ -181,6 +190,8 @@ class NewnoticeAction extends Action {
 			common_raw(common_render_content($notice->content, $notice));
 		}
 		common_element_end('p');
+
+
 		$noticeurl = common_local_url('shownotice', array('notice' => $notice->id));
 		# XXX: we need to figure this out better. Is this right?
 		if (strcmp($notice->uri, $noticeurl) != 0 && preg_match('/^http/', $notice->uri)) {