Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
fixed AStyle with astyle-all.sh
  • Loading branch information
luipir committed Feb 1, 2019
1 parent 82e48f9 commit 0e9af79
Show file tree
Hide file tree
Showing 15 changed files with 254 additions and 252 deletions.
Expand Up @@ -128,7 +128,7 @@ By default it will be generated.

void toSld( QDomDocument &doc, QDomElement &element ) const;
%Docstring
! Write ContrastEnhancement tags following SLD v1.0 specs
Write ContrastEnhancement tags following SLD v1.0 specs
SLD1.0 is limited to the parameters listed in:
https://docs.geoserver.org/stable/en/user/styling/sld/reference/rastersymbolizer.html#contrastenhancement
Btw only <Normalize> + vendor options are supported because there is no clear mapping
Expand Down
2 changes: 1 addition & 1 deletion python/core/auto_generated/raster/qgsrasterrenderer.sip.in
Expand Up @@ -115,7 +115,7 @@ Sets origin of min/max values

virtual void toSld( QDomDocument &doc, QDomElement &element, const QgsStringMap &props = QgsStringMap() ) const;
%Docstring
! Used from subclasses to create SLD Rule elements following SLD v1.0 specs
Used from subclasses to create SLD Rule elements following SLD v1.0 specs

.. versionadded:: 3.6
%End
Expand Down
6 changes: 3 additions & 3 deletions src/core/qgsmaplayer.cpp
Expand Up @@ -1394,7 +1394,7 @@ void QgsMapLayer::exportSldStyle( QDomDocument &doc, QString &errorMsg )

const QgsVectorLayer *vlayer = qobject_cast<const QgsVectorLayer *>( this );
QgsRasterLayer *rlayer = qobject_cast<QgsRasterLayer *>( this );
if (!vlayer && !rlayer)
if ( !vlayer && !rlayer )
{
errorMsg = tr( "Could not save symbology because:\n%1" )
.arg( QStringLiteral( "Non vector or raster layers are supported yet" ) );
Expand Down Expand Up @@ -1441,7 +1441,7 @@ void QgsMapLayer::exportSldStyle( QDomDocument &doc, QString &errorMsg )
props[ QStringLiteral( "scaleMaxDenom" ) ] = QString::number( mMaxScale );
}

if (vlayer)
if ( vlayer )
{
if ( !vlayer->writeSld( layerNode, myDocument, errorMsg, props ) )
{
Expand All @@ -1450,7 +1450,7 @@ void QgsMapLayer::exportSldStyle( QDomDocument &doc, QString &errorMsg )
}
}

