Skip to content

Commit

Permalink
[oracle] More code modernization
Browse files Browse the repository at this point in the history
  • Loading branch information
nyalldawson committed Apr 22, 2018
1 parent 2273dfa commit 3032ab7
Show file tree
Hide file tree
Showing 8 changed files with 161 additions and 171 deletions.
146 changes: 73 additions & 73 deletions src/providers/oracle/qgsoracleconn.cpp

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions src/providers/oracle/qgsoracleconn.h
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ class QgsOracleConn : public QObject
static QString displayStringForWkbType( QgsWkbTypes::Type wkbType );
static QgsWkbTypes::Type wkbTypeFromDatabase( int gtype );

static QString databaseTypeFilter( QString alias, QString geomCol, QgsWkbTypes::Type wkbType );
static QString databaseTypeFilter( const QString &alias, QString geomCol, QgsWkbTypes::Type wkbType );

static QgsWkbTypes::Type wkbTypeFromGeomType( QgsWkbTypes::GeometryType geomType );

Expand All @@ -179,7 +179,7 @@ class QgsOracleConn : public QObject
static bool allowGeometrylessTables( const QString &connName );
static bool estimatedMetadata( const QString &connName );
static bool onlyExistingTypes( const QString &connName );
static void deleteConnection( QString connName );
static void deleteConnection( const QString &connName );
static QString databaseName( const QString &database, const QString &host, const QString &port );
static QString toPoolName( const QgsDataSourceUri &uri );

Expand Down
86 changes: 41 additions & 45 deletions src/providers/oracle/qgsoracledataitems.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,11 @@
QGISEXTERN bool deleteLayer( const QString &uri, QString &errCause );

// ---------------------------------------------------------------------------
QgsOracleConnectionItem::QgsOracleConnectionItem( QgsDataItem *parent, QString name, QString path )
QgsOracleConnectionItem::QgsOracleConnectionItem( QgsDataItem *parent, const QString &name, const QString &path )
: QgsDataCollectionItem( parent, name, path )
, mColumnTypeThread( nullptr )
{
mIconName = "mIconConnect.svg";
mIconName = QStringLiteral( "mIconConnect.svg" );
mCapabilities |= Collapse;
}

Expand Down Expand Up @@ -95,10 +95,10 @@ QVector<QgsDataItem *> QgsOracleConnectionItem::createChildren()
/* useEstimatedMetadata */ true,
QgsOracleConn::allowGeometrylessTables( mName ) );

connect( mColumnTypeThread, SIGNAL( setLayerType( QgsOracleLayerProperty ) ),
this, SLOT( setLayerType( QgsOracleLayerProperty ) ) );
connect( mColumnTypeThread, SIGNAL( started() ), this, SLOT( threadStarted() ) );
connect( mColumnTypeThread, SIGNAL( finished() ), this, SLOT( threadFinished() ) );
connect( mColumnTypeThread, &QgsOracleColumnTypeThread::setLayerType,
this, &QgsOracleConnectionItem::setLayerType );
connect( mColumnTypeThread, &QThread::started, this, &QgsOracleConnectionItem::threadStarted );
connect( mColumnTypeThread, &QThread::finished, this, &QgsOracleConnectionItem::threadFinished );

