Empty query does not get processed in query builder

[qgib]

Author Name: Leyan Ouyang (Leyan Ouyang)
Original Redmine Issue: 5641
Affected QGIS version: master

---

If the query text box is empty when clicking the OK button, the empty string is not used and evaluated. This means that if there was already a query existing, it is kept, instead of the expected result of clearing it. Validating an empty query should be equivalent to clearing the query.

This is due to a test made in QgsQueryBuilder::accept(), skipping the validation for an empty query. I do not see the point of this test, so I propose simply to remove it. If it is useful to avoid validation of an empty query for another reason, we should still validate it with mLayer->setSubsetString( "" );

Patch attached.

---

• empty_query.txt (Leyan Ouyang)

[qgib]

Author Name: Jürgen Fischer (@jef-n)

---

fixed in ed70b94

---

• resolution was changed from to fixed
• status_id was changed from Open to Closed