diff options
author | Evan Prodromou <evan@controlyourself.ca> | 2009-08-14 13:05:29 -0700 |
---|---|---|
committer | Evan Prodromou <evan@controlyourself.ca> | 2009-08-14 13:05:29 -0700 |
commit | 1b204fde39271acd38bb8f9dd16734f15c68138a (patch) | |
tree | 946f756cde56cee6d0fe8e131c863639a8ad6113 /actions/showfavorites.php | |
parent | 2cf50ea432eab61e8cc2b007d486d0de3cd5aecf (diff) |
Revert "Return a 404 when a page is request that has no notices"
An empty list still exists; we shouldn't return a 404.
This reverts commit c29892c87573e0575b9a4a13210c562e08411148.
Diffstat (limited to 'actions/showfavorites.php')
-rw-r--r-- | actions/showfavorites.php | 44 |
1 files changed, 20 insertions, 24 deletions
diff --git a/actions/showfavorites.php b/actions/showfavorites.php index 91287cc96..9f549baf2 100644 --- a/actions/showfavorites.php +++ b/actions/showfavorites.php @@ -114,29 +114,6 @@ class ShowfavoritesAction extends OwnerDesignAction common_set_returnto($this->selfUrl()); - $cur = common_current_user(); - - if (!empty($cur) && $cur->id == $this->user->id) { - - // Show imported/gateway notices as well as local if - // the user is looking at his own favorites - - $this->notice = $this->user->favoriteNotices(($this->page-1)*NOTICES_PER_PAGE, - NOTICES_PER_PAGE + 1, true); - } else { - $this->notice = $this->user->favoriteNotices(($this->page-1)*NOTICES_PER_PAGE, - NOTICES_PER_PAGE + 1, false); - } - - if (empty($this->notice)) { - $this->serverError(_('Could not retrieve favorite notices.')); - return; - } - - if($this->page > 1 && $this->notice->N == 0){ - $this->serverError(_('No such page'),$code=404); - } - return true; } @@ -216,7 +193,26 @@ class ShowfavoritesAction extends OwnerDesignAction function showContent() { - $nl = new NoticeList($this->notice, $this); + $cur = common_current_user(); + + if (!empty($cur) && $cur->id == $this->user->id) { + + // Show imported/gateway notices as well as local if + // the user is looking at his own favorites + + $notice = $this->user->favoriteNotices(($this->page-1)*NOTICES_PER_PAGE, + NOTICES_PER_PAGE + 1, true); + } else { + $notice = $this->user->favoriteNotices(($this->page-1)*NOTICES_PER_PAGE, + NOTICES_PER_PAGE + 1, false); + } + + if (empty($notice)) { + $this->serverError(_('Could not retrieve favorite notices.')); + return; + } + + $nl = new NoticeList($notice, $this); $cnt = $nl->show(); if (0 == $cnt) { |