Should have choices which type of pagination link to be returned |
|||
|---|---|---|---|
| Date: | 06/30/2008 | Severity: | Minor |
| Status: | New | Reporter: | Jeffrey04 |
| Version: | 1.6.3 | ||
| Keywords: | Libraries, Pagination Class | ||
Description
Besides the missing AMP constant, there is another small problem that I don’t know whether it is considered a bug.
I need to enable query string to ensure my YUI scripts work, however, I use static link most of the time. So I do not really want my pagination link in query string style.
Code Sample
// instead of using the following condition (line 173)
if ($CI->config->item('enable_query_strings') === TRUE OR $this->page_query_string === TRUE)
// isn't this better?
if ($CI->config->item('uri_protocol') == 'QUERY_STRING' OR $this->page_query_string === TRUE)
Expected Result
I am expected to see static link being outputted for pagination.
Actual Result
link with query strings returned instead, not forgetting there is an undefined constant AMP #4775