if ( QgsOracleRootItem::sMainWindow )
{
Expand All @@ -123,26 +123,26 @@ QVector<QgsDataItem *> QgsOracleConnectionItem::createChildren()

void QgsOracleConnectionItem::threadStarted()
{
QgsDebugMsgLevel( "Entering.", 3 );
QgsDebugMsgLevel( QStringLiteral( "Entering." ), 3 );
}

void QgsOracleConnectionItem::threadFinished()
{
QgsDebugMsgLevel( "Entering.", 3 );
QgsDebugMsgLevel( QStringLiteral( "Entering." ), 3 );
setAllAsPopulated();
}

void QgsOracleConnectionItem::setLayerType( QgsOracleLayerProperty layerProperty )
void QgsOracleConnectionItem::setLayerType( const QgsOracleLayerProperty &layerProperty )
{
QgsDebugMsgLevel( layerProperty.toString(), 3 );
QgsOracleOwnerItem *ownerItem = mOwnerMap.value( layerProperty.ownerName, 0 );
QgsOracleOwnerItem *ownerItem = mOwnerMap.value( layerProperty.ownerName, nullptr );

for ( int i = 0 ; i < layerProperty.size(); i++ )
{
QgsWkbTypes::Type wkbType = layerProperty.types.at( i );
if ( wkbType == QgsWkbTypes::Unknown )
{
QgsDebugMsgLevel( "skip unknown geometry type", 3 );
QgsDebugMsgLevel( QStringLiteral( "skip unknown geometry type" ), 3 );
continue;
}

Expand All @@ -155,7 +155,7 @@ void QgsOracleConnectionItem::setLayerType( QgsOracleLayerProperty layerProperty
mOwnerMap[ layerProperty.ownerName ] = ownerItem;
}

QgsDebugMsgLevel( "ADD LAYER", 3 );
QgsDebugMsgLevel( QStringLiteral( "ADD LAYER" ), 3 );
ownerItem->addLayer( layerProperty.at( i ) );
}
}
Expand All @@ -176,27 +176,27 @@ QList<QAction *> QgsOracleConnectionItem::actions( QWidget *parent )
QList<QAction *> lst;

QAction *actionRefresh = new QAction( tr( "Refresh" ), parent );
connect( actionRefresh, SIGNAL( triggered() ), this, SLOT( refreshConnection() ) );
connect( actionRefresh, &QAction::triggered, this, &QgsOracleConnectionItem::refreshConnection );
lst.append( actionRefresh );

QAction *separator = new QAction( parent );
separator->setSeparator( true );
lst.append( separator );

QAction *actionEdit = new QAction( tr( "Edit Connection…" ), parent );
connect( actionEdit, SIGNAL( triggered() ), this, SLOT( editConnection() ) );
connect( actionEdit, &QAction::triggered, this, &QgsOracleConnectionItem::editConnection );
lst.append( actionEdit );

QAction *actionDelete = new QAction( tr( "Delete Connection" ), parent );
connect( actionDelete, SIGNAL( triggered() ), this, SLOT( deleteConnection() ) );
connect( actionDelete, &QAction::triggered, this, &QgsOracleConnectionItem::deleteConnection );
lst.append( actionDelete );

return lst;
}

