From 89704bcdeec2bac07b0528d21cd2d15bedf8c4ec Mon Sep 17 00:00:00 2001
From: Mikael Nordfeldth <mmn@hethane.se>
Date: Mon, 28 Jul 2014 10:57:45 +0200
Subject: [PATCH] Fixes some non-defined variables in commit eb2f93a

---
 lib/activitystreamjsondocument.php | 10 +++++-----
 lib/atomnoticefeed.php             |  8 ++++----
 lib/atomusernoticefeed.php         |  7 ++-----
 3 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/lib/activitystreamjsondocument.php b/lib/activitystreamjsondocument.php
index 767fb5dff7..0466045fef 100644
--- a/lib/activitystreamjsondocument.php
+++ b/lib/activitystreamjsondocument.php
@@ -54,6 +54,7 @@ class ActivityStreamJSONDocument extends JSONActivityCollection
 
     /* The current authenticated user */
     protected $cur;
+    protected $scoped = null;
 
     /* Title of the document */
     protected $title;
@@ -75,7 +76,8 @@ class ActivityStreamJSONDocument extends JSONActivityCollection
     {
         parent::__construct($items, $url);
 
-        $this->cur = $cur;
+        $this->cur = $cur ?: common_current_user();
+        $this->scoped = !is_null($this->cur) ? $this->cur->getProfile() : null;
 
         /* Title of the JSON document */
         $this->title = $title;
@@ -138,10 +140,8 @@ class ActivityStreamJSONDocument extends JSONActivityCollection
 
     function addItemFromNotice($notice)
     {
-        $cur = empty($this->cur) ? common_current_user() : $this->cur;
-
-        $act          = $notice->asActivity($cur);
-        $act->extra[] = $notice->noticeInfo($cur->getProfile());
+        $act          = $notice->asActivity($this->scoped);
+        $act->extra[] = $notice->noticeInfo($this->scoped);
         array_push($this->items, $act->asArray());
         $this->count++;
     }
diff --git a/lib/atomnoticefeed.php b/lib/atomnoticefeed.php
index b882172918..292bc97e85 100644
--- a/lib/atomnoticefeed.php
+++ b/lib/atomnoticefeed.php
@@ -45,6 +45,7 @@ if (!defined('STATUSNET'))
 class AtomNoticeFeed extends Atom10Feed
 {
     var $cur;
+    protected $scoped=null;
 
     /**
      * Constructor - adds a bunch of XML namespaces we need in our
@@ -58,7 +59,8 @@ class AtomNoticeFeed extends Atom10Feed
     function __construct($cur = null, $indent = true) {
         parent::__construct($indent);
 
-        $this->cur = $cur;
+        $this->cur = $cur ?: common_current_user();
+        $this->scoped = !is_null($this->cur) ? $this->cur->getProfile() : null;
 
         // Feeds containing notice info use these namespaces
 
@@ -129,9 +131,7 @@ class AtomNoticeFeed extends Atom10Feed
             $source = $this->showSource();
             $author = $this->showAuthor();
 
-            $cur = empty($this->cur) ? common_current_user() : $this->cur;
-
-            $this->addEntryRaw($notice->asAtomEntry(false, $source, $author, $cur));
+            $this->addEntryRaw($notice->asAtomEntry(false, $source, $author, $this->scoped));
         } catch (Exception $e) {
             common_log(LOG_ERR, $e->getMessage());
             // we continue on exceptions
diff --git a/lib/atomusernoticefeed.php b/lib/atomusernoticefeed.php
index d7f9b3d233..7b7f0a749f 100644
--- a/lib/atomusernoticefeed.php
+++ b/lib/atomusernoticefeed.php
@@ -63,11 +63,8 @@ class AtomUserNoticeFeed extends AtomNoticeFeed
             $profile = $user->getProfile();
 
             $ao = $profile->asActivityObject();
-
-            // quick fix until we start using Profile everywhere instead of User
-            $scoped = !is_null($cur) ? $cur->getProfile() : null;
-
-            array_push($ao->extra, $profile->profileInfo($scoped));
+            
+            array_push($ao->extra, $profile->profileInfo($this->scoped));
 
             $this->addAuthorRaw($ao->asString('author'));
         }