sharpetronics/QGIS
1
0
Fork 0
mirror of https://github.com/qgis/QGIS.git synced 2025-04-14 00:07:35 -04:00
Code Issues Packages Projects Releases Wiki Activity

Move management of avoidIntersectionLayers to QgsProject

Browse Source
This commit is contained in:
Matthias Kuhn 2016-10-18 16:27:30 +02:00
parent 5c919fbcca
commit 85c105d6b0
9 changed files with 72 additions and 51 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
python/core/qgsproject.sip
View File

@ -390,6 +390,20 @@ class QgsProject : QObject
*/
void setSnappingConfig( const QgsSnappingConfig& snappingConfig );
/**
* A list of layers with which intersections should be avoided.
*
* @note Added in QGIS 3.0
*/
QStringList avoidIntersectionsList() const;
/**
* A list of layers with which intersections should be avoided.
*
* @note Added in QGIS 3.0
*/
void setAvoidIntersectionsList(const QStringList& avoidIntersectionsList);
signals:
//! emitted when project is being read
void readProject( const QDomDocument & );
@ -462,6 +476,13 @@ class QgsProject : QObject
*/
void topologicalEditingChanged();
/**
* Emitted whenever avoidIntersectionsList has changed.
*
* @note Added in QGIS 3.0
*/
void avoidIntersectionsListChanged();
public slots:
/**
* Flag the project as dirty (modified). If this flag is set, the user will

9
python/core/qgssnappingconfig.sip
View File

@ -41,9 +41,8 @@ class QgsSnappingConfig
* @param type
* @param tolerance
* @param units
* @param avoidIntersection
*/
IndividualLayerSettings( bool enabled, QgsSnappingConfig::SnappingType type, double tolerance, QgsTolerance::UnitType units, bool avoidIntersection = false );
IndividualLayerSettings( bool enabled, QgsSnappingConfig::SnappingType type, double tolerance, QgsTolerance::UnitType units );
/**
* Constructs an invalid setting
@ -77,12 +76,6 @@ class QgsSnappingConfig
//! set the type of units
void setUnits( QgsTolerance::UnitType units );
//! return if it shall avoid intersection (polygon layers only)
bool avoidIntersection() const;
//! set if it shall avoid intersection (polygon layers only)
void setAvoidIntersection( bool avoidIntersection );
/**
* Compare this configuration to other.
*/

2
src/app/qgsmaptooladdfeature.cpp
View File

