diff options
| author | Meitar Moscovitz <meitarm@gmail.com> | 2009-02-10 19:42:01 +1100 |
|---|---|---|
| committer | Meitar Moscovitz <meitarm@gmail.com> | 2009-02-10 20:25:44 +1100 |
| commit | b240a1719485b15c4a6497b6053c24e8a40a4e3c (patch) | |
| tree | cb3ab3fa2e76c971c70bffaf37984d079cee7185 /actions/public.php | |
| parent | 3585012380bd8352b8e7bbb605bed05a9148fe11 (diff) | |
Beginning to refactor document relationship links to reduce common code.
My attempts here are to mimic the `pagination()` method shared by
actions. I'm tentatively adding the `$count` property to actions so that
we can query the number of notices ''being displayed'' per page prior to
calling the actual `pagination()` method itself, since document
relationship `<link>` elements need to be output inside of `showHead()`,
before `showContent()`, which is where `pagination()` is, gets called.
Diffstat (limited to 'actions/public.php')
| -rw-r--r-- | actions/public.php | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/actions/public.php b/actions/public.php index 25889eee5..eb2a4b1b0 100644 --- a/actions/public.php +++ b/actions/public.php @@ -56,6 +56,18 @@ class PublicAction extends Action var $page = null; + /** + * Number of notices being shown on this page. + */ + // Does this need to be here? Should it be? + // If it does, this property needs to be + // added to other actions as well, like $page. + // I'm trying to find a way to capture the + // output of the $cnt variable from this + // action's showContent() method but need + // to do so earlier, I think...? + var $count = null; + function isReadOnly() { return true; @@ -134,17 +146,8 @@ class PublicAction extends Action */ function showRelationshipLinks() { - // Machine-readable pagination - if ($this->page > 1) { - $this->element('link', array('rel' => 'next', - 'href' => common_local_url('public', - array('page' => $this->page - 1)), - 'title' => _('Next Notices'))); - } - $this->element('link', array('rel' => 'prev', - 'href' => common_local_url('public', - array('page' => $this->page + 1)), - 'title' => _('Previous Notices'))); + $this->sequenceRelationships($this->page > 1, $this->count > NOTICES_PER_PAGE, // FIXME + $this->page, 'public'); } /** |