Comments on: CodeIgniter simple CRUD improved http://www.marthijnvandenheuvel.com/2010/06/27/codeigniter-simple-crud-improved/ Fri, 03 Feb 2012 18:11:48 +0000 hourly 1 http://wordpress.org/?v=3.2.1 By: Marthijn http://www.marthijnvandenheuvel.com/2010/06/27/codeigniter-simple-crud-improved/comment-page-1/#comment-132 Marthijn Tue, 29 Jun 2010 15:27:28 +0000 http://www.marthijnvandenheuvel.com/?p=208#comment-132 The index function is a controller action indeed, it's the same index function as in Henri's version. And thanks for your uri segment tip, I didn't know that! The index function is a controller action indeed, it’s the same index function as in Henri’s version. And thanks for your uri segment tip, I didn’t know that!

]]> By: Stephen http://www.marthijnvandenheuvel.com/2010/06/27/codeigniter-simple-crud-improved/comment-page-1/#comment-131 Stephen Tue, 29 Jun 2010 15:22:00 +0000 http://www.marthijnvandenheuvel.com/?p=208#comment-131 Also, I should note that your sort URIs should be formatted as follows (under CodeIgniter convention): <pre><code>anchor('person/index/'.$offset.'/name/'.($order_type == 'asc' ? 'desc' : 'asc')</code></pre> This allows the function to automatically receive the arguments passed in the URI. You should of course initialize their values to defaults if they are empty. Also, I should note that your sort URIs should be formatted as follows (under CodeIgniter convention):

anchor('person/index/'.$offset.'/name/'.($order_type == 'asc' ? 'desc' : 'asc')

This allows the function to automatically receive the arguments passed in the URI. You should of course initialize their values to defaults if they are empty.

]]> By: Stephen http://www.marthijnvandenheuvel.com/2010/06/27/codeigniter-simple-crud-improved/comment-page-1/#comment-130 Stephen Tue, 29 Jun 2010 15:18:29 +0000 http://www.marthijnvandenheuvel.com/?p=208#comment-130 I assume that the '<code>index</code>' function is intended to be in a controller. You're doing lots of extra work with the <code>$this->uri->segment()</code> calls. The arguments of the function should already be filled in by segments 3 through 5. Assuming you have your URI configured properly in your configuration file. See: http://codeigniter.com/user_guide/general/controllers.html#passinguri I assume that the ‘index‘ function is intended to be in a controller. You’re doing lots of extra work with the $this->uri->segment() calls. The arguments of the function should already be filled in by segments 3 through 5. Assuming you have your URI configured properly in your configuration file.

See: http://codeigniter.com/user_guide/general/controllers.html#passinguri

]]> By: Tweets that mention CodeIgniter simple CRUD improved – marthijn. -- Topsy.com http://www.marthijnvandenheuvel.com/2010/06/27/codeigniter-simple-crud-improved/comment-page-1/#comment-126 Tweets that mention CodeIgniter simple CRUD improved – marthijn. -- Topsy.com Mon, 28 Jun 2010 20:21:33 +0000 http://www.marthijnvandenheuvel.com/?p=208#comment-126 [...] This post was mentioned on Twitter by ioros, Abdullah Al Mamun. Abdullah Al Mamun said: CodeIgniter simple CRUD improved – marthijn. http://bit.ly/a4HiAK #codeigniter. [...] [...] This post was mentioned on Twitter by ioros, Abdullah Al Mamun. Abdullah Al Mamun said: CodeIgniter simple CRUD improved – marthijn. http://bit.ly/a4HiAK #codeigniter. [...]

]]>