@ -296,7 +296,7 @@ void QgsMapToolAddFeature::cadCanvasReleaseEvent( QgsMapMouseEvent* e )
//use always topological editing for avoidIntersection.
//Otherwise, no way to guarantee the geometries don't have a small gap in between.
QStringList intersectionLayers = QgsProject::instance()->readListEntry( "Digitizing", "/AvoidIntersectionsList" );
QStringList intersectionLayers = QgsProject::instance()->avoidIntersectionsList();
bool avoidIntersection = !intersectionLayers.isEmpty();
if ( avoidIntersection ) //try to add topological points also to background layers
{

18
src/app/qgssnappinglayertreemodel.cpp
View File

@ -494,7 +494,7 @@ QVariant QgsSnappingLayerTreeModel::data( const QModelIndex& idx, int role ) con
{
if ( role == Qt::CheckStateRole && vl->geometryType() == QgsWkbTypes::PolygonGeometry )
{
if ( ls.avoidIntersection() )
if ( mProject->avoidIntersectionsList().contains( vl->id() ) )
{
return Qt::Checked;
}
@ -614,20 +614,20 @@ bool QgsSnappingLayerTreeModel::setData( const QModelIndex& index, const QVarian
if ( index.column() == AvoidIntersectionColumn && role == Qt::CheckStateRole )
{
QgsVectorLayer *vl = vectorLayer( index );
QgsVectorLayer* vl = vectorLayer( index );
if ( vl )
{
if ( !mIndividualLayerSettings.contains( vl ) )
return false;
QgsSnappingConfig::IndividualLayerSettings ls = mIndividualLayerSettings.value( vl );
if ( !ls.valid() )
return false;
QStringList avoidIntersectionsList = mProject->avoidIntersectionsList();
ls.setAvoidIntersection( value.toInt() == Qt::Checked );
QgsSnappingConfig config = mProject->snappingConfig();
config.setIndividualLayerSettings( vl, ls );
mProject->setSnappingConfig( config );
if ( value.toInt() == Qt::Checked && !avoidIntersectionsList.contains( vl->id() ) )
avoidIntersectionsList.append( vl->id() );
else
avoidIntersectionsList.removeAll( vl->id() );
mProject->setAvoidIntersectionsList( avoidIntersectionsList );
return true;
}
}

6
src/core/geometry/qgsgeometryeditutils.cpp
View File

@ -241,10 +241,8 @@ QgsAbstractGeometry* QgsGeometryEditUtils::avoidIntersections( const QgsAbstract
return nullptr;
}
//read avoid intersections list from project properties
bool listReadOk;
QStringList avoidIntersectionsList = QgsProject::instance()->readListEntry( "Digitizing", "/AvoidIntersectionsList", QStringList(), &listReadOk );
if ( !listReadOk )
QStringList avoidIntersectionsList = QgsProject::instance()->avoidIntersectionsList();
if ( avoidIntersectionsList.isEmpty() )
return nullptr; //no intersections stored in project does not mean error
QList< QgsAbstractGeometry* > nearGeometries;

11
src/core/qgsproject.cpp
View File

@ -1002,6 +1002,17 @@ void QgsProject::loadEmbeddedNodes( QgsLayerTreeGroup *group )
}
}
QStringList QgsProject::avoidIntersectionsList() const
{
return readListEntry( "Digitizing", "/AvoidIntersectionsList", QStringList() );
}
void QgsProject::setAvoidIntersectionsList( const QStringList& avoidIntersectionsList )
{
writeEntry( "Digitizing", "/AvoidIntersectionsList", avoidIntersectionsList );
emit avoidIntersectionsListChanged();
}
QgsExpressionContext QgsProject::createExpressionContext() const
{
QgsExpressionContext context;

22
src/core/qgsproject.h
View File

@ -80,6 +80,7 @@ class CORE_EXPORT QgsProject : public QObject, public QgsExpressionContextGenera
Q_PROPERTY( QgsCoordinateReferenceSystem crs READ crs WRITE setCrs )
Q_PROPERTY( QgsMapThemeCollection* mapThemeCollection READ mapThemeCollection )
Q_PROPERTY( QgsSnappingConfig snappingConfig READ snappingConfig WRITE setSnappingConfig NOTIFY snappingConfigChanged )
Q_PROPERTY( QStringList avoidIntersectionsList READ avoidIntersectionsList WRITE setAvoidIntersectionsList NOTIFY avoidIntersectionsListChanged )
public:
@ -469,6 +470,20 @@ class CORE_EXPORT QgsProject : public QObject, public QgsExpressionContextGenera
*/
void setSnappingConfig( const QgsSnappingConfig& snappingConfig );
/**
* A list of layers with which intersections should be avoided.
*
* @note Added in QGIS 3.0
*/
QStringList avoidIntersectionsList() const;
/**
* A list of layers with which intersections should be avoided.
*
* @note Added in QGIS 3.0
*/
void setAvoidIntersectionsList( const QStringList& avoidIntersectionsList );
signals:
//! emitted when project is being read
void readProject( const QDomDocument& );
@ -543,6 +558,13 @@ class CORE_EXPORT QgsProject : public QObject, public QgsExpressionContextGenera
*/
void topologicalEditingChanged();
/**
* Emitted whenever avoidIntersectionsList has changed.
*
* @note Added in QGIS 3.0
*/
void avoidIntersectionsListChanged();
public slots:
/**
* Flag the project as dirty (modified). If this flag is set, the user will

24
src/core/qgssnappingconfig.cpp
View File

@ -30,17 +30,15 @@ QgsSnappingConfig::IndividualLayerSettings::IndividualLayerSettings()
, mType( Vertex )
, mTolerance( 0 )
, mUnits( QgsTolerance::Pixels )
, mAvoidIntersection( false )
{}
QgsSnappingConfig::IndividualLayerSettings::IndividualLayerSettings( bool enabled, SnappingType type, double tolerance, QgsTolerance::UnitType units, bool avoidIntersection )
QgsSnappingConfig::IndividualLayerSettings::IndividualLayerSettings( bool enabled, SnappingType type, double tolerance, QgsTolerance::UnitType units )
: mValid( true )
, mEnabled( enabled )
, mType( type )
, mTolerance( tolerance )
, mUnits( units )
, mAvoidIntersection( avoidIntersection )
{}
bool QgsSnappingConfig::IndividualLayerSettings::valid() const
@ -88,24 +86,13 @@ void QgsSnappingConfig::IndividualLayerSettings::setUnits( QgsTolerance::UnitTyp
mUnits = units;
}
bool QgsSnappingConfig::IndividualLayerSettings::avoidIntersection() const
{
return mAvoidIntersection;
}
void QgsSnappingConfig::IndividualLayerSettings::setAvoidIntersection( bool avoidIntersection )
{
mAvoidIntersection = avoidIntersection;
}
bool QgsSnappingConfig::IndividualLayerSettings::operator !=( const QgsSnappingConfig::IndividualLayerSettings& other ) const
{
return mValid != other.mValid
|| mEnabled != other.mEnabled
|| mType != other.mType
|| mTolerance != other.mTolerance
|| mUnits != other.mUnits
|| mAvoidIntersection != other.mAvoidIntersection;
|| mUnits != other.mUnits;
}
bool QgsSnappingConfig::IndividualLayerSettings::operator ==( const QgsSnappingConfig::IndividualLayerSettings& other ) const
@ -114,8 +101,7 @@ bool QgsSnappingConfig::IndividualLayerSettings::operator ==( const QgsSnappingC
&& mEnabled == other.mEnabled
&& mType == other.mType
&& mTolerance == other.mTolerance
&& mUnits == other.mUnits
&& mAvoidIntersection == other.mAvoidIntersection;
&& mUnits == other.mUnits;
}
@ -347,7 +333,6 @@ void QgsSnappingConfig::readProject( const QDomDocument& doc )
SnappingType type = ( SnappingType )settingElement.attribute( "type" ).toInt();
double tolerance = settingElement.attribute( "tolerance" ).toDouble();
QgsTolerance::UnitType units = ( QgsTolerance::UnitType )settingElement.attribute( "units" ).toInt();
bool avoidIntersection = settingElement.attribute( "avoid-intersection" ) == "1";
QgsMapLayer* ml = QgsMapLayerRegistry::instance()->mapLayer( layerId );
if ( !ml || ml->type() != QgsMapLayer::VectorLayer )
@ -355,7 +340,7 @@ void QgsSnappingConfig::readProject( const QDomDocument& doc )
QgsVectorLayer* vl = qobject_cast<QgsVectorLayer*>( ml );
IndividualLayerSettings setting = IndividualLayerSettings( enabled, type, tolerance, units, avoidIntersection );
IndividualLayerSettings setting = IndividualLayerSettings( enabled, type, tolerance, units );
mIndividualLayerSettings.insert( vl, setting );
}
}
@ -382,7 +367,6 @@ void QgsSnappingConfig::writeProject( QDomDocument& doc )
layerElement.setAttribute( "type", ( int )setting.type() );
layerElement.setAttribute( "tolerance", setting.tolerance() );
layerElement.setAttribute( "units", ( int )setting.units() );
layerElement.setAttribute( "avoid-intersection", QString::number( setting.avoidIntersection() ) );
ilsElement.appendChild( layerElement );
}
snapSettingsElem.appendChild( ilsElement );

10
src/core/qgssnappingconfig.h
View File

@ -62,9 +62,8 @@ class CORE_EXPORT QgsSnappingConfig
* @param type
* @param tolerance
* @param units
* @param avoidIntersection
*/
IndividualLayerSettings( bool enabled, QgsSnappingConfig::SnappingType type, double tolerance, QgsTolerance::UnitType units, bool avoidIntersection = false );
IndividualLayerSettings( bool enabled, QgsSnappingConfig::SnappingType type, double tolerance, QgsTolerance::UnitType units );
/**
* Constructs an invalid setting
@ -98,12 +97,6 @@ class CORE_EXPORT QgsSnappingConfig
//! set the type of units
void setUnits( QgsTolerance::UnitType units );
//! return if it shall avoid intersection (polygon layers only)
bool avoidIntersection() const;
//! set if it shall avoid intersection (polygon layers only)
void setAvoidIntersection( bool avoidIntersection );
/**
* Compare this configuration to other.
*/
@ -117,7 +110,6 @@ class CORE_EXPORT QgsSnappingConfig
SnappingType mType;
double mTolerance;
QgsTolerance::UnitType mUnits;
bool mAvoidIntersection;
};
/**