Merge pull request #9763 from rldhont/sld-point-symbolizer-on-polygon…

…-centroid-not-all-parts Apply SLD PointSymbolizer on single centroid for polygon
qgis · Apr 12, 2019 · aa297b5 · aa297b5
2 parents a0a8e49 + 052b222
commit aa297b5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/core/symbology/qgsfillsymbollayer.cpp b/src/core/symbology/qgsfillsymbollayer.cpp
@@ -3512,6 +3512,7 @@ QgsSymbolLayer *QgsCentroidFillSymbolLayer::createFromSld( QDomElement &element
 
   std::unique_ptr< QgsCentroidFillSymbolLayer > sl = qgis::make_unique< QgsCentroidFillSymbolLayer >();
   sl->setSubSymbol( marker.release() );
+  sl->setPointOnAllParts( false );
   return sl.release();
 }
 

diff --git a/tests/src/python/test_qgssymbollayer.py b/tests/src/python/test_qgssymbollayer.py
@@ -698,6 +698,11 @@ def testQgsCentroidFillSymbolLayer(self):
         mMessage = 'Expected "%s" got "%s"' % (mExpectedValue, mValue)
         assert mExpectedValue == mValue, mMessage
 
+        mExpectedValue = False
+        mValue = mSymbolLayer.pointOnAllParts()
+        mMessage = 'Expected "%s" got "%s"' % (mExpectedValue, mValue)
+        assert mExpectedValue == mValue, mMessage
+
         # test colors, need to make sure colors are passed/retrieved from subsymbol
         mSymbolLayer.setColor(QColor(150, 50, 100))
         self.assertEqual(mSymbolLayer.color(), QColor(150, 50, 100))