Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[processing] Port matrix widget wrapper to newer c++ API
Allows matrix parameters to be correctly set for model child algorithms

Fixes #20914
  • Loading branch information
nyalldawson committed Feb 5, 2019
1 parent c6f3d5f commit eeff02f
Show file tree
Hide file tree
Showing 8 changed files with 598 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/gui/CMakeLists.txt
Expand Up @@ -199,6 +199,7 @@ SET(QGIS_GUI_SRCS
processing/qgsprocessingalgorithmdialogbase.cpp
processing/qgsprocessingconfigurationwidgets.cpp
processing/qgsprocessingguiregistry.cpp
processing/qgsprocessingmatrixparameterdialog.cpp
processing/qgsprocessingmodelerparameterwidget.cpp
processing/qgsprocessingrecentalgorithmlog.cpp
processing/qgsprocessingtoolboxmodel.cpp
Expand Down Expand Up @@ -741,6 +742,7 @@ SET(QGIS_GUI_MOC_HDRS
processing/qgsprocessingalgorithmconfigurationwidget.h
processing/qgsprocessingalgorithmdialogbase.h
processing/qgsprocessingconfigurationwidgets.h
processing/qgsprocessingmatrixparameterdialog.h
processing/qgsprocessingmodelerparameterwidget.h
processing/qgsprocessingrecentalgorithmlog.h
processing/qgsprocessingtoolboxmodel.h
Expand Down
1 change: 1 addition & 0 deletions src/gui/processing/qgsprocessingguiregistry.cpp
Expand Up @@ -34,6 +34,7 @@ QgsProcessingGuiRegistry::QgsProcessingGuiRegistry()
addParameterWidgetFactory( new QgsProcessingDistanceWidgetWrapper() );
addParameterWidgetFactory( new QgsProcessingRangeWidgetWrapper() );
addParameterWidgetFactory( new QgsProcessingAuthConfigWidgetWrapper() );
addParameterWidgetFactory( new QgsProcessingMatrixWidgetWrapper() );
}

QgsProcessingGuiRegistry::~QgsProcessingGuiRegistry()
Expand Down
192 changes: 192 additions & 0 deletions src/gui/processing/qgsprocessingmatrixparameterdialog.cpp
@@ -0,0 +1,192 @@
/***************************************************************************
qgsprocessingmatrixparameterdialog.cpp
------------------------------------
Date : February 2019
Copyright : (C) 2019 Nyall Dawson
Email : nyall dot dawson at gmail dot com
***************************************************************************
* *
* This program is free software; you can redistribute it and/or modify *
* it under the terms of the GNU General Public License as published by *
* the Free Software Foundation; either version 2 of the License, or *
* (at your option) any later version. *
* *
***************************************************************************/

#include "qgsprocessingmatrixparameterdialog.h"
#include "qgsgui.h"
#include <QStandardItemModel>
#include <QStandardItem>
#include <QPushButton>
#include <QLineEdit>
#include <QToolButton>

///@cond NOT_STABLE

QgsProcessingMatrixParameterDialog::QgsProcessingMatrixParameterDialog( QWidget *parent, Qt::WindowFlags flags, const QgsProcessingParameterMatrix *param, const QVariantList &initialTable )
: QDialog( parent, flags )
, mParam( param )
{
setupUi( this );

QgsGui::enableAutoGeometryRestore( this );

mTblView->setSelectionBehavior( QAbstractItemView::SelectRows );
mTblView->setSelectionMode( QAbstractItemView::ExtendedSelection );

mButtonAdd = new QPushButton( tr( "Add Row" ) );
mButtonBox->addButton( mButtonAdd, QDialogButtonBox::ActionRole );

mButtonRemove = new QPushButton( tr( "Remove Row(s)" ) );
mButtonBox->addButton( mButtonRemove, QDialogButtonBox::ActionRole );

mButtonRemoveAll = new QPushButton( tr( "Remove All" ) );
mButtonBox->addButton( mButtonRemoveAll, QDialogButtonBox::ActionRole );

connect( mButtonAdd, &QPushButton::clicked, this, &QgsProcessingMatrixParameterDialog::addRow );
connect( mButtonRemove, &QPushButton::clicked, this, &QgsProcessingMatrixParameterDialog::deleteRow );
connect( mButtonRemoveAll, &QPushButton::clicked, this, &QgsProcessingMatrixParameterDialog::deleteAllRows );

if ( param && param->hasFixedNumberRows() )
{
mButtonAdd->setEnabled( false );
mButtonRemove->setEnabled( false );
mButtonRemoveAll->setEnabled( false );
}

populateTable( initialTable );
}

QVariantList QgsProcessingMatrixParameterDialog::table() const
{
const int cols = mModel->columnCount();
const int rows = mModel->rowCount();
// Table MUST BE 1-dimensional to match core QgsProcessingParameterMatrix expectations
QVariantList res;
res.reserve( cols * rows );
for ( int row = 0; row < rows; ++row )
{
for ( int col = 0; col < cols; ++col )
{
res << mModel->item( row, col )->text();
}
}
return res;
}

void QgsProcessingMatrixParameterDialog::addRow()
{
QList< QStandardItem * > items;
for ( int i = 0; i < mTblView->model()->columnCount(); ++i )
{
items << new QStandardItem( '0' );
}
mModel->appendRow( items );
}

void QgsProcessingMatrixParameterDialog::deleteRow()
{
QModelIndexList selected = mTblView->selectionModel()->selectedRows();
QSet< int > rows;
rows.reserve( selected.count() );
for ( const QModelIndex &i : selected )
rows << i.row();

QList< int > rowsToDelete = rows.toList();
std::sort( rowsToDelete.begin(), rowsToDelete.end(), std::greater<int>() );
mTblView->setUpdatesEnabled( false );
for ( int i : qgis::as_const( rowsToDelete ) )
mModel->removeRows( i, 1 );

mTblView->setUpdatesEnabled( true );
}

void QgsProcessingMatrixParameterDialog::deleteAllRows()
{
mModel->clear();
if ( mParam )
mModel->setHorizontalHeaderLabels( mParam->headers() );
}

void QgsProcessingMatrixParameterDialog::populateTable( const QVariantList &contents )
{
if ( !mParam )
return;

const int cols = mParam->headers().count();
const int rows = contents.length() / cols;
mModel = new QStandardItemModel( rows, cols, this );
mModel->setHorizontalHeaderLabels( mParam->headers() );

for ( int row = 0; row < rows; ++row )
{
for ( int col = 0; col < cols; ++col )
{
QStandardItem *item = new QStandardItem( contents.at( row * cols + col ).toString() );
mModel->setItem( row, col, item );
}
}
mTblView->setModel( mModel );
}

//
// QgsProcessingMatrixParameterPanel
//

QgsProcessingMatrixParameterPanel::QgsProcessingMatrixParameterPanel( QWidget *parent, const QgsProcessingParameterMatrix *param )
: QWidget( parent )
, mParam( param )
{
QHBoxLayout *hl = new QHBoxLayout();
hl->setMargin( 0 );
hl->setContentsMargins( 0, 0, 0, 0 );

mLineEdit = new QLineEdit();
mLineEdit->setEnabled( false );
hl->addWidget( mLineEdit, 1 );

mToolButton = new QToolButton();
mToolButton->setText( tr( "" ) );
hl->addWidget( mToolButton );

setLayout( hl );

if ( mParam )
{
for ( int row = 0; row < mParam->numberRows(); ++row )
{
for ( int col = 0; col < mParam->headers().count(); ++col )
{
mTable.append( '0' );
}
}
mLineEdit->setText( tr( "Fixed table (%1x%2)" ).arg( mParam->numberRows() ).arg( mParam->headers().count() ) );
}

connect( mToolButton, &QToolButton::clicked, this, &QgsProcessingMatrixParameterPanel::showDialog );
}

void QgsProcessingMatrixParameterPanel::setValue( const QVariantList &value )
{
mTable = value;
updateSummaryText();
emit changed();
}

void QgsProcessingMatrixParameterPanel::showDialog()
{
QgsProcessingMatrixParameterDialog dlg( this, nullptr, mParam, mTable );
if ( dlg.exec() )
{
setValue( dlg.table() );
}
}

void QgsProcessingMatrixParameterPanel::updateSummaryText()
{
if ( mParam )
mLineEdit->setText( tr( "Fixed table (%1x%2)" ).arg( mTable.count() / mParam->headers().count() ).arg( mParam->headers().count() ) );
}


///@endcond
115 changes: 115 additions & 0 deletions src/gui/processing/qgsprocessingmatrixparameterdialog.h
@@ -0,0 +1,115 @@
/***************************************************************************
qgsprocessingmatrixparameterdialog.h
----------------------------------
Date : February 2019
Copyright : (C) 2019 Nyall Dawson
Email : nyall dot dawson at gmail dot com
***************************************************************************
* *
* This program is free software; you can redistribute it and/or modify *
* it under the terms of the GNU General Public License as published by *
* the Free Software Foundation; either version 2 of the License, or *
* (at your option) any later version. *
* *
***************************************************************************/

#ifndef QGSPROCESSINGMATRIXPARAMETERDIALOG_H
#define QGSPROCESSINGMATRIXPARAMETERDIALOG_H

#include "qgis.h"
#include "qgis_gui.h"
#include "ui_qgsprocessingmatrixparameterdialogbase.h"
#include "qgsprocessingparameters.h"

#define SIP_NO_FILE

class QStandardItemModel;
class QToolButton;

///@cond NOT_STABLE

/**
* \ingroup gui
* \brief Dialog for configuration of a matrix (fixed table) parameter.
* \note Not stable API
* \since QGIS 3.6
*/
class GUI_EXPORT QgsProcessingMatrixParameterDialog : public QDialog, private Ui::QgsProcessingMatrixParameterDialogBase
{
Q_OBJECT

public:

/**
* Constructor for QgsProcessingMatrixParameterDialog.
*/
QgsProcessingMatrixParameterDialog( QWidget *parent SIP_TRANSFERTHIS = nullptr, Qt::WindowFlags flags = nullptr, const QgsProcessingParameterMatrix *param = nullptr,
const QVariantList &initialTable = QVariantList() );

/**
* Returns the table's contents as a 1 dimensional array.
*/
QVariantList table() const;

private slots:

void addRow();
void deleteRow();
void deleteAllRows();

private:

QPushButton *mButtonAdd = nullptr;
QPushButton *mButtonRemove = nullptr;
QPushButton *mButtonRemoveAll = nullptr;
const QgsProcessingParameterMatrix *mParam = nullptr;
QStandardItemModel *mModel = nullptr;

void populateTable( const QVariantList &contents );

friend class TestProcessingGui;
};


/**
* \ingroup gui
* \brief Widget for configuration of a matrix (fixed table) parameter.
* \note Not stable API
* \since QGIS 3.6
*/
class GUI_EXPORT QgsProcessingMatrixParameterPanel : public QWidget
{
Q_OBJECT

public:

QgsProcessingMatrixParameterPanel( QWidget *parent = nullptr, const QgsProcessingParameterMatrix *param = nullptr );

QVariantList value() const { return mTable; }

void setValue( const QVariantList &value );

signals:

void changed();

private slots:

void showDialog();

private:

void updateSummaryText();

const QgsProcessingParameterMatrix *mParam = nullptr;
QLineEdit *mLineEdit = nullptr;
QToolButton *mToolButton = nullptr;

QVariantList mTable;

friend class TestProcessingGui;
};

///@endcond

#endif // QGSPROCESSINGMATRIXPARAMETERDIALOG_H

0 comments on commit eeff02f

Please sign in to comment.