FuelPHP Forums

Dregond Rahl

I don't know for sure if its a bug, but, while I was testing id's being passed in the url i tried symbols too like @%^* () and such, and it returns an error:

Notice!

ErrorException [ Notice ]: Undefined offset: 0

COREPATH/classes/router.php @ line 108:

107: // First port of call: request for a module?
108: if (\Fuel::module_exists($match->segments[0]))
109: &#123;

ErrorException [ Warning ]: preg_match() [function.preg-match]: Compilation failed: missing ) at offset 14

COREPATH/classes/route.php @ line 156:

155: 
156: if (preg_match('@^'.$route->search.'$@uD', $uri, $params) != false)
157: &#123;

Notice!

ErrorException [ Notice ]: Undefined offset: 0

COREPATH/classes/router.php @ line 108:

107: // First port of call: request for a module?
108: if (\Fuel::module_exists($match->segments[0]))
109: &#123;

of-course its right to give an error, that's what i expected and was hoping for, but shouldn't it handle these symbols differently like escape them ?

Jelmer Schreuder

Could you check if this solves it: https://github.com/fuel/core/commit/8d2e3656434c328723fd5ac33cd75a6df0142ae3

Dregond Rahl

Jelmer Schreuder wrote on Saturday 21st of May 2011:

Could you check if this solves it: https://github.com/fuel/core/commit/8d2e3656434c328723fd5ac33cd75a6df0142ae3

Yups seems to have solved it, Thanks.

FuelPHP Forums

Ask your question about FuelPHP in the appropriate forum, or help others by answering their questions.

Howdy, Stranger!

Categories

In this Discussion