CodeIgniter 2 and segment-based URLs with query strings

[eluser]Prophet[/eluser]
Hi all. I was a little frustrated to find that the old

Code:

$config['uri_protocol']    = "PATH_INFO";
$config['enable_query_strings'] = TRUE;

trick doesn't work in CI2. The reason being that Config:ite_url() has been improved since CI1 and takes your query string configuration into account.

Using this configuration in CI2 will give you URLs like www.example.com/?/ and www.example.com/?controller/method which I don't think is ideal.

I came up with a really simple fix. I don't know if it's the right thing to do, but it makes sense to me. In system/core/Config.php, change line 221 from:

Code:

if ($this->item('enable_query_strings') == FALSE)

to:

Code:

if ($this->item('enable_query_strings') == FALSE OR $this->item('uri_protocol') == 'PATH_INFO')

I recommend creating application/core/MY_Config.php and overriding the site_url() method to avoid overwriting your changes when you pull the latest CI2 source in.