Changeset 3570 for trunk/system/views

Show
Ignore:
Timestamp:
10/10/2008 11:17:21 AM (6 weeks ago)
Author:
Geert
Message:
  • Fix #867 via pagination.clean_url_page1 and new url() method
  • Removed call() which only was a slow superfluous wrapper for get()
  • Note: pagination views need to be updated!
Location:
trunk/system/views/pagination
Files:
4 modified

Legend:

Unmodified
Added
Removed
  • trunk/system/views/pagination/classic.php

    r1766 r3570  
    99<p class="pagination"> 
    1010 
    11         <?php if ($first_page): ?> 
    12                 <a href="<?php echo str_replace('{page}', 1, $url) ?>">&lsaquo;&nbsp;<?php echo Kohana::lang('pagination.first') ?></a> 
     11        <?php if ($p->first_page): ?> 
     12                <a href="<?php echo $p->url(1) ?>">&lsaquo;&nbsp;<?php echo Kohana::lang('pagination.first') ?></a> 
    1313        <?php endif ?> 
    1414 
    15         <?php if ($previous_page): ?> 
    16                 <a href="<?php echo str_replace('{page}', $previous_page, $url) ?>">&lt;</a> 
     15        <?php if ($p->previous_page): ?> 
     16                <a href="<?php echo $p->url($p->previous_page) ?>">&lt;</a> 
    1717        <?php endif ?> 
    1818 
    1919 
    20         <?php for ($i = 1; $i <= $total_pages; $i++): ?> 
     20        <?php for ($i = 1; $i <= $p->total_pages; $i++): ?> 
    2121 
    22                 <?php if ($i == $current_page): ?> 
     22                <?php if ($i == $p->current_page): ?> 
    2323                        <strong><?php echo $i ?></strong> 
    2424                <?php else: ?> 
    25                         <a href="<?php echo str_replace('{page}', $i, $url) ?>"><?php echo $i ?></a> 
     25                        <a href="<?php echo $p->url($i) ?>"><?php echo $i ?></a> 
    2626                <?php endif ?> 
    2727 
     
    2929 
    3030 
    31         <?php if ($next_page): ?> 
    32                 <a href="<?php echo str_replace('{page}', $next_page, $url) ?>">&gt;</a> 
     31        <?php if ($p->next_page): ?> 
     32                <a href="<?php echo $p->url($p->next_page) ?>">&gt;</a> 
    3333        <?php endif ?> 
    3434 
    35         <?php if ($last_page): ?> 
    36                 <a href="<?php echo str_replace('{page}', $last_page, $url) ?>"><?php echo Kohana::lang('pagination.last') ?>&nbsp;&rsaquo;</a> 
     35        <?php if ($p->last_page): ?> 
     36                <a href="<?php echo $p->url($p->last_page) ?>"><?php echo Kohana::lang('pagination.last') ?>&nbsp;&rsaquo;</a> 
    3737        <?php endif ?> 
    3838 
  • trunk/system/views/pagination/digg.php

    r1766 r3570  
    99<p class="pagination"> 
    1010 
    11         <?php if ($previous_page): ?> 
    12                 <a href="<?php echo str_replace('{page}', $previous_page, $url) ?>">&laquo;&nbsp;<?php echo Kohana::lang('pagination.previous') ?></a> 
     11        <?php if ($p->previous_page): ?> 
     12                <a href="<?php echo $p->url($p->previous_page) ?>">&laquo;&nbsp;<?php echo Kohana::lang('pagination.previous') ?></a> 
    1313        <?php else: ?> 
    1414                &laquo;&nbsp;<?php echo Kohana::lang('pagination.previous') ?> 
     
    1616 
    1717 
    18         <?php if ($total_pages < 13): /* « Previous  1 2 3 4 5 6 7 8 9 10 11 12  Next » */ ?> 
     18        <?php if ($p->total_pages < 13): /* « Previous  1 2 3 4 5 6 7 8 9 10 11 12  Next » */ ?> 
    1919 
    20                 <?php for ($i = 1; $i <= $total_pages; $i++): ?> 
    21                         <?php if ($i == $current_page): ?> 
     20                <?php for ($i = 1; $i <= $p->total_pages; $i++): ?> 
     21                        <?php if ($i == $p->current_page): ?> 
    2222                                <strong><?php echo $i ?></strong> 
    2323                        <?php else: ?> 
    24                                 <a href="<?php echo str_replace('{page}', $i, $url) ?>"><?php echo $i ?></a> 
     24                                <a href="<?php echo $p->url($i) ?>"><?php echo $i ?></a> 
    2525                        <?php endif ?> 
    2626                <?php endfor ?> 
    2727 
    28         <?php elseif ($current_page < 9): /* « Previous  1 2 3 4 5 6 7 8 9 10 … 25 26  Next » */ ?> 
     28        <?php elseif ($p->current_page < 9): /* « Previous  1 2 3 4 5 6 7 8 9 10 … 25 26  Next » */ ?> 
    2929 
    3030                <?php for ($i = 1; $i <= 10; $i++): ?> 
    31                         <?php if ($i == $current_page): ?> 
     31                        <?php if ($i == $p->current_page): ?> 
    3232                                <strong><?php echo $i ?></strong> 
    3333                        <?php else: ?> 
    34                                 <a href="<?php echo str_replace('{page}', $i, $url) ?>"><?php echo $i ?></a> 
     34                                <a href="<?php echo $p->url($i) ?>"><?php echo $i ?></a> 
    3535                        <?php endif ?> 
    3636                <?php endfor ?> 
    3737 
    38                 &hellip; 
    39                 <a href="<?php echo str_replace('{page}', $total_pages - 1, $url) ?>"><?php echo $total_pages - 1 ?></a> 
    40                 <a href="<?php echo str_replace('{page}', $total_pages, $url) ?>"><?php echo $total_pages ?></a> 
     38                 
     39                <a href="<?php echo $p->url($p->total_pages - 1) ?>"><?php echo $p->total_pages - 1 ?></a> 
     40                <a href="<?php echo $p->url($p->total_pages) ?>"><?php echo $p->total_pages ?></a> 
    4141 
    42         <?php elseif ($current_page > $total_pages - 8): /* « Previous  1 2 … 17 18 19 20 21 22 23 24 25 26  Next » */ ?> 
     42        <?php elseif ($p->current_page > $p->total_pages - 8): /* « Previous  1 2 … 17 18 19 20 21 22 23 24 25 26  Next » */ ?> 
    4343 
    44                 <a href="<?php echo str_replace('{page}', 1, $url) ?>">1</a> 
    45                 <a href="<?php echo str_replace('{page}', 2, $url) ?>">2</a> 
    46                 &hellip; 
     44                <a href="<?php echo $p->url(1) ?>">1</a> 
     45                <a href="<?php echo $p->url(2) ?>">2</a> 
     46                 
    4747 
    48                 <?php for ($i = $total_pages - 9; $i <= $total_pages; $i++): ?> 
    49                         <?php if ($i == $current_page): ?> 
     48                <?php for ($i = $p->total_pages - 9; $i <= $p->total_pages; $i++): ?> 
     49                        <?php if ($i == $p->current_page): ?> 
    5050                                <strong><?php echo $i ?></strong> 
    5151                        <?php else: ?> 
    52                                 <a href="<?php echo str_replace('{page}', $i, $url) ?>"><?php echo $i ?></a> 
     52                                <a href="<?php echo $p->url($i) ?>"><?php echo $i ?></a> 
    5353                        <?php endif ?> 
    5454                <?php endfor ?> 
     
    5656        <?php else: /* « Previous  1 2 … 5 6 7 8 9 10 11 12 13 14 … 25 26  Next » */ ?> 
    5757 
    58                 <a href="<?php echo str_replace('{page}', 1, $url) ?>">1</a> 
    59                 <a href="<?php echo str_replace('{page}', 2, $url) ?>">2</a> 
    60                 &hellip; 
     58                <a href="<?php echo $p->url(1) ?>">1</a> 
     59                <a href="<?php echo $p->url(2) ?>">2</a> 
     60                 
    6161 
    62                 <?php for ($i = $current_page - 5; $i <= $current_page + 5; $i++): ?> 
    63                         <?php if ($i == $current_page): ?> 
     62                <?php for ($i = $p->current_page - 5; $i <= $p->current_page + 5; $i++): ?> 
     63                        <?php if ($i == $p->current_page): ?> 
    6464                                <strong><?php echo $i ?></strong> 
    6565                        <?php else: ?> 
    66                                 <a href="<?php echo str_replace('{page}', $i, $url) ?>"><?php echo $i ?></a> 
     66                                <a href="<?php echo $p->url($i) ?>"><?php echo $i ?></a> 
    6767                        <?php endif ?> 
    6868                <?php endfor ?> 
    6969 
    70                 &hellip; 
    71                 <a href="<?php echo str_replace('{page}', $total_pages - 1, $url) ?>"><?php echo $total_pages - 1 ?></a> 
    72                 <a href="<?php echo str_replace('{page}', $total_pages, $url) ?>"><?php echo $total_pages ?></a> 
     70                 
     71                <a href="<?php echo $p->url($p->total_pages - 1) ?>"><?php echo $p->total_pages - 1 ?></a> 
     72                <a href="<?php echo $p->url($p->total_pages) ?>"><?php echo $p->total_pages ?></a> 
    7373 
    7474        <?php endif ?> 
    7575 
    7676 
    77         <?php if ($next_page): ?> 
    78                 <a href="<?php echo str_replace('{page}', $next_page, $url) ?>"><?php echo Kohana::lang('pagination.next') ?>&nbsp;&raquo;</a> 
     77        <?php if ($p->next_page): ?> 
     78                <a href="<?php echo $p->url($p->next_page) ?>"><?php echo Kohana::lang('pagination.next') ?>&nbsp;&raquo;</a> 
    7979        <?php else: ?> 
    8080                <?php echo Kohana::lang('pagination.next') ?>&nbsp;&raquo; 
  • trunk/system/views/pagination/extended.php

    r1766 r3570  
    33 * Extended pagination style 
    44 *  
    5  * @preview  « Previous | Page 2 of 11 | Showing items 6-10 of 52 | Next » 
     5 * @preview  « Previous | Page 2 of 11 | Showing items 610 of 52 | Next » 
    66 */ 
    77?> 
     
    99<p class="pagination"> 
    1010 
    11         <?php if ($previous_page): ?> 
    12                 <a href="<?php echo str_replace('{page}', $previous_page, $url) ?>">&laquo;&nbsp;<?php echo Kohana::lang('pagination.previous') ?></a> 
     11        <?php if ($p->previous_page): ?> 
     12                <a href="<?php echo $p->url($p->previous_page) ?>">&laquo;&nbsp;<?php echo Kohana::lang('pagination.previous') ?></a> 
    1313        <?php else: ?> 
    1414                &laquo;&nbsp;<?php echo Kohana::lang('pagination.previous') ?> 
    1515        <?php endif ?> 
    1616 
    17         | <?php echo Kohana::lang('pagination.page') ?> <?php echo $current_page ?> <?php echo Kohana::lang('pagination.of') ?> <?php echo $total_pages ?> 
     17        | <?php echo Kohana::lang('pagination.page') ?> <?php echo $p->current_page ?> <?php echo Kohana::lang('pagination.of') ?> <?php echo $p->total_pages ?> 
    1818 
    19         | <?php echo Kohana::lang('pagination.items') ?> <?php echo $current_first_item ?>&ndash;<?php echo $current_last_item ?> <?php echo Kohana::lang('pagination.of') ?> <?php echo $total_items ?> 
     19        | <?php echo Kohana::lang('pagination.items') ?> <?php echo $p->current_first_item ?>–<?php echo $p->current_last_item ?> <?php echo Kohana::lang('pagination.of') ?> <?php echo $p->total_items ?> 
    2020 
    21         | <?php if ($next_page): ?> 
    22                 <a href="<?php echo str_replace('{page}', $next_page, $url) ?>"><?php echo Kohana::lang('pagination.next') ?>&nbsp;&raquo;</a> 
     21        | <?php if ($p->next_page): ?> 
     22                <a href="<?php echo $p->url($p->next_page) ?>"><?php echo Kohana::lang('pagination.next') ?>&nbsp;&raquo;</a> 
    2323        <?php else: ?> 
    2424                <?php echo Kohana::lang('pagination.next') ?>&nbsp;&raquo; 
  • trunk/system/views/pagination/punbb.php

    r1766 r3570  
    1111        <?php echo Kohana::lang('pagination.pages') ?>: 
    1212 
    13         <?php if ($current_page > 3): ?> 
    14                 <a href="<?php echo str_replace('{page}', 1, $url) ?>">1</a> 
    15                 <?php if ($current_page != 4) echo '&hellip;' ?> 
     13        <?php if ($p->current_page > 3): ?> 
     14                <a href="<?php echo $p->url(1) ?>">1</a> 
     15                <?php if ($p->current_page != 4) echo '…' ?> 
    1616        <?php endif ?> 
    1717 
    1818 
    19         <?php for ($i = $current_page - 2, $stop = $current_page + 3; $i < $stop; ++$i): ?> 
     19        <?php for ($i = $p->current_page - 2, $stop = $p->current_page + 3; $i < $stop; ++$i): ?> 
    2020 
    21                 <?php if ($i < 1 OR $i > $total_pages) continue ?> 
     21                <?php if ($i < 1 OR $i > $p->total_pages) continue ?> 
    2222 
    23                 <?php if ($current_page == $i): ?> 
     23                <?php if ($p->current_page == $i): ?> 
    2424                        <strong><?php echo $i ?></strong> 
    2525                <?php else: ?> 
    26                         <a href="<?php echo str_replace('{page}', $i, $url) ?>"><?php echo $i ?></a> 
     26                        <a href="<?php echo $p->url($i) ?>"><?php echo $i ?></a> 
    2727                <?php endif ?> 
    2828 
     
    3030 
    3131 
    32         <?php if ($current_page <= $total_pages - 3): ?> 
    33                 <?php if ($current_page != $total_pages - 3) echo '&hellip;' ?> 
    34                 <a href="<?php echo str_replace('{page}', $total_pages, $url) ?>"><?php echo $total_pages ?></a> 
     32        <?php if ($p->current_page <= $p->total_pages - 3): ?> 
     33                <?php if ($p->current_page != $p->total_pages - 3) echo '…' ?> 
     34                <a href="<?php echo $p->url($p->total_pages) ?>"><?php echo $p->total_pages ?></a> 
    3535        <?php endif ?> 
    3636