How do I send data into a Code Igniter Controller index method?

Question

I tried to encode the ID of a journal in the URL of my Code Igniter application and the retrieve it in my controller. My end goal is to access the page http://mysite.com/journal/3 and get to access a page containing details about the journal with ID 3.

In my journal.php controller file, I have

<?php
if (!defined('BASEPATH'))
    exit('No direct script access allowed');

class Journal extends Controller {
    public function index($journalId) {
        $data['journalId'] = $journalId;
        $this->view->load('journalPage', $data);
    }
}
?>

In my journalPage.php view file, I have

This event has ID <?= $journalId ?>.

I wrote this rule in my routes.php file.

$route['journal/(:num)'] = 'journal/$1';

Here is the .htaccess file in my html public folder.

<IfModule mod_rewrite.c>
Options +FollowSymLinks
RewriteEngine on
RewriteBase /
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ index.php?$1 [QSA,L]
</IfModule>

However, when I go to mysite.com/journal/3, I get a 404 error. Why?

Do you have a .htaccess file set up? If not, you will need to access the site through its index page: mysite.com/index.php/journal/3 — rjz
– rjz, Commented Mar 19, 2012 at 21:15
I do have an .htaccess file. Controllers without data encoded in the URL (and whose methods do not accept parameters) are working. Also, neither mysite.com/journal/3 nor mysite.com/index.php/journal/3 are working. Interestingly, going to mysite.com/journal/ works. However, the text displayed is This event has ID . without an actual event ID. — John Hoffman
– John Hoffman, Commented Mar 19, 2012 at 21:17
Following on @safarov's line of thought, posting the relevant pieces of .htaccess and config/routes.php might help us make a better guess. — rjz
– rjz, Commented Mar 19, 2012 at 21:23
why do you want to encode the ID? All your doing is adding another step for php to proccess — Philip
– Philip, Commented Mar 19, 2012 at 21:30

Philip · Accepted Answer · 2012-03-19 21:39:25Z

3

$route['journal/(:num)'] = 'journal/index/$1';

(:num) will become invalid if you encode

Edit: If you use CI's encryption class to encode your ID(pointless) you will need to modify it to make sure the string is uri safe.

edited Mar 19, 2012 at 21:39

answered Mar 19, 2012 at 21:33

Philip

4,5822 gold badges22 silver badges28 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

John Hoffman Over a year ago

Thank you, I had forgotten "index" in the string 'journal/index/$1'.

NemC · Accepted Answer · 2012-03-19 22:13:14Z

Based on what I see above this is just begginers mistake on CI routing.

You .htaccess is ok (you are just removing index.php from url).

On other 2 steps (you have problem in your controller and in your routes config).

First in controller, when creating new controller you should extend CI_Controller To make story short, this is how your journal.php file should like like:

<?php
if (!defined('BASEPATH'))
    exit('No direct script access allowed');

class Journal extends CI_Controller {
    public function __construct() {
        parent::__construct(); // This is MUST!
    }
    public function index($journalId) {
        $data['journalId'] = $journalId;
        $this->view->load('journalPage', $data);
    }
}

Now when you have updated this, we come to routes config.

Config line you wrote only can confuse CI, nothing more, nothing less.

Structure of CI route shoud be like this:

$route['journal/(:any)'] = 'journal/index/$1';

This would redirect all traffic from journal/[ID] to controller named journal, to method named index with param [ID].

You must define index part in routing.

Try this, and everything should be working fine. Cheers.

Jonas Erik Barreto · Accepted Answer · 2012-04-09 19:05:35Z

First of all: You don't need to create a empty constructor (This is must [nothing!]). If you want to load some libraries, helpers or models at the beginning of all functions for that controller, so it is needed, otherwise it don't.

Other tip: Try to do not pass parameters on a controller function. You can catch and manipulate uri values with CodeIgniter's URI Class.

Just do this:

<?php
if (!defined('BASEPATH'))
    exit('No direct script access allowed');

class Journal extends CI_Controller
{
    public function index()
    {
        $data['journalId'] = $this->uri->segment(2);
        $this->view->load('journalPage', $data);
    }
}

You better take a look on the URI Class. It is quite simple a handy! http://codeigniter.com/user_guide/libraries/uri.html

PS: Don't need to fix anything on routes too.

Hug

Collectives™ on Stack Overflow

How do I send data into a Code Igniter Controller index method?

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related