From: Zach Copley <zach@status.net>
Date: Sat, 6 Mar 2010 00:40:35 +0000 (-0800)
Subject: Fix undefined variable error and some other cleanup
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=f653c3b914d7983618bd4141fe57a37e9537ec45;p=quix0rs-gnu-social.git

Fix undefined variable error and some other cleanup
---

diff --git a/actions/userrss.php b/actions/userrss.php
index 6029f44318..77bd316b2d 100644
--- a/actions/userrss.php
+++ b/actions/userrss.php
@@ -29,6 +29,8 @@ class UserrssAction extends Rss10Action
 
     function prepare($args)
     {
+        common_debug("UserrssAction");
+
         parent::prepare($args);
         $nickname   = $this->trimmed('nickname');
         $this->user = User::staticGet('nickname', $nickname);
@@ -38,8 +40,8 @@ class UserrssAction extends Rss10Action
             $this->clientError(_('No such user.'));
             return false;
         } else {
-            if ($this->tag) {
-                $this->notices = $this->getTaggedNotices($tag, $this->limit);
+            if (!empty($this->tag)) {
+                $this->notices = $this->getTaggedNotices($this->tag, $this->limit);
             } else {
                 $this->notices = $this->getNotices($this->limit);
             }
@@ -47,15 +49,15 @@ class UserrssAction extends Rss10Action
         }
     }
 
-    function getTaggedNotices($tag = null, $limit=0)
+    function getTaggedNotices()
     {
-        $user = $this->user;
-
-        if (is_null($user)) {
-            return null;
-        }
-
-        $notice = $user->getTaggedNotices(0, ($limit == 0) ? NOTICES_PER_PAGE : $limit, 0, 0, null, $tag);
+        $notice = $this->user->getTaggedNotices(
+            $this->tag,
+            0,
+            ($this->limit == 0) ? NOTICES_PER_PAGE : $this->limit,
+            0,
+            0
+        );
 
         $notices = array();
         while ($notice->fetch()) {
@@ -66,15 +68,12 @@ class UserrssAction extends Rss10Action
     }
 
 
-    function getNotices($limit=0)
+    function getNotices()
     {
-        $user = $this->user;
-        
-        if (is_null($user)) {
-            return null;
-        }
-
-        $notice = $user->getNotices(0, ($limit == 0) ? NOTICES_PER_PAGE : $limit);
+        $notice = $this->user->getNotices(
+            0,
+            ($limit == 0) ? NOTICES_PER_PAGE : $limit
+        );
 
         $notices = array();
         while ($notice->fetch()) {