void QgsOracleConnectionItem::editConnection()
{
QgsOracleNewConnection nc( NULL, mName );
QgsOracleNewConnection nc( nullptr, mName );
if ( nc.exec() )
{
// the parent should be updated
Expand Down Expand Up @@ -238,7 +238,7 @@ bool QgsOracleConnectionItem::handleDrop( const QMimeData *data, Qt::DropAction
QgsMimeDataUtils::UriList lst = QgsMimeDataUtils::decodeUriList( data );
Q_FOREACH ( const QgsMimeDataUtils::Uri &u, lst )
{
if ( u.layerType != "vector" )
if ( u.layerType != QLatin1String( "vector" ) )
{
importResults.append( tr( "%1: Not a vector layer!" ).arg( u.name ) );
hasError = true; // only vectors can be imported
Expand All @@ -250,10 +250,10 @@ bool QgsOracleConnectionItem::handleDrop( const QMimeData *data, Qt::DropAction

if ( srcLayer->isValid() )
{
uri.setDataSource( QString(), u.name.left( 30 ).toUpper(), "GEOM" );
uri.setDataSource( QString(), u.name.left( 30 ).toUpper(), QStringLiteral( "GEOM" ) );
uri.setWkbType( srcLayer->wkbType() );
QString authid = srcLayer->crs().authid();
if ( authid.startsWith( "EPSG:", Qt::CaseInsensitive ) )
if ( authid.startsWith( QStringLiteral( "EPSG:" ), Qt::CaseInsensitive ) )
{
uri.setSrid( authid.mid( 5 ) );
}
Expand Down Expand Up @@ -301,32 +301,28 @@ bool QgsOracleConnectionItem::handleDrop( const QMimeData *data, Qt::DropAction
{
QgsMessageOutput *output = QgsMessageOutput::createMessageOutput();
output->setTitle( tr( "Import to Oracle database" ) );
output->setMessage( tr( "Failed to import some layers!\n\n" ) + importResults.join( "\n" ), QgsMessageOutput::MessageText );
output->setMessage( tr( "Failed to import some layers!\n\n" ) + importResults.join( '\n' ), QgsMessageOutput::MessageText );
output->showMessage();
}

return true;
}

// ---------------------------------------------------------------------------
QgsOracleLayerItem::QgsOracleLayerItem( QgsDataItem *parent, QString name, QString path, QgsLayerItem::LayerType layerType, QgsOracleLayerProperty layerProperty )
: QgsLayerItem( parent, name, path, QString(), layerType, "oracle" )
QgsOracleLayerItem::QgsOracleLayerItem( QgsDataItem *parent, const QString &name, const QString &path, QgsLayerItem::LayerType layerType, const QgsOracleLayerProperty &layerProperty )
: QgsLayerItem( parent, name, path, QString(), layerType, QStringLiteral( "oracle" ) )
, mLayerProperty( layerProperty )
{
mUri = createUri();
setState( Populated );
}

QgsOracleLayerItem::~QgsOracleLayerItem()
{
}

QList<QAction *> QgsOracleLayerItem::actions( QWidget *parent )
{
QList<QAction *> lst;

QAction *actionDeleteLayer = new QAction( tr( "Delete Table" ), parent );
connect( actionDeleteLayer, SIGNAL( triggered() ), this, SLOT( deleteLayer() ) );
connect( actionDeleteLayer, &QAction::triggered, this, &QgsOracleLayerItem::deleteLayer );
lst.append( actionDeleteLayer );

return lst;
Expand All @@ -343,23 +339,23 @@ void QgsOracleLayerItem::deleteLayer()
bool res = ::deleteLayer( mUri, errCause );
if ( !res )
{
QMessageBox::warning( 0, tr( "Delete Table" ), errCause );
QMessageBox::warning( nullptr, tr( "Delete Table" ), errCause );
}
else
{
QMessageBox::information( 0, tr( "Delete Table" ), tr( "Table deleted successfully." ) );
QMessageBox::information( nullptr, tr( "Delete Table" ), tr( "Table deleted successfully." ) );
deleteLater();
}
}

QString QgsOracleLayerItem::createUri()
{
Q_ASSERT( mLayerProperty.size() == 1 );
QgsOracleConnectionItem *connItem = qobject_cast<QgsOracleConnectionItem *>( parent() ? parent()->parent() : 0 );
QgsOracleConnectionItem *connItem = qobject_cast<QgsOracleConnectionItem *>( parent() ? parent()->parent() : nullptr );

if ( !connItem )
{
QgsDebugMsg( "connection item not found." );
QgsDebugMsg( QStringLiteral( "connection item not found." ) );
return QString();
}

Expand All @@ -369,36 +365,36 @@ QString QgsOracleLayerItem::createUri()
uri.setWkbType( mLayerProperty.types.at( 0 ) );
if ( mLayerProperty.isView && mLayerProperty.pkCols.size() > 0 )
uri.setKeyColumn( mLayerProperty.pkCols[0] );
QgsDebugMsgLevel( QString( "layer uri: %1" ).arg( uri.uri() ), 3 );
QgsDebugMsgLevel( QString( QStringLiteral( "layer uri: %1" ) ).arg( uri.uri() ), 3 );
return uri.uri();
}

// ---------------------------------------------------------------------------
QgsOracleOwnerItem::QgsOracleOwnerItem( QgsDataItem *parent, QString name, QString path )
QgsOracleOwnerItem::QgsOracleOwnerItem( QgsDataItem *parent, const QString &name, const QString &path )
: QgsDataCollectionItem( parent, name, path )
{
mIconName = "mIconDbOwner.png";
mIconName = QStringLiteral( "mIconDbOwner.png" );
//not fertile, since children are created by QgsOracleConnectionItem
mCapabilities &= ~( Fertile );
}

QVector<QgsDataItem *> QgsOracleOwnerItem::createChildren()
{
QgsDebugMsgLevel( "Entering.", 3 );
QgsDebugMsgLevel( QStringLiteral( "Entering." ), 3 );
return QVector<QgsDataItem *>();
}

QgsOracleOwnerItem::~QgsOracleOwnerItem()
{
}

void QgsOracleOwnerItem::addLayer( QgsOracleLayerProperty layerProperty )
void QgsOracleOwnerItem::addLayer( const QgsOracleLayerProperty &layerProperty )
{
QgsDebugMsgLevel( layerProperty.toString(), 3 );

Q_ASSERT( layerProperty.size() == 1 );
QgsWkbTypes::Type wkbType = layerProperty.types.at( 0 );
QString tip = tr( "%1 as %2 in %3" ).arg( layerProperty.geometryColName ).arg( QgsOracleConn::displayStringForWkbType( wkbType ) ).arg( layerProperty.srids.at( 0 ) );
QString tip = tr( "%1 as %2 in %3" ).arg( layerProperty.geometryColName, QgsOracleConn::displayStringForWkbType( wkbType ) ).arg( layerProperty.srids.at( 0 ) );

QgsLayerItem::LayerType layerType;
switch ( wkbType )
Expand Down Expand Up @@ -433,16 +429,16 @@ void QgsOracleOwnerItem::addLayer( QgsOracleLayerProperty layerProperty )
}
}

QgsOracleLayerItem *layerItem = new QgsOracleLayerItem( this, layerProperty.tableName, mPath + "/" + layerProperty.tableName, layerType, layerProperty );
QgsOracleLayerItem *layerItem = new QgsOracleLayerItem( this, layerProperty.tableName, mPath + '/' + layerProperty.tableName, layerType, layerProperty );
layerItem->setToolTip( tip );
addChildItem( layerItem, true );
}

// ---------------------------------------------------------------------------
QgsOracleRootItem::QgsOracleRootItem( QgsDataItem *parent, QString name, QString path )
QgsOracleRootItem::QgsOracleRootItem( QgsDataItem *parent, const QString &name, const QString &path )
: QgsDataCollectionItem( parent, name, path )
{
mIconName = "mIconOracle.svg";
mIconName = QStringLiteral( "mIconOracle.svg" );
populate();
}

Expand All @@ -455,7 +451,7 @@ QVector<QgsDataItem *> QgsOracleRootItem::createChildren()
QVector<QgsDataItem *> connections;
Q_FOREACH ( QString connName, QgsOracleConn::connectionList() )
{
connections << new QgsOracleConnectionItem( this, connName, mPath + "/" + connName );
connections << new QgsOracleConnectionItem( this, connName, mPath + '/' + connName );
}
return connections;
}
Expand All @@ -465,7 +461,7 @@ QList<QAction *> QgsOracleRootItem::actions( QWidget *parent )
QList<QAction *> lst;

QAction *actionNew = new QAction( tr( "New Connection…" ), parent );
connect( actionNew, SIGNAL( triggered() ), this, SLOT( newConnection() ) );
connect( actionNew, &QAction::triggered, this, &QgsOracleRootItem::newConnection );
lst.append( actionNew );

return lst;
Expand All @@ -474,7 +470,7 @@ QList<QAction *> QgsOracleRootItem::actions( QWidget *parent )
QWidget *QgsOracleRootItem::paramWidget()
{
QgsOracleSourceSelect *select = new QgsOracleSourceSelect();
connect( select, SIGNAL( connectionsChanged() ), this, SLOT( connectionsChanged() ) );
connect( select, &QgsAbstractDataSourceWidget::connectionsChanged, this, &QgsOracleRootItem::connectionsChanged );
return select;
}

Expand All @@ -485,14 +481,14 @@ void QgsOracleRootItem::connectionsChanged()

void QgsOracleRootItem::newConnection()
{
QgsOracleNewConnection nc( NULL );
QgsOracleNewConnection nc( nullptr );
if ( nc.exec() )
{
refreshConnections();
}
}

QMainWindow *QgsOracleRootItem::sMainWindow = 0;
QMainWindow *QgsOracleRootItem::sMainWindow = nullptr;

QGISEXTERN void registerGui( QMainWindow *mainWindow )
{
Expand Down
17 changes: 8 additions & 9 deletions src/providers/oracle/qgsoracledataitems.h
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@ class QgsOracleRootItem : public QgsDataCollectionItem
{
Q_OBJECT
public:
QgsOracleRootItem( QgsDataItem *parent, QString name, QString path );
~QgsOracleRootItem();
QgsOracleRootItem( QgsDataItem *parent, const QString &name, const QString &path );
~QgsOracleRootItem() override;

QVector<QgsDataItem *> createChildren() override;

Expand All @@ -58,8 +58,8 @@ class QgsOracleConnectionItem : public QgsDataCollectionItem
{
Q_OBJECT
public:
QgsOracleConnectionItem( QgsDataItem *parent, QString name, QString path );
~QgsOracleConnectionItem();
QgsOracleConnectionItem( QgsDataItem *parent, const QString &name, const QString &path );
~QgsOracleConnectionItem() override;

QVector<QgsDataItem *> createChildren() override;
virtual bool equal( const QgsDataItem *other ) override;
Expand All @@ -78,7 +78,7 @@ class QgsOracleConnectionItem : public QgsDataCollectionItem
void deleteConnection();
void refreshConnection();

void setLayerType( QgsOracleLayerProperty layerProperty );
void setLayerType( const QgsOracleLayerProperty &layerProperty );

void threadStarted();
void threadFinished();
Expand All @@ -94,21 +94,20 @@ class QgsOracleOwnerItem : public QgsDataCollectionItem
{
Q_OBJECT
public:
QgsOracleOwnerItem( QgsDataItem *parent, QString name, QString path );
QgsOracleOwnerItem( QgsDataItem *parent, const QString &name, const QString &path );
~QgsOracleOwnerItem();

QVector<QgsDataItem *> createChildren();

void addLayer( QgsOracleLayerProperty layerProperty );
void addLayer( const QgsOracleLayerProperty &layerProperty );
};

class QgsOracleLayerItem : public QgsLayerItem
{
Q_OBJECT

public:
QgsOracleLayerItem( QgsDataItem *parent, QString name, QString path, QgsLayerItem::LayerType layerType, QgsOracleLayerProperty layerProperties );
~QgsOracleLayerItem();
QgsOracleLayerItem( QgsDataItem *parent, const QString &name, const QString &path, QgsLayerItem::LayerType layerType, const QgsOracleLayerProperty &layerProperties );

QString createUri();

Expand Down
3 changes: 0 additions & 3 deletions src/providers/oracle/qgsoraclefeatureiterator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,6 @@

QgsOracleFeatureIterator::QgsOracleFeatureIterator( QgsOracleFeatureSource *source, bool ownSource, const QgsFeatureRequest &request )
: QgsAbstractFeatureIteratorFromSource<QgsOracleFeatureSource>( source, ownSource, request )
, mRewind( false )
, mExpressionCompiled( false )
, mFetchGeometry( false )
{
mConnection = QgsOracleConnPool::instance()->acquireConnection( QgsOracleConn::toPoolName( mSource->mUri ) );
if ( !mConnection )
Expand Down
6 changes: 3 additions & 3 deletions src/providers/oracle/qgsoraclefeatureiterator.h
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,9 @@ class QgsOracleFeatureIterator : public QgsAbstractFeatureIteratorFromSource<Qgs

QgsOracleConn *mConnection = nullptr;
QSqlQuery mQry;
bool mRewind;
bool mExpressionCompiled;
bool mFetchGeometry;
bool mRewind = false;
bool mExpressionCompiled = false;
bool mFetchGeometry = false;
QgsAttributeList mAttributeList;
QString mSql;
QVariantList mArgs;
Expand Down
Loading

0 comments on commit 3032ab7

Please sign in to comment.