If svg files with params do not have a default value set, then don't

reset the fill/border color and border width when changing svg
marker/svg fill SVG files

This change makes the behaviour consistent between the svg marker
symbol and the other marker symbols. Additionally, svg files
which have customisable colors and NO default values set will
be shaded in gray fill/black outline in the svg selector widget,
to follow the same behaviour as the other marker symbol selectors.

Note that this change has NO EFFECT unless the svg files are
modified to remove the default param value, so there will be no
change for users' custom symbols. A follow up commit will need
to remove the default param values from the preinstalled SVG
files though. If you want to test in the meantime, I've modified
just the first two symbols in the accomodation group to make
this change for testing.

(refs #10908)

Author: nyalldawson

images/svg/accommodation/accommodation_alpinehut.svg

@@ -88,6 +88,25 @@ class QgsSvgCache : QObject
     void containsParams( const QString& path, bool& hasFillParam, QColor& defaultFillColor, bool& hasOutlineParam, QColor& defaultOutlineColor, bool& hasOutlineWidthParam,
                          double& defaultOutlineWidth ) const;
 
+    /** Tests if an svg file contains parameters for fill, outline color, outline width. If yes, possible default values are returned. If there are several
+     * default values in the svg file, only the first one is considered.
+     * @param path path to SVG file
+     * @param hasFillParam will be true if fill param present in SVG
+     * @param hasDefaultFillParam will be true if fill param has a default value specified
+     * @param defaultFillColor will be set to default fill color specified in SVG, if present
+     * @param hasOutlineParam will be true if outline param present in SVG
+     * @param hasDefaultOutlineColor will be true if outline param has a default value specified
+     * @param defaultOutlineColor will be set to default outline color specified in SVG, if present
+     * @param hasOutlineWidthParam will be true if outline width param present in SVG
+     * @param hasDefaultOutlineWidth will be true if outline width param has a default value specified
+     * @param defaultOutlineWidth will be set to default outline width specified in SVG, if present
+     * @note available in python bindings as containsParamsV2
+     * @note added in QGIS 2.12
+     */
+    void containsParams( const QString& path, bool& hasFillParam, bool& hasDefaultFillParam, QColor& defaultFillColor,
+                         bool& hasOutlineParam, bool& hasDefaultOutlineColor, QColor& defaultOutlineColor,
+                         bool& hasOutlineWidthParam, bool& hasDefaultOutlineWidth, double& defaultOutlineWidth ) const /PyName=containsParamsV2/;
+
     /** Get image data*/
     QByteArray getImageData( const QString &path ) const;
 

images/svg/accommodation/accommodation_bed_and_breakfast.svg

@@ -1712,6 +1712,9 @@ QgsSVGFillSymbolLayer::QgsSVGFillSymbolLayer( const QString& svgFilePath, double
   setSvgFilePath( svgFilePath );
   mOutlineWidth = 0.3;
   mAngle = angle;
+  mSvgFillColor = QColor( 0, 0, 0 );
+  mSvgOutlineColor = QColor( 0, 0, 0 );
+  mSvgOutlineWidth = 0.3;
   setDefaultSvgParams();
   mSvgPattern = 0;
 }
@@ -1725,6 +1728,9 @@ QgsSVGFillSymbolLayer::QgsSVGFillSymbolLayer( const QByteArray& svgData, double
   storeViewBox();
   mOutlineWidth = 0.3;
   mAngle = angle;
+  mSvgFillColor = QColor( 0, 0, 0 );
+  mSvgOutlineColor = QColor( 0, 0, 0 );
+  mSvgOutlineWidth = 0.3;
   setSubSymbol( new QgsLineSymbolV2() );
   setDefaultSvgParams();
   mSvgPattern = 0;
@@ -2224,31 +2230,28 @@ void QgsSVGFillSymbolLayer::storeViewBox()
 
 void QgsSVGFillSymbolLayer::setDefaultSvgParams()
 {
-  //default values
-  mSvgFillColor = QColor( 0, 0, 0 );
-  mSvgOutlineColor = QColor( 0, 0, 0 );
-  mSvgOutlineWidth = 0.3;
-
   if ( mSvgFilePath.isEmpty() )
   {
     return;
   }
 
   bool hasFillParam, hasOutlineParam, hasOutlineWidthParam;
+  bool hasDefaultFillColor, hasDefaultOutlineColor, hasDefaultOutlineWidth;
   QColor defaultFillColor, defaultOutlineColor;
   double defaultOutlineWidth;
-  QgsSvgCache::instance()->containsParams( mSvgFilePath, hasFillParam, defaultFillColor, hasOutlineParam, defaultOutlineColor, hasOutlineWidthParam,
-      defaultOutlineWidth );
+  QgsSvgCache::instance()->containsParams( mSvgFilePath, hasFillParam, hasDefaultFillColor, defaultFillColor,
+      hasOutlineParam, hasDefaultOutlineColor, defaultOutlineColor,
+      hasOutlineWidthParam, hasDefaultOutlineWidth, defaultOutlineWidth );
 
-  if ( hasFillParam )
+  if ( hasDefaultFillColor )
   {
     mSvgFillColor = defaultFillColor;
   }
-  if ( hasOutlineParam )
+  if ( hasDefaultOutlineColor )
   {
     mSvgOutlineColor = defaultOutlineColor;
   }
-  if ( hasOutlineWidthParam )
+  if ( hasDefaultOutlineWidth )
   {
     mSvgOutlineWidth = defaultOutlineWidth;
   }

python/core/symbology-ng/qgssvgcache.sip

@@ -1070,17 +1070,20 @@ QgsSymbolLayerV2* QgsSvgMarkerSymbolLayerV2::create( const QgsStringMap& props )
   {
     QColor fillColor, outlineColor;
     double outlineWidth;
-    bool hasFillParam, hasOutlineParam, hasOutlineWidthParam;
-    QgsSvgCache::instance()->containsParams( name, hasFillParam, fillColor, hasOutlineParam, outlineColor, hasOutlineWidthParam, outlineWidth );
-    if ( hasFillParam )
+    bool hasFillParam = false, hasOutlineParam = false, hasOutlineWidthParam = false;
+    bool hasDefaultFillColor = false, hasDefaultOutlineColor = false, hasDefaultOutlineWidth = false;
+    QgsSvgCache::instance()->containsParams( name, hasFillParam, hasDefaultFillColor, fillColor,
+        hasOutlineParam, hasDefaultOutlineColor, outlineColor,
+        hasOutlineWidthParam, hasDefaultOutlineWidth, outlineWidth );
+    if ( hasDefaultFillColor )
     {
       m->setFillColor( fillColor );
     }
-    if ( hasOutlineParam )
+    if ( hasDefaultOutlineColor )
     {
       m->setOutlineColor( outlineColor );
     }
-    if ( hasOutlineWidthParam )
+    if ( hasDefaultOutlineWidth )
     {
       m->setOutlineWidth( outlineWidth );
     }
@@ -1163,17 +1166,20 @@ void QgsSvgMarkerSymbolLayerV2::setPath( const QString& path )
   mPath = path;
   QColor fillColor, outlineColor;
   double outlineWidth;
-  bool hasFillParam, hasOutlineParam, hasOutlineWidthParam;
-  QgsSvgCache::instance()->containsParams( path, hasFillParam, fillColor, hasOutlineParam, outlineColor, hasOutlineWidthParam, outlineWidth );
-  if ( hasFillParam )
+  bool hasFillParam = false, hasOutlineParam = false, hasOutlineWidthParam = false;
+  bool hasDefaultFillColor = false, hasDefaultOutlineColor = false, hasDefaultOutlineWidth = false;
+  QgsSvgCache::instance()->containsParams( path, hasFillParam, hasDefaultFillColor, fillColor,
+      hasOutlineParam, hasDefaultOutlineColor, outlineColor,
+      hasOutlineWidthParam, hasDefaultOutlineWidth, outlineWidth );
+  if ( hasDefaultFillColor )
   {
     setFillColor( fillColor );
   }
-  if ( hasOutlineParam )
+  if ( hasDefaultOutlineColor )
   {
     setOutlineColor( outlineColor );
   }
-  if ( hasOutlineWidthParam )
+  if ( hasDefaultOutlineWidth )
   {
     setOutlineWidth( outlineWidth );
   }

src/core/symbology-ng/qgsfillsymbollayerv2.cpp

@@ -226,22 +226,42 @@ QgsSvgCacheEntry* QgsSvgCache::insertSVG( const QString& file, double size, cons
 
 void QgsSvgCache::containsParams( const QString& path, bool& hasFillParam, QColor& defaultFillColor, bool& hasOutlineParam, QColor& defaultOutlineColor,
                                   bool& hasOutlineWidthParam, double& defaultOutlineWidth ) const
+{
+  bool hasDefaultFillColor = false;
+  bool hasDefaultOutlineColor = false;
+  bool hasDefaultOutlineWidth = false;
+
+  containsParams( path, hasFillParam, hasDefaultFillColor, defaultFillColor,
+                  hasOutlineParam, hasDefaultOutlineColor, defaultOutlineColor,
+                  hasOutlineWidthParam, hasDefaultOutlineWidth, defaultOutlineWidth );
+}
+
+void QgsSvgCache::containsParams( const QString& path,
+                                  bool& hasFillParam, bool& hasDefaultFillParam, QColor& defaultFillColor,
+                                  bool& hasOutlineParam, bool& hasDefaultOutlineColor, QColor& defaultOutlineColor,
+                                  bool& hasOutlineWidthParam, bool& hasDefaultOutlineWidth, double& defaultOutlineWidth ) const
 {
   hasFillParam = false;
   hasOutlineParam = false;
   hasOutlineWidthParam = false;
-  defaultFillColor = QColor( Qt::black );
+  defaultFillColor = QColor( Qt::white );
   defaultOutlineColor = QColor( Qt::black );
   defaultOutlineWidth = 0.2;
 
+  hasDefaultFillParam = false;
+  hasDefaultOutlineColor = false;
+  hasDefaultOutlineWidth = false;
+
   QDomDocument svgDoc;
   if ( !svgDoc.setContent( getImageData( path ) ) )
   {
     return;
   }
 
   QDomElement docElem = svgDoc.documentElement();
-  containsElemParams( docElem, hasFillParam, defaultFillColor, hasOutlineParam, defaultOutlineColor, hasOutlineWidthParam, defaultOutlineWidth );
+  containsElemParams( docElem, hasFillParam, hasDefaultFillParam, defaultFillColor,
+                      hasOutlineParam, hasDefaultOutlineColor, defaultOutlineColor,
+                      hasOutlineWidthParam, hasDefaultOutlineWidth, defaultOutlineWidth );
 }
 
 void QgsSvgCache::replaceParamsAndCacheSvg( QgsSvgCacheEntry* entry )
@@ -632,8 +652,8 @@ void QgsSvgCache::replaceElemParams( QDomElement& elem, const QColor& fill, cons
   }
 }
 
-void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillParam, QColor& defaultFill, bool& hasOutlineParam, QColor& defaultOutline,
-                                      bool& hasOutlineWidthParam, double& defaultOutlineWidth ) const
+void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillParam, bool& hasDefaultFill, QColor& defaultFill, bool& hasOutlineParam, bool& hasDefaultOutline, QColor& defaultOutline,
+                                      bool& hasOutlineWidthParam, bool& hasDefaultOutlineWidth, double& defaultOutlineWidth ) const
 {
   if ( elem.isNull() )
   {
@@ -675,6 +695,7 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
           if ( valueSplit.size() > 1 )
           {
             defaultFill = QColor( valueSplit.at( 1 ) );
+            hasDefaultFill = true;
           }
         }
         else if ( !hasOutlineParam && value.startsWith( "param(outline)" ) )
@@ -683,6 +704,7 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
           if ( valueSplit.size() > 1 )
           {
             defaultOutline = QColor( valueSplit.at( 1 ) );
+            hasDefaultOutline = true;
           }
         }
         else if ( !hasOutlineWidthParam && value.startsWith( "param(outline-width)" ) )
@@ -691,6 +713,7 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
           if ( valueSplit.size() > 1 )
           {
             defaultOutlineWidth = valueSplit.at( 1 ).toDouble();
+            hasDefaultOutlineWidth = true;
           }
         }
       }
