Comments on: Debugging tip: “Disallowed Key Character” error in CodeIgniter

By: matthewfedak

matthewfedak — Mon, 16 Aug 2010 16:07:00 +0000

I just experienced this as I had a file name
and was missing the first quote.

By: kalodont

kalodont — Mon, 12 Jul 2010 13:27:53 +0000

This might be a bit off-topic, but might help someone seeking a solution for “Disallowed Key Character” in CI. The error may be also triggered by sending POST form data with POST array key names that contain “non-english” characters – I mean leters like ó or ń.

By: Glen Barnhardt

Glen Barnhardt — Fri, 02 Jul 2010 02:38:55 +0000

Hi Tiffany,

This error drove me nuts for quite some time. Even your suggestion didn't fix the problem. Here is what I did to find the issue.

In the core CI code in system/libraries is a file called input.php I made a small modification to that file so that it would show the actual data in question.

Around line number 199

function _clean_input_keys($str)
{
if ( ! preg_match("/^[a-z0-9:_/-]+$/i", $str))
{
exit('Disallowed Key Characters: '.$str); // Added the variable to display.
}

return $str;
}

I certainly hope this helps others.

By: tiffanybbrown

tiffanybbrown — Fri, 29 Jan 2010 23:27:56 +0000

Vlad, delete *anything* after the closing ?> — including white space and line breaks — or remove the ?>.

By: Vlad

Vlad — Mon, 25 Jan 2010 13:39:09 +0000

WOW………..anyone here talk English , like what button do I press to get rid off the annoying “Disallowed Key Characters” there is obviously a solution, I just haven't found one in English yet, only in ITist
I get it while replying on Mercatornet, and that seem to be the only sight.

By: anon

anon — Sat, 14 Nov 2009 09:14:43 +0000

I have experienced this error as well when I am in a hurry and forget to close my form tag

By: tiffanybbrown

tiffanybbrown — Mon, 02 Nov 2009 17:54:26 +0000

Thanks for restating what iBSpoof, manafta, and Ben all said above. White space errors are not CodeIgniter specific. PHP will display a "Cannot send header" error.

"Disallowed Key Character," however, IS the specific language CodeIgniter uses. That, in my opinion isn't as clear or familiar as the PHP error and can trip up users new to both. That's what this post is addressing.

By: Everett

Everett — Mon, 02 Nov 2009 17:16:10 +0000

It's not just CodeIgniter… this is a "feature" of PHP. It's best to leave off the closing tag unless it's required.

By: tiffany

tiffany — Tue, 13 May 2008 14:35:16 +0000

@ben: but but… i like the ?>. actually, if i unlearn the habit, i’d probably wind up with the converse problem. i’m weird like that.

still, dropping the ?> is a good way to prevent the problem. thanks for reiterating the point.

@markus: in CI, each key name in the data array becomes a variable. $data['foo'] in your controller becomes $foo in your view. using a foreach loop seems unnecessary to me, unless you’re returning multiple database rows.

By: Ben Ramsey

Ben Ramsey — Mon, 12 May 2008 04:42:37 +0000

Some others have already mentioned that the ?> closing tag is not required, but I wanted to reiterate the point. In fact, as you have found, it can cause errors and unintended effects, particularly in library code (such as in a framework), because stray characters (line breaks, spaces, etc.) after the ?> are echoed immediately. Thus, any code that sets a header later in the code (i.e. cookies, etc.) ends up being broken because headers cannot be set after content has been sent (echoed) to the client.

For this reason, it’s a good practice to leave out the trailing ?> closing tag when writing anything that’s not specifically a view/template.

You can get around this, though, by turning on output buffering.

By: Markus

Markus — Fri, 09 May 2008 18:21:05 +0000

While we’re on the topic of CI (but a little off your original topic)… Tiffany, your original post on PHP Frameworks really inspired me to take a long hard look at Frameworks, and I wanted to first thank you for opening my eyes. I too decided to go with CI (and have yet to look back). My question is on output techniques. I am curious about how you output data from your models. Currently I pass the entire result set to my view via my controller. I then run a foreach statement in my view to output the results. I have no logical reason why I do this, but something tells me that its not the most efficient and effective way to use CI. What do you think?

By: wenbert

wenbert — Thu, 08 May 2008 05:26:57 +0000

the closing tag ?> is not required. in fact, in Zend Framework, when doing .php files, it is encourage that you avoid it.

By: tiffany

tiffany — Wed, 07 May 2008 20:38:14 +0000

Thank you for pointing that out manafta. I’ve experienced this before with other PHP projects as well.

CodeIgniter, however, will output a specific “Disallowed Key Characters” error. If you don’t know that line breaks will cause the error, it can be tricky to track down. PHP will usually report that it can’t re-send headers.

I’ve posted it here so that other CI users can learn about this specific cause and cure.

By: manafta

manafta — Wed, 07 May 2008 19:15:30 +0000

The extra line break causes PHP to start outputting text (the line break). If your not supposed to output anything before a certain point, any application will fail. This is not codeigniter specific. Learned that the hard way.

By: tiffany

tiffany — Wed, 07 May 2008 17:59:32 +0000

It’s a force of coding habit, and a good one, IMO. The important thing to remember is that excess white space can cause CodeIgniter to chuck a wobbly.