SLD : an invalid sld is generated when exporting a polygon with fill styles

[qgib]

Author Name: Yewondwossen Assefa (Yewondwossen Assefa)
Original Redmine Issue: 8567
Affected QGIS version: master
Redmine category:symbology

---

This was initially reported on the mailing list:

• http://lists.osgeo.org/pipermail/qgis-user/2013-August/023650.html
• http://lists.osgeo.org/pipermail/qgis-developer/2013-August/028011.html

The sld produced is invalid at the GraphicFill level:

se:GraphicFill
se:Graphic/
se:Mark
se:WellKnownNamex</se:WellKnownName>
se:Stroke
<se:SvgParameter name="stroke">#0000ff</se:SvgParameter>
</se:Stroke>
</se:Mark>
</se:GraphicFill>

The Mark element sould be a child of the se:Graphic element and not se:GraphicFill:
se:GraphicFill
se:Graphic
se:Mark
se:WellKnownNamex</se:WellKnownName>
se:Stroke
<se:SvgParameter name="stroke">#0000ff</se:SvgParameter>
</se:Stroke>
</se:Mark>
</se:Graphic>
</se:GraphicFill>

The error comes from file ./src/core/symbology-ng/qgssymbollayerv2utils.cpp QgsSymbolLayerV2Utils::fillToSld (around line 1523).
Attached is the diff to correct it. This has been tested locally and gives the expected result.

---

• qgis_sld_error.diff (Yewondwossen Assefa)

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

Hi, thanks for the patch. Would you mind make a pull request on github? here patches are likely to be overlooked.

---

• operating_system was changed from all to
• status_id was changed from Open to Feedback
• category_id was configured as Symbology

[qgib]

Author Name: Giuseppe Sucameli (@brushtyler)

---

Fixed in changeset "4bc1beb844cef09b5b58570a4e402a5c422bc32c".

---

• status_id was changed from Feedback to Closed

[qgib]

Author Name: Giuseppe Sucameli (@brushtyler)

---

The patch is applied, thanks!
Please, the next time create a pull request on GitHub.

---

• resolution was changed from to fixed