@@ -705,6 +728,7 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
         if ( valueSplit.size() > 1 )
         {
           defaultFill = QColor( valueSplit.at( 1 ) );
+          hasDefaultFill = true;
         }
       }
       else if ( !hasOutlineParam && value.startsWith( "param(outline)" ) )
@@ -713,6 +737,7 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
         if ( valueSplit.size() > 1 )
         {
           defaultOutline = QColor( valueSplit.at( 1 ) );
+          hasDefaultOutline = true;
         }
       }
       else if ( !hasOutlineWidthParam && value.startsWith( "param(outline-width)" ) )
@@ -721,6 +746,7 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
         if ( valueSplit.size() > 1 )
         {
           defaultOutlineWidth = valueSplit.at( 1 ).toDouble();
+          hasDefaultOutlineWidth = true;
         }
       }
     }
@@ -732,7 +758,9 @@ void QgsSvgCache::containsElemParams( const QDomElement& elem, bool& hasFillPara
   for ( int i = 0; i < nChildren; ++i )
   {
     QDomElement childElem = childList.at( i ).toElement();
-    containsElemParams( childElem, hasFillParam, defaultFill, hasOutlineParam, defaultOutline, hasOutlineWidthParam, defaultOutlineWidth );
+    containsElemParams( childElem, hasFillParam, hasDefaultFill, defaultFill,
+                        hasOutlineParam, hasDefaultOutline, defaultOutline,
+                        hasOutlineWidthParam, hasDefaultOutlineWidth, defaultOutlineWidth );
   }
 }