Lists expressed as strings should be properly escaped

Bug #394001 reported by Marc Tardif
6
This bug affects 1 person
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

Marc Tardif (cr3)
Changed in checkbox:
assignee: nobody → Marc Tardif (cr3)
importance: Undecided → High
milestone: none → 0.8-alpha3
status: New → In Progress
Marc Tardif (cr3)
Changed in checkbox:
status: In Progress → Fix Committed
Marc Tardif (cr3)
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.