| [Solved] Codeigniter Pagination Will Not Go Back Page |
I have got a List controller where I can sort the table thead links all working fine.
The problem is when I click on a pagination link and go to that page I click on it will not let me click and go back to a different page. $this->uri->segmet(5); is where I think issue is because it is the page number. Every thing else works fine it just will not let me go back to a different page when uri->segment(5) active. PHP Code: <?phpAny suggestion and example world be great I have been reading this http://www.codeigniter.com/user_guide/li...ation.html I also have attached the controller
There's only one rule - please don't tell anyone to go and read the manual. Sometimes the manual just SUCKS!
I can advice you to check what links are generated and if* there is something wrong, to search for the pattern which is causing it.
I see that you have many customisations at your code and I the problem should be some where inside .. Best VPS Hosting : Digital Ocean
I'm not sure whether I can find the problem, but I see a few places to start looking:
- Why is the constructor loading a library named 'paginations'? I'd imagine that if the library didn't exist, you'd get an error right away, but if it does exist, you don't appear to be using it in the controller. - You check your URI segments multiple times and handle them inconsistently throughout your index method. For example, $sort and $order are prefixed with a slash if the related URI segment does not exist, otherwise they are not prefixed with a slash. - Since you don't appear to be handling re-ordering of the URI segments, the $url variable should be built with all of the segments populated, even when they are not populated in the current URI, so it would probably be best to use the $sort and $order variables rather than checking segments 3 and 4 again. - After loading the pagination library, you check whether segment 3 == 'name' and set the $start variable accordingly with an if/else statement. Then you check segment 3 again (this time whether it's == 'status') and set the $start variable in another if/else statement. The if ($this->uri->segment(3) == 'status') block makes the if ($this->uri->segment(3) == 'name') block completely useless, so you should either remove the first if/else block, or fix the logic to reflect your intent (side note: this is one of the reasons people recommend commenting your code, so a developer can see the intent of the code from your comments and understand what the code should be doing). - After your weird dance with the session's flashdata method, you reset your $url variable (why?), then check whether $order == 'asc', even though your default value for $order is '/asc', and you add '/desc' to the $url if $order is 'asc' (but you would add '/asc' if $order is '/asc'), so whatever the intent here, you're likely to see some odd behavior. - Next you add the value of the 'page' $_GET variable to $url if it exists, even though I didn't see any other use of this variable in the controller, and it doesn't do anything to ensure the data is safe or otherwise appropriate to append to $url - After resetting $url again, you rebuild most of the original $url, then don't do anything with it. My recommendation would be to set a fixed use for each URI segment and set $config['uri_segment'] before initializing the pagination library so the page number is set to a fixed segment, then set the arguments on the index() method with default values, for example: PHP Code: public function index($sort = 'name', $order = 'asc', $start = 0)If you want to be able to omit the $sort and/or $order in the URL, then make the page segment 3: PHP Code: public function index($start = 0, $sort = 'name', $order = 'asc')Then, you won't have to juggle them around (since you need a $sort for $order to do anything useful, right?). If the page number absolutely has to be last, and the other items have to be optional, it just goes back to being more consistent with the handling of the URI segments. I would recommend just calling $this->uri->segment() once for each segment, then validating the values against the valid values, then setting $sort, $order, and $start accordingly. In your current code, $start appears to only be the value of segment(5) (if segment(3) == 'status' and segment(5) is truthy), the value of segment(3) (if segment(3) is not 'status' but is still truthy) or 0. However, your code is perfectly capable of generating a URI in which the $start value should be in segment(4) (if segment(4) does not contain an $order value but segment(3) contains a $sort value). In fact, when you go to /layout/lists, it seems that your pagination would generate a series of links in the form /layout/lists/$start, but your code would assign the page number to $sort and potentially generate some links on the new page which cause even more problems.
Thank you for advice now all working.
PHP Code: <?phpWhen I go and click on edit button and then press cancel button now it goes back to the correct page on pagination On the edit controller I need to use code below PHP Code: if ($this->uri->segment(5) == TRUE) {Lists View Table Example PHP Code: <table class="table table-bordered table-hover">
There's only one rule - please don't tell anyone to go and read the manual. Sometimes the manual just SUCKS!
|
| Welcome Guest, Not a member yet? Register Sign In |
