Skip to content

Commit 3f0902e

Browse files
committed
Move enum DevToolsNodeRole to Qgis namespace
1 parent bacbd2c commit 3f0902e

File tree

7 files changed

+28
-24
lines changed

7 files changed

+28
-24
lines changed

src/app/devtools/querylogger/qgsappquerylogger.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ void QgsAppQueryLogger::removeRequestRows( const QList<int> &rows )
141141

142142
for ( int row : std::as_const( res ) )
143143
{
144-
int popId = data( index( row, 0, QModelIndex() ), static_cast<int>( QgsDevToolsModelNode::Roles::RoleId ) ).toInt();
144+
int popId = data( index( row, 0, QModelIndex() ), static_cast<int>( Qgis::DevToolsNodeRole::Id ) ).toInt();
145145
mQueryGroups.remove( popId );
146146

147147
beginRemoveRows( QModelIndex(), row, row );
@@ -217,11 +217,11 @@ QVariant QgsAppQueryLogger::data( const QModelIndex &index, int role ) const
217217
switch ( role )
218218
{
219219
case Qt::DisplayRole:
220-
case static_cast<int>( QgsDevToolsModelNode::Roles::RoleElapsedTime ):
221-
case static_cast<int>( QgsDevToolsModelNode::Roles::RoleSort ):
222-
return node->data( static_cast<int>( QgsDevToolsModelNode::Roles::RoleElapsedTime ) );
220+
case static_cast<int>( Qgis::DevToolsNodeRole::ElapsedTime ):
221+
case static_cast<int>( Qgis::DevToolsNodeRole::Sort ):
222+
return node->data( static_cast<int>( Qgis::DevToolsNodeRole::ElapsedTime ) );
223223

224-
case static_cast<int>( QgsDevToolsModelNode::Roles::RoleMaximumTime ):
224+
case static_cast<int>( Qgis::DevToolsNodeRole::MaximumTime ):
225225
return mMaxCost;
226226

227227
default:

src/app/devtools/querylogger/qgsdatabasequeryloggernode.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ QVariant QgsDatabaseQueryLoggerQueryGroup::data( int role ) const
8888
case Qt::DisplayRole:
8989
return QStringLiteral( "%1 %2" ).arg( QString::number( mQueryId ), mSql );
9090

91-
case static_cast<int>( QgsDevToolsModelNode::Roles::RoleSort ):
91+
case static_cast<int>( Qgis::DevToolsNodeRole::Sort ):
9292
return mQueryId;
9393

9494
case Qt::ToolTipRole:
@@ -118,13 +118,13 @@ QVariant QgsDatabaseQueryLoggerQueryGroup::data( int role ) const
118118
#endif
119119
}
120120

121-
case static_cast<int>( Roles::RoleStatus ):
121+
case static_cast<int>( Qgis::DevToolsNodeRole::Status ):
122122
return static_cast<int>( mStatus );
123123

124-
case static_cast<int>( Roles::RoleId ):
124+
case static_cast<int>( Qgis::DevToolsNodeRole::Id ):
125125
return mQueryId;
126126

127-
case static_cast<int>( Roles::RoleElapsedTime ):
127+
case static_cast<int>( Qgis::DevToolsNodeRole::ElapsedTime ):
128128
return mElapsed;
129129

130130
case Qt::ForegroundRole:

src/app/devtools/querylogger/qgsqueryloggerpanelwidget.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ QgsDatabaseQueryLoggerTreeView::QgsDatabaseQueryLoggerTreeView( QgsAppQueryLogge
4848
setFont( QFontDatabase::systemFont( QFontDatabase::FixedFont ) );
4949

5050
mProxyModel = new QgsDatabaseQueryLoggerProxyModel( mLogger, this );
51-
mProxyModel->setSortRole( static_cast<int>( QgsDevToolsModelNode::Roles::RoleSort ) );
51+
mProxyModel->setSortRole( static_cast<int>( Qgis::DevToolsNodeRole::Sort ) );
5252
setModel( mProxyModel );
5353

5454
connect( mProxyModel, &QAbstractItemModel::rowsInserted, this, [this]( const QModelIndex &parent, int first, int last ) {
@@ -168,7 +168,7 @@ QgsDatabaseQueryLoggerPanelWidget::QgsDatabaseQueryLoggerPanelWidget( QgsAppQuer
168168
setupUi( this );
169169

170170
mTreeView = new QgsDatabaseQueryLoggerTreeView( mLogger );
171-
mTreeView->setItemDelegateForColumn( 1, new QueryCostDelegate( static_cast<int>( QgsDevToolsModelNode::Roles::RoleElapsedTime ), static_cast<int>( QgsDevToolsModelNode::Roles::RoleMaximumTime ), mTreeView ) );
171+
mTreeView->setItemDelegateForColumn( 1, new QueryCostDelegate( static_cast<int>( Qgis::DevToolsNodeRole::ElapsedTime ), static_cast<int>( Qgis::DevToolsNodeRole::MaximumTime ), mTreeView ) );
172172
mTreeView->setSortingEnabled( true );
173173
mTreeView->sortByColumn( 0, Qt::SortOrder::AscendingOrder );
174174

src/core/qgis.h

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6080,6 +6080,20 @@ class CORE_EXPORT Qgis
60806080
Q_DECLARE_FLAGS( RasterProcessingParameterCapabilities, RasterProcessingParameterCapability )
60816081
Q_FLAG( RasterProcessingParameterCapabilities )
60826082

6083+
/**
6084+
* Dev tools node custom data roles.
6085+
* \since QGIS 4.0
6086+
*/
6087+
enum class DevToolsNodeRole
6088+
{
6089+
Status = Qt::UserRole + 1, //!< Request status role
6090+
Id, //!< Request ID role
6091+
ElapsedTime, //!< Elapsed time
6092+
MaximumTime, //!< Maximum encountered elapsed time
6093+
Sort, //!< Sort order role
6094+
};
6095+
Q_ENUM( DevToolsNodeRole )
6096+
60836097
/**
60846098
* Identify search radius in mm
60856099
*/

src/gui/devtools/networklogger/qgsnetworklogger.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ void QgsNetworkLogger::removeRequestRows( const QList<int> &rows )
215215

216216
for ( int row : std::as_const( res ) )
217217
{
218-
int popId = data( index( row, 0, QModelIndex() ), static_cast<int>( QgsDevToolsModelNode::Roles::RoleId ) ).toInt();
218+
int popId = data( index( row, 0, QModelIndex() ), static_cast<int>( Qgis::DevToolsNodeRole::Id ) ).toInt();
219219
mRequestGroups.remove( popId );
220220

221221
beginRemoveRows( QModelIndex(), row, row );

src/gui/devtools/networklogger/qgsnetworkloggernode.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,10 +101,10 @@ QVariant QgsNetworkLoggerRequestGroup::data( int role ) const
101101
.arg( mUrl.url(), statusToString( mStatus ), QString::number( mHttpStatus ), mContentType, bytes, mStatus == Status::Pending ? QString::number( mTimer.elapsed() / 1000 ) : QString::number( mTotalTime ), QString::number( mReplies ) );
102102
}
103103

104-
case static_cast<int>( Roles::RoleStatus ):
104+
case static_cast<int>( Qgis::DevToolsNodeRole::Status ):
105105
return static_cast<int>( mStatus );
106106

107-
case static_cast<int>( Roles::RoleId ):
107+
case static_cast<int>( Qgis::DevToolsNodeRole::Id ):
108108
return mRequestId;
109109

110110
case Qt::ForegroundRole:

src/gui/devtools/qgsdevtoolsmodelnode.h

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -37,16 +37,6 @@ class QgsDevToolsModelGroup;
3737
class GUI_EXPORT QgsDevToolsModelNode
3838
{
3939
public:
40-
//! Custom node data roles
41-
enum class Roles
42-
{
43-
RoleStatus = Qt::UserRole + 1, //!< Request status role
44-
RoleId, //!< Request ID role
45-
RoleElapsedTime, //!< Elapsed time
46-
RoleMaximumTime, //!< Maximum encountered elapsed time
47-
RoleSort, //!< Sort order role
48-
};
49-
5040
virtual ~QgsDevToolsModelNode();
5141

5242
/**

0 commit comments

Comments
 (0)