if (rlayer)
if ( rlayer )
{
if ( !rlayer->writeSld( layerNode, myDocument, errorMsg, props ) )
{
Expand Down
2 changes: 1 addition & 1 deletion src/core/raster/qgscontrastenhancement.cpp
Expand Up @@ -383,7 +383,7 @@ void QgsContrastEnhancement::toSld( QDomDocument &doc, QDomElement &element ) co
return;

QString algName;
switch( contrastEnhancementAlgorithm() )
switch ( contrastEnhancementAlgorithm() )
{
case StretchToMinimumMaximum:
algName = QStringLiteral( "StretchToMinimumMaximum" );
Expand Down
3 changes: 2 additions & 1 deletion src/core/raster/qgscontrastenhancement.h
Expand Up @@ -159,7 +159,8 @@ class CORE_EXPORT QgsContrastEnhancement

void readXml( const QDomElement &elem );

/*! Write ContrastEnhancement tags following SLD v1.0 specs
/**
* Write ContrastEnhancement tags following SLD v1.0 specs
* SLD1.0 is limited to the parameters listed in:
* https://docs.geoserver.org/stable/en/user/styling/sld/reference/rastersymbolizer.html#contrastenhancement
* Btw only <Normalize> + vendor options are supported because there is no clear mapping
Expand Down
10 changes: 5 additions & 5 deletions src/core/raster/qgshillshaderenderer.cpp
Expand Up @@ -552,29 +552,29 @@ void QgsHillshadeRenderer::toSld( QDomDocument &doc, QDomElement &element, const

// look for RasterSymbolizer tag
QDomNodeList elements = element.elementsByTagName( QStringLiteral( "sld:RasterSymbolizer" ) );
if ( elements.size() == 0)
if ( elements.size() == 0 )
return;

// there SHOULD be only one
QDomElement rasterSymbolizerElem = elements.at(0).toElement();
QDomElement rasterSymbolizerElem = elements.at( 0 ).toElement();

// add Channel Selection tags (if band is not default 1)
// Need to insert channelSelection in the correct sequence as in SLD standard e.g.
// after opacity or geometry or as first element after sld:RasterSymbolizer
if ( band() != 1)
if ( band() != 1 )
{
QDomElement channelSelectionElem = doc.createElement( QStringLiteral( "sld:ChannelSelection" ) );
elements = rasterSymbolizerElem.elementsByTagName( QStringLiteral( "sld:Opacity" ) );
if ( elements.size() != 0 )
{
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at(0) );
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at( 0 ) );
}
else
{
elements = rasterSymbolizerElem.elementsByTagName( QStringLiteral( "sld:Geometry" ) );
if ( elements.size() != 0 )
{
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at(0) );
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at( 0 ) );
}
else
{
Expand Down
12 changes: 6 additions & 6 deletions src/core/raster/qgsmultibandcolorrenderer.cpp
Expand Up @@ -476,7 +476,7 @@ void QgsMultiBandColorRenderer::toSld( QDomDocument &doc, QDomElement &element,
return;

// there SHOULD be only one
QDomElement rasterSymbolizerElem = elements.at(0).toElement();
QDomElement rasterSymbolizerElem = elements.at( 0 ).toElement();

// add Channel Selection tags
// Need to insert channelSelection in the correct sequence as in SLD standard e.g.
Expand All @@ -485,14 +485,14 @@ void QgsMultiBandColorRenderer::toSld( QDomDocument &doc, QDomElement &element,
elements = rasterSymbolizerElem.elementsByTagName( QStringLiteral( "sld:Opacity" ) );
if ( elements.size() != 0 )
{
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at(0) );
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at( 0 ) );
}
else
{
elements = rasterSymbolizerElem.elementsByTagName( QStringLiteral( "sld:Geometry" ) );
if ( elements.size() != 0 )
{
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at(0) );
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at( 0 ) );
}
else
{
Expand All @@ -504,16 +504,16 @@ void QgsMultiBandColorRenderer::toSld( QDomDocument &doc, QDomElement &element,
QStringList tags;
tags << QStringLiteral( "sld:RedChannel" ) << QStringLiteral( "sld:GreenChannel" ) << QStringLiteral( "sld:BlueChannel" );

QList<QgsContrastEnhancement*> contrastEnhancements;
QList<QgsContrastEnhancement *> contrastEnhancements;
contrastEnhancements.append( mRedContrastEnhancement );
contrastEnhancements.append( mGreenContrastEnhancement );
contrastEnhancements.append( mBlueContrastEnhancement );

QList<int> bands = usesBands();
QList<int>::const_iterator bandIt = bands.constBegin();
for ( int tagCounter = 0 ; bandIt != bands.constEnd(); ++bandIt, ++tagCounter)
for ( int tagCounter = 0 ; bandIt != bands.constEnd(); ++bandIt, ++tagCounter )
{
if ( *bandIt < 0)
if ( *bandIt < 0 )
continue;

QDomElement channelElem = doc.createElement( tags[ tagCounter ] );
Expand Down
6 changes: 3 additions & 3 deletions src/core/raster/qgspalettedrasterrenderer.cpp
Expand Up @@ -254,11 +254,11 @@ void QgsPalettedRasterRenderer::toSld( QDomDocument &doc, QDomElement &element,

// look for RasterSymbolizer tag
QDomNodeList elements = element.elementsByTagName( QStringLiteral( "sld:RasterSymbolizer" ) );
if ( elements.size() == 0)
if ( elements.size() == 0 )
return;

// there SHOULD be only one
QDomElement rasterSymbolizerElem = elements.at(0).toElement();
QDomElement rasterSymbolizerElem = elements.at( 0 ).toElement();

// add Channel Selection tags
QDomElement channelSelectionElem = doc.createElement( QStringLiteral( "sld:ChannelSelection" ) );
Expand All @@ -284,7 +284,7 @@ void QgsPalettedRasterRenderer::toSld( QDomDocument &doc, QDomElement &element,
// e.g. <ColorMapEntry color="#EEBE2F" quantity="-300" label="label" opacity="0"/>
QList<QgsPalettedRasterRenderer::Class> classes = this->classes();
QList<QgsPalettedRasterRenderer::Class>::const_iterator classDataIt = classes.constBegin();
for (; classDataIt != classes.constEnd(); ++classDataIt )
for ( ; classDataIt != classes.constEnd(); ++classDataIt )
{
QDomElement colorMapEntryElem = doc.createElement( QStringLiteral( "sld:ColorMapEntry" ) );
colorMapElem.appendChild( colorMapEntryElem );
Expand Down
40 changes: 20 additions & 20 deletions src/core/raster/qgsrasterlayer.cpp
Expand Up @@ -1348,15 +1348,15 @@ bool QgsRasterLayer::writeSld( QDomNode &node, QDomDocument &doc, QString &error
if ( hasScaleBasedVisibility() )
{
// note that denominator is the inverted value of scale
if ( maximumScale() != 0.0)
if ( maximumScale() != 0.0 )
{
QDomElement minScaleElem = doc.createElement( QStringLiteral( "sld:MinScaleDenominator" ) );
minScaleElem.appendChild( doc.createTextNode( QString::number( maximumScale() ) ) );
typeStyleRuleElem.appendChild( minScaleElem );
}

QDomElement maxScaleElem = doc.createElement( QStringLiteral( "sld:MaxScaleDenominator" ) );
maxScaleElem.appendChild( doc.createTextNode( QString::number( minimumScale() ) ) );
maxScaleElem.appendChild( doc.createTextNode( QString::number( minimumScale() ) ) );
typeStyleRuleElem.appendChild( maxScaleElem );
}

Expand All @@ -1370,10 +1370,10 @@ bool QgsRasterLayer::writeSld( QDomNode &node, QDomDocument &doc, QString &error
if ( elements.size() != 0 )
{
// there SHOULD be only one
QDomElement rasterSymbolizerElem = elements.at(0).toElement();
QDomElement rasterSymbolizerElem = elements.at( 0 ).toElement();

// lamda helper used below to reduce code redundancy
auto vendorOptionWriter = [&](QString name, QString value)
auto vendorOptionWriter = [&]( QString name, QString value )
{
QDomElement vendorOptionElem = doc.createElement( QStringLiteral( "sld:VendorOption" ) );
vendorOptionElem.setAttribute( QStringLiteral( "name" ), name );
Expand All @@ -1385,19 +1385,19 @@ bool QgsRasterLayer::writeSld( QDomNode &node, QDomDocument &doc, QString &error
if ( hueSaturationFilter()->grayscaleMode() != QgsHueSaturationFilter::GrayscaleOff )
{
QString property;
switch( hueSaturationFilter()->grayscaleMode() )
switch ( hueSaturationFilter()->grayscaleMode() )
{
case (QgsHueSaturationFilter::GrayscaleLightness):
property = QStringLiteral( "lightness" );
break;
case (QgsHueSaturationFilter::GrayscaleLuminosity):
property = QStringLiteral( "luminosity" );
break;
case (QgsHueSaturationFilter::GrayscaleAverage):
property = QStringLiteral( "average" );
break;
default:
break;
case ( QgsHueSaturationFilter::GrayscaleLightness ):
property = QStringLiteral( "lightness" );
break;
case ( QgsHueSaturationFilter::GrayscaleLuminosity ):
property = QStringLiteral( "luminosity" );
break;
case ( QgsHueSaturationFilter::GrayscaleAverage ):
property = QStringLiteral( "average" );
break;
default:
break;
}
if ( !property.isEmpty() )
vendorOptionWriter( QStringLiteral( "grayScale" ), property );
Expand All @@ -1421,7 +1421,7 @@ bool QgsRasterLayer::writeSld( QDomNode &node, QDomDocument &doc, QString &error
{
// normlize value [-100:100] -> [0:1]
int s = hueSaturationFilter()->saturation();
double sF = ( s - (-100.0) ) / (100.0 - (-100.0) );
double sF = ( s - ( -100.0 ) ) / ( 100.0 - ( -100.0 ) );
vendorOptionWriter( QStringLiteral( "saturation" ), QString::number( sF ) );
}
}
Expand All @@ -1431,7 +1431,7 @@ bool QgsRasterLayer::writeSld( QDomNode &node, QDomDocument &doc, QString &error
{
// normalize value [-255:255] -> [0:1]
int b = brightnessFilter()->brightness();
double bF = ( b - (-255.0) ) / (255.0 - (-255.0) );
double bF = ( b - ( -255.0 ) ) / ( 255.0 - ( -255.0 ) );
vendorOptionWriter( QStringLiteral( "brightness" ), QString::number( bF ) );
}

Expand All @@ -1440,8 +1440,8 @@ bool QgsRasterLayer::writeSld( QDomNode &node, QDomDocument &doc, QString &error
{
// normlize value [-100:100] -> [0:1]
int c = brightnessFilter()->contrast();
double cF = ( c - (-100.0) ) / (100.0 - (-100.0) );
vendorOptionWriter( QStringLiteral( "contrast" ), QString::number(cF ) );
double cF = ( c - ( -100.0 ) ) / ( 100.0 - ( -100.0 ) );
vendorOptionWriter( QStringLiteral( "contrast" ), QString::number( cF ) );
}

// TODO: check if the below mapping formula make sense to map QGIS contrast with SLD gamma value
Expand Down
3 changes: 2 additions & 1 deletion src/core/raster/qgsrasterrenderer.h
Expand Up @@ -113,7 +113,8 @@ class CORE_EXPORT QgsRasterRenderer : public QgsRasterInterface
//! Sets origin of min/max values
void setMinMaxOrigin( const QgsRasterMinMaxOrigin &origin ) { mMinMaxOrigin = origin; }

/*! Used from subclasses to create SLD Rule elements following SLD v1.0 specs
/**
* Used from subclasses to create SLD Rule elements following SLD v1.0 specs
* \since QGIS 3.6 */
virtual void toSld( QDomDocument &doc, QDomElement &element, const QgsStringMap &props = QgsStringMap() ) const;

Expand Down
28 changes: 14 additions & 14 deletions src/core/raster/qgssinglebandgrayrenderer.cpp
Expand Up @@ -228,11 +228,11 @@ void QgsSingleBandGrayRenderer::toSld( QDomDocument &doc, QDomElement &element,

// look for RasterSymbolizer tag
QDomNodeList elements = element.elementsByTagName( QStringLiteral( "sld:RasterSymbolizer" ) );
if ( elements.size() == 0)
if ( elements.size() == 0 )
return;

// there SHOULD be only one
QDomElement rasterSymbolizerElem = elements.at(0).toElement();
QDomElement rasterSymbolizerElem = elements.at( 0 ).toElement();

// add Channel Selection tags
// Need to insert channelSelection in the correct sequence as in SLD standard e.g.
Expand All @@ -241,14 +241,14 @@ void QgsSingleBandGrayRenderer::toSld( QDomDocument &doc, QDomElement &element,
elements = rasterSymbolizerElem.elementsByTagName( QStringLiteral( "sld:Opacity" ) );
if ( elements.size() != 0 )
{
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at(0) );
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at( 0 ) );
}
else
{
elements = rasterSymbolizerElem.elementsByTagName( QStringLiteral( "sld:Geometry" ) );
if ( elements.size() != 0 )
{
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at(0) );
rasterSymbolizerElem.insertAfter( channelSelectionElem, elements.at( 0 ) );
}
else
{
Expand Down Expand Up @@ -289,12 +289,12 @@ void QgsSingleBandGrayRenderer::toSld( QDomDocument &doc, QDomElement &element,
QDomNodeList elements = contrastEnhancementElem.elementsByTagName( QStringLiteral( "sld:VendorOption" ) );
for ( int i = 0; i < elements.size(); ++i )
{
QDomElement vendorOption = elements.at(i).toElement();
if ( vendorOption.attribute( QStringLiteral( "name" ) ) != QStringLiteral( "minValue" ))
QDomElement vendorOption = elements.at( i ).toElement();
if ( vendorOption.attribute( QStringLiteral( "name" ) ) != QStringLiteral( "minValue" ) )
continue;

// remove old value and add the new one
vendorOption.removeChild(vendorOption.firstChild());
vendorOption.removeChild( vendorOption.firstChild() );
vendorOption.appendChild( doc.createTextNode( QString::number( myRasterBandStats.minimumValue ) ) );
}
}
Expand Down Expand Up @@ -325,27 +325,27 @@ void QgsSingleBandGrayRenderer::toSld( QDomDocument &doc, QDomElement &element,
// each ContrastEnhancementAlgorithm need a specfic management.
// set type of ColorMap ramp [ramp, intervals, values]
// basing on interpolation algorithm of the raster shader
QList< QPair< QString, QColor > > colorMapping(classes);
QList< QPair< QString, QColor > > colorMapping( classes );
switch ( contrastEnhancement()->contrastEnhancementAlgorithm() )
{
case ( QgsContrastEnhancement::StretchAndClipToMinimumMaximum ):
case ( QgsContrastEnhancement::ClipToMinimumMaximum ):
{
QString lowValue = classes[0].first;
QColor lowColor = classes[0].second;
lowColor.setAlpha(0);
lowColor.setAlpha( 0 );
QString highValue = classes[1].first;
QColor highColor = classes[1].second;
highColor.setAlpha(0);
highColor.setAlpha( 0 );

colorMapping.prepend( QPair< QString, QColor >( lowValue, lowColor));
colorMapping.append( QPair< QString, QColor >( highValue, highColor));
colorMapping.prepend( QPair< QString, QColor >( lowValue, lowColor ) );
colorMapping.append( QPair< QString, QColor >( highValue, highColor ) );
break;
}
case ( QgsContrastEnhancement::NoEnhancement ):
break;
case ( QgsContrastEnhancement::StretchToMinimumMaximum ):
// use default
// use default
default:
{
colorMapping[0].first = QStringLiteral( "0" );
Expand All @@ -356,7 +356,7 @@ void QgsSingleBandGrayRenderer::toSld( QDomDocument &doc, QDomElement &element,

// create tags
QList< QPair< QString, QColor > >::ConstIterator it;
for ( it = colorMapping.begin(); it != colorMapping.constEnd() ; ++it)
for ( it = colorMapping.begin(); it != colorMapping.constEnd() ; ++it )
{
// set low level color mapping
QDomElement lowColorMapEntryElem = doc.createElement( QStringLiteral( "sld:ColorMapEntry" ) );
Expand Down

0 comments on commit 0e9af79

Please sign in to comment.