Skip to content

Commit

Permalink
Remove unused variable
Browse files Browse the repository at this point in the history
  • Loading branch information
m-kuhn committed Jan 7, 2018
1 parent 7605fde commit 8b60b4c
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 7 deletions.
5 changes: 1 addition & 4 deletions src/server/qgsfilterrestorer.h
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,7 @@ class SERVER_EXPORT QgsOWSServerFilterRestorer
{
public:

QgsOWSServerFilterRestorer( const QgsAccessControl *accessControl )
: mAccessControl( accessControl )
{}
QgsOWSServerFilterRestorer() = default;

//! Destructor. When object is destroyed all original layer filters will be restored.
~QgsOWSServerFilterRestorer()
Expand Down Expand Up @@ -72,7 +70,6 @@ class SERVER_EXPORT QgsOWSServerFilterRestorer
static void applyAccessControlLayerFilters( const QgsAccessControl *accessControl, QgsMapLayer *mapLayer );

private:
const QgsAccessControl *mAccessControl = nullptr;
QHash<QgsMapLayer *, QString> mOriginalLayerFilters;

};
Expand Down
2 changes: 1 addition & 1 deletion src/server/services/wfs/qgswfsgetfeature.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ namespace QgsWfs

//scoped pointer to restore all original layer filters (subsetStrings) when pointer goes out of scope
//there's LOTS of potential exit paths here, so we avoid having to restore the filters manually
std::unique_ptr< QgsOWSServerFilterRestorer > filterRestorer( new QgsOWSServerFilterRestorer( accessControl ) );
std::unique_ptr< QgsOWSServerFilterRestorer > filterRestorer( new QgsOWSServerFilterRestorer() );

// features counters
long sentFeatures = 0;
Expand Down
2 changes: 1 addition & 1 deletion src/server/services/wfs/qgswfstransaction.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ namespace QgsWfs

//scoped pointer to restore all original layer filters (subsetStrings) when pointer goes out of scope
//there's LOTS of potential exit paths here, so we avoid having to restore the filters manually
std::unique_ptr< QgsOWSServerFilterRestorer > filterRestorer( new QgsOWSServerFilterRestorer( accessControl ) );
std::unique_ptr< QgsOWSServerFilterRestorer > filterRestorer( new QgsOWSServerFilterRestorer() );

// get layers
QStringList wfsLayerIds = QgsServerProjectUtils::wfsLayerIds( *project );
Expand Down
2 changes: 1 addition & 1 deletion src/server/services/wfs/qgswfstransaction_1_0_0.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ namespace QgsWfs

//scoped pointer to restore all original layer filters (subsetStrings) when pointer goes out of scope
//there's LOTS of potential exit paths here, so we avoid having to restore the filters manually
std::unique_ptr< QgsOWSServerFilterRestorer > filterRestorer( new QgsOWSServerFilterRestorer( accessControl ) );
std::unique_ptr< QgsOWSServerFilterRestorer > filterRestorer( new QgsOWSServerFilterRestorer() );

// get layers
QStringList wfsLayerIds = QgsServerProjectUtils::wfsLayerIds( *project );
Expand Down
2 changes: 2 additions & 0 deletions src/server/services/wms/qgswmsrenderer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,9 @@ namespace QgsWms
const QgsProject *project,
const QgsServerRequest::Parameters &parameters )
: mParameters( parameters )
#ifdef HAVE_SERVER_PYTHON_PLUGINS
, mAccessControl( serverIface->accessControls() )
#endif
, mSettings( *serverIface->serverSettings() )
, mProject( project )
{
Expand Down
2 changes: 2 additions & 0 deletions src/server/services/wms/qgswmsrenderer.h
Original file line number Diff line number Diff line change
Expand Up @@ -286,8 +286,10 @@ namespace QgsWms

const QgsServerRequest::Parameters &mParameters;

#ifdef HAVE_SERVER_PYTHON_PLUGINS
//! The access control helper
QgsAccessControl *mAccessControl = nullptr;
#endif
QgsFeatureFilter mFeatureFilter;

const QgsServerSettings &mSettings;
Expand Down

0 comments on commit 8b60b4c

Please sign in to comment.