Lists expressed as strings should be properly escaped
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox |
Fix Released
|
High
|
Marc Tardif |
Bug Description
Suite names could have spaces in their names. So, when specifying the following whitelist:
Sound And Video, gedit chains
This gets interpreted as:
['Sound', 'And', 'Video', 'gedit', 'chains']
The problem is that the current regular expression for separating the parts of a list is: \s*,?\s+. The motivation was to support both space and comma separated lists.
One solution might be to change the regular expression to: \s*,\s+. This would only support comma separated lists which would address the above use case. However, this would only postpone the innevitable case where a suite name will have a comma in its name.
Another solution would be to support proper escaping. This would make it possible to remain backward compatible with the current support for space separated lists. Plus, this would make it possible to escape spaces when they should not be interpreted as a separator. For example, the above example could be expressed as:
Sound\ And\ Video gedit\ chains
Related branches
Changed in checkbox: | |
assignee: | nobody → Marc Tardif (cr3) |
importance: | Undecided → High |
milestone: | none → 0.8-alpha3 |
status: | New → In Progress |
Changed in checkbox: | |
status: | In Progress → Fix Committed |
Changed in checkbox: | |
status: | Fix Committed → Fix Released |