diff options
author | Evan Prodromou <evan@controlyourself.ca> | 2009-08-14 13:05:57 -0700 |
---|---|---|
committer | Evan Prodromou <evan@controlyourself.ca> | 2009-08-14 13:05:57 -0700 |
commit | 6c597eabecf2368224938a136d7ac588702bbfd7 (patch) | |
tree | 852f779da78b98ff0cbcd238eb642db99df0248c /actions | |
parent | 1b204fde39271acd38bb8f9dd16734f15c68138a (diff) |
Revert "Return a 404 when a page is request that has no notices"
An empty list still exists; we should not return a 404 for empty lists.
This reverts commit 355effe6315ab7927a54f44fc9ce903085769b5f.
Diffstat (limited to 'actions')
-rw-r--r-- | actions/public.php | 23 |
1 files changed, 9 insertions, 14 deletions
diff --git a/actions/public.php b/actions/public.php index ca352faf8..dd128925b 100644 --- a/actions/public.php +++ b/actions/public.php @@ -59,7 +59,6 @@ class PublicAction extends Action */ var $page = null; - var $notice; function isReadOnly($args) { @@ -85,18 +84,6 @@ class PublicAction extends Action common_set_returnto($this->selfUrl()); - $this->notice = Notice::publicStream(($this->page-1)*NOTICES_PER_PAGE, - NOTICES_PER_PAGE + 1); - - if (!$this->notice) { - $this->serverError(_('Could not retrieve public stream.')); - return; - } - - if($this->page > 0 && $this->notice->N == 0){ - $this->serverError(_('No such page'),$code=404); - } - return true; } @@ -217,7 +204,15 @@ class PublicAction extends Action function showContent() { - $nl = new NoticeList($this->notice, $this); + $notice = Notice::publicStream(($this->page-1)*NOTICES_PER_PAGE, + NOTICES_PER_PAGE + 1); + + if (!$notice) { + $this->serverError(_('Could not retrieve public stream.')); + return; + } + + $nl = new NoticeList($notice, $this); $cnt = $nl->show(); |