Allow underscore in dynamic variables for reports

[kirean@…]

Not really sure if it's a bug or a change request.

background

In 0.11 there are examples using underscore in state names. We have a state called in_QA. I'm using the timingandestimationplugin and unfortunately it uses the state name as a dynamic variable name. Thus all reports fails with: Report execution failed: Dynamic variable '$IN' not defined.

Simple example

Create a report like:

select $IN_QA

Call it like report:x?$IN_QA='test' and it will fail with error: Report execution failed: Dynamic variable '$IN' not defined.

Solution

coderanger suggested changing source:trunk/trac/ticket/report.py#L490​ which for me worked fine with: re.compile("[$]([A-Z_]+)") (Not sure though that this is the correct one to use)

Could something like this be adapted to "main Trac"?

[Remy Blank]

This is related to #6354, which asks for digits in report variables. I'll implement both.

[Remy Blank]

Implemented in [7632].