summaryrefslogtreecommitdiff
path: root/actions/public.php
diff options
context:
space:
mode:
authorCraig Andrews <candrews@integralblue.com>2009-08-11 22:09:02 -0400
committerCraig Andrews <candrews@integralblue.com>2009-08-11 22:09:02 -0400
commit355effe6315ab7927a54f44fc9ce903085769b5f (patch)
treefaee7ec147b42cb64992cb10decef47776f88c9c /actions/public.php
parent7eda7295e47688fd582338ef6c83e6b6267f202f (diff)
Return a 404 when a page is request that has no notices
Diffstat (limited to 'actions/public.php')
-rw-r--r--actions/public.php23
1 files changed, 14 insertions, 9 deletions
diff --git a/actions/public.php b/actions/public.php
index dd128925b..ca352faf8 100644
--- a/actions/public.php
+++ b/actions/public.php
@@ -59,6 +59,7 @@ class PublicAction extends Action
*/
var $page = null;
+ var $notice;
function isReadOnly($args)
{
@@ -84,6 +85,18 @@ 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;
}
@@ -204,15 +217,7 @@ class PublicAction extends Action
function showContent()
{
- $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);
+ $nl = new NoticeList($this->notice, $this);
$cnt = $nl->show();