Revert "Revert "Return a 404 when a page is request that has no notices""
This reverts commit 6c597eabec
.
This commit is contained in:
parent
6c597eabec
commit
e0815de960
@ -59,6 +59,7 @@ class PublicAction extends Action
|
|||||||
*/
|
*/
|
||||||
|
|
||||||
var $page = null;
|
var $page = null;
|
||||||
|
var $notice;
|
||||||
|
|
||||||
function isReadOnly($args)
|
function isReadOnly($args)
|
||||||
{
|
{
|
||||||
@ -84,6 +85,18 @@ class PublicAction extends Action
|
|||||||
|
|
||||||
common_set_returnto($this->selfUrl());
|
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;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -204,15 +217,7 @@ class PublicAction extends Action
|
|||||||
|
|
||||||
function showContent()
|
function showContent()
|
||||||
{
|
{
|
||||||
$notice = Notice::publicStream(($this->page-1)*NOTICES_PER_PAGE,
|
$nl = new NoticeList($this->notice, $this);
|
||||||
NOTICES_PER_PAGE + 1);
|
|
||||||
|
|
||||||
if (!$notice) {
|
|
||||||
$this->serverError(_('Could not retrieve public stream.'));
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
$nl = new NoticeList($notice, $this);
|
|
||||||
|
|
||||||
$cnt = $nl->show();
|
$cnt = $nl->show();
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user