Skip to content

Commit 063c7e6

Browse files
committed
Use the new expression builder for rule filter editing
1 parent a201ae9 commit 063c7e6

9 files changed

+19
-20
lines changed

src/app/qgsattributeactiondialog.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ void QgsAttributeActionDialog::insertExpression()
173173
dlg.setWindowTitle( tr( "Insert expression" ) );
174174
if ( dlg.exec() == QDialog::Accepted )
175175
{
176-
QString expression = dlg.expressionBuilder()->getExpressionString();
176+
QString expression = dlg.expressionBuilder()->expressionText();
177177
//Only add the expression if the user has entered some text.
178178
if ( !expression.isEmpty() )
179179
{

src/app/qgsfieldcalculator.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ QgsFieldCalculator::~QgsFieldCalculator()
6565
void QgsFieldCalculator::accept()
6666
{
6767

68-
QString calcString = builder->getExpressionString();
68+
QString calcString = builder->expressionText();
6969
QgsExpression exp( calcString );
7070

7171
if ( !mVectorLayer || !mVectorLayer->isEditable() )

src/app/qgslabelinggui.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -503,7 +503,7 @@ void QgsLabelingGui::showExpressionDialog()
503503
dlg.setWindowTitle( tr( "Expression based label" ) );
504504
if ( dlg.exec() == QDialog::Accepted )
505505
{
506-
QString expression = dlg.getExpressionText();
506+
QString expression = dlg.expressionText();
507507
//Only add the expression if the user has entered some text.
508508
if ( !expression.isEmpty() )
509509
{

src/gui/qgsexpressionbuilderdialog.cpp

+5-5
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ QgsExpressionBuilderDialog::QgsExpressionBuilderDialog( QgsVectorLayer* layer, Q
2525
connect( builder, SIGNAL( expressionParsed( bool ) ), okButuon, SLOT( setEnabled( bool ) ) );
2626

2727
builder->setLayer( layer );
28-
builder->setExpressionString( startText );
28+
builder->setExpressionText( startText );
2929
builder->loadFieldNames();
3030

3131
QSettings settings;
@@ -37,14 +37,14 @@ QgsExpressionBuilderWidget* QgsExpressionBuilderDialog::expressionBuilder()
3737
return builder;
3838
}
3939

40-
void QgsExpressionBuilderDialog::setExpressionText( QString text )
40+
void QgsExpressionBuilderDialog::setExpressionText( const QString& text )
4141
{
42-
builder->setExpressionString( text );
42+
builder->setExpressionText( text );
4343
}
4444

45-
QString QgsExpressionBuilderDialog::getExpressionText()
45+
QString QgsExpressionBuilderDialog::expressionText()
4646
{
47-
return builder->getExpressionString();
47+
return builder->expressionText();
4848
}
4949

5050
void QgsExpressionBuilderDialog::closeEvent( QCloseEvent *event )

src/gui/qgsexpressionbuilderdialog.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ class GUI_EXPORT QgsExpressionBuilderDialog : public QDialog, private Ui::QgsExp
3030
/** The builder widget that is used by the dialog */
3131
QgsExpressionBuilderWidget* expressionBuilder();
3232

33-
void setExpressionText( QString text );
33+
void setExpressionText( const QString& text );
3434

35-
QString getExpressionText();
35+
QString expressionText();
3636

3737
protected:
3838
/**

src/gui/qgsexpressionbuilderwidget.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -221,14 +221,14 @@ bool QgsExpressionBuilderWidget::isExpressionValid()
221221
return mExpressionValid;
222222
}
223223

224-
QString QgsExpressionBuilderWidget::getExpressionString()
224+
QString QgsExpressionBuilderWidget::expressionText()
225225
{
226226
return txtExpressionString->toPlainText();
227227
}
228228

229-
void QgsExpressionBuilderWidget::setExpressionString( const QString expressionString )
229+
void QgsExpressionBuilderWidget::setExpressionText( const QString& expression )
230230
{
231-
txtExpressionString->setPlainText( expressionString );
231+
txtExpressionString->setPlainText( expression );
232232
}
233233

234234
void QgsExpressionBuilderWidget::on_txtExpressionString_textChanged()

src/gui/qgsexpressionbuilderwidget.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -124,10 +124,10 @@ class GUI_EXPORT QgsExpressionBuilderWidget : public QWidget, private Ui::QgsExp
124124

125125
/** Gets the expression string that has been set in the expression area.
126126
* @returns The expression as a string. */
127-
QString getExpressionString();
127+
QString expressionText();
128128

129129
/** Sets the expression string for the widget */
130-
void setExpressionString( const QString expressionString );
130+
void setExpressionText( const QString& expression );
131131

132132
/** Registers a node item for the expression builder.
133133
* @param group The group the item will be show in the tree view. If the group doesn't exsit it will be created.

src/gui/symbology-ng/qgsrulebasedrendererv2widget.cpp

+3-4
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ void QgsRuleBasedRendererV2Widget::currentRuleChanged( const QModelIndex& curren
182182
#include "qgscategorizedsymbolrendererv2widget.h"
183183
#include "qgsgraduatedsymbolrendererv2.h"
184184
#include "qgsgraduatedsymbolrendererv2widget.h"
185-
#include "qgssearchquerybuilder.h"
185+
#include "qgsexpressionbuilderdialog.h"
186186
#include <QDialogButtonBox>
187187
#include <QInputDialog>
188188

@@ -416,11 +416,10 @@ QgsRendererRulePropsDialog::~QgsRendererRulePropsDialog()
416416

417417
void QgsRendererRulePropsDialog::buildExpression()
418418
{
419-
QgsSearchQueryBuilder dlg( mLayer, this );
420-
dlg.setSearchString( editFilter->text() );
419+
QgsExpressionBuilderDialog dlg( mLayer, editFilter->text(), this );
421420

422421
if ( dlg.exec() )
423-
editFilter->setText( dlg.searchString() );
422+
editFilter->setText( dlg.expressionText() );
424423
}
425424

426425
void QgsRendererRulePropsDialog::testFilter()

src/providers/wfs/qgswfssourceselect.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ void QgsWFSSourceSelect::on_treeWidget_itemDoubleClicked( QTreeWidgetItem* item,
434434

435435
if ( d.exec() == QDialog::Accepted )
436436
{
437-
item->setText( 4, w->getExpressionString() );
437+
item->setText( 4, w->expressionText() );
438438
}
439439
}
440440
}

0 commit comments

Comments
 (0)