Processing: GRASS commands not escaped correctly

[qgib]

Author Name: Paolo Cavallini (@pcav)
Original Redmine Issue: 16104
Affected QGIS version: 2.18.2
Redmine category:processing/grass
Assignee: Victor Olaya

---

A simple GRASS command, e.g.

processing.runalg("grass7:v.clean","/home/paolo/Aree_Elementari.shp",2,0.1,"955403.9934,1054537.9351,4596890.3389,4678902.6515",-1,0.0001,None,None)

results in:

v.in.ogr min_area=0.0001 snap=-1 input="/home/paolo/" layer=Aree Elementari output=tmp14851690580813 --overwrite -o

instead of:

v.in.ogr min_area=0.0001 snap=-1 input="/home/paolo/" layer="Aree Elementari" output=tmp14851690580813 --overwrite -o

(the space in filename is not escaped) and as a consequence the command fails with:

ERROR: Sorry is not a valid option

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

If is regression is a severe, not high issue.

Anyway I cannot replicate (2.18.3, grass72), so please add

1. sample data
2. exact steps on how replicate

---

• status_id was changed from Open to Feedback

[qgib]

Author Name: Victor Olaya (@volaya)

---

Giovanni, i guess it will happen when you use any layer with blank spaces in its name as input

Should be easy to fix, i will work on it

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

Victor Olaya wrote:

Giovanni, i guess it will happen when you use any layer with blank spaces in its name as input

Should be easy to fix, i will work on it

Hi Victor,
yes it seems to happen when input layer name has spaces. But the above example does not have spaces in the input file name

processing.runalg("grass7:v.clean","/home/paolo/Aree_Elementari.shp",2,0.1,"955403.9934,1054537.9351,4596890.3389,4678902.6515",-1,0.0001,None,None)

and in another test using an input with underscores this are not replaced by spaces (as the above example would suggest).

[qgib]

Author Name: luca barbadoro (luca barbadoro)

---

• fixed_version_id was configured as Version 2.14

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

I believe Victor has already fixed this.

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

https://github.com/qgis/QGIS/pull/4046/files#diff-0

---

• resolution was changed from to fixed/implemented
• status_id was changed from Feedback to Closed