xdebugOutputFormat regexp is too restrictive

[GoogleCodeExporter]

the current outputname is determined as follows:
$outputName = '/^'.preg_replace('/(%[^%])+/', '[a-zA-Z0-9%_-~=]+',
$outputName).'$/';

However; this is far too restrictive; my xdebug.profiler_output_name is set
to 'profile.%R'; which means that the filename can include all characters
that are allowed in a URL; such as ~ and =. I suggest changing the match to
simply .

Original issue reported on code.google.com by Sjon.Hor...@gmail.com on 9 Jul 2008 at 10:11

[GoogleCodeExporter]

Changed in r126

Original comment by gugakf...@gmail.com on 17 Jul 2008 at 10:57

• Changed state: Fixed

[GoogleCodeExporter]

Now to permissive. It makes the preprocessed data files appear in the trace 
list. It should specifically exclude 
.webgrind files. Alternatively we could change prepend the preprocessed files 
with something.

Original comment by oett...@gmail.com on 18 Jul 2008 at 10:13

• Changed state: Started

[GoogleCodeExporter]

That's odd, it works for me. 

Changing from `[a-zA-Z0-9%_-~=]+` to `.+` shouldn't have any effect on whether 
.webgrind files are 
included...

Are you looking into this?

Original comment by gugakf...@gmail.com on 18 Jul 2008 at 3:33

[GoogleCodeExporter]

Obviously the .webgrind files are included because dots are now allowed in the 
filename. Never occured to me 
because my storageDir was in fact different from the xdebug storagedir. 

Fixed in r130

Original comment by gugakf...@gmail.com on 1 Aug 2008 at 9:40

• Changed state: Fixed