Skip to content

Commit

Permalink
query builder: support null values in values lists
Browse files Browse the repository at this point in the history
  • Loading branch information
jef-n authored and alexbruy committed Dec 28, 2011
1 parent a13385e commit 9611ec3
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions src/app/qgsquerybuilder.cpp
Expand Up @@ -19,6 +19,7 @@
#include <QMessageBox>
#include <QRegExp>
#include <QPushButton>
#include <QSettings>
#include "qgsvectorlayer.h"
#include "qgsvectordataprovider.h"

Expand Down Expand Up @@ -100,12 +101,18 @@ void QgsQueryBuilder::fillValues( int idx, int limit )
QList<QVariant> values;
mLayer->uniqueValues( idx, values, limit );

QSettings settings;
QString nullValue = settings.value( "qgis/nullValue", "NULL" ).toString();

QgsDebugMsg( QString( "nullValue: %1" ).arg( nullValue ) );

for ( int i = 0; i < values.size(); i++ )
{
QStandardItem *myItem = new QStandardItem( values[i].toString() );
QStandardItem *myItem = new QStandardItem( values[i].isNull() ? nullValue : values[i].toString() );
myItem->setEditable( false );
myItem->setData( values[i] );
myItem->setData( values[i], Qt::UserRole + 1 );
mModelValues->insertRow( mModelValues->rowCount(), myItem );
QgsDebugMsg( QString( "Value is null: %1\nvalue: %2" ).arg( values[i].isNull() ).arg( values[i].isNull() ? nullValue : values[i].toString() ) );
}
}

Expand Down Expand Up @@ -269,9 +276,9 @@ void QgsQueryBuilder::on_lstFields_doubleClicked( const QModelIndex &index )
void QgsQueryBuilder::on_lstValues_doubleClicked( const QModelIndex &index )
{
QVariant value = mModelValues->data( index, Qt::UserRole + 1 );
if( value.isNull() )
if ( value.isNull() )
txtSQL->insertPlainText( "NULL" );
else if( value.type() == QVariant::Int || value.type() == QVariant::Double || value.type() == QVariant::LongLong )
else if ( value.type() == QVariant::Int || value.type() == QVariant::Double || value.type() == QVariant::LongLong )
txtSQL->insertPlainText( value.toString() );
else
txtSQL->insertPlainText( "'" + value.toString() + "'" );
Expand Down

0 comments on commit 9611ec3

Please sign in to comment.