Skip to content

Commit

Permalink
avoid initially loading value map previews from the first layer (migh…
Browse files Browse the repository at this point in the history
…t be expensive)
  • Loading branch information
jef-n committed Jul 17, 2014
1 parent e696158 commit 4eddfba
Showing 1 changed file with 28 additions and 30 deletions.
58 changes: 28 additions & 30 deletions src/gui/qgsattributetypeloaddialog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -46,63 +46,69 @@ QgsAttributeTypeLoadDialog::QgsAttributeTypeLoadDialog( QgsVectorLayer *vl )
connect( previewButton, SIGNAL( pressed( ) ), this, SLOT( previewButtonPushed( ) ) );

fillLayerList();
}

keyComboBox->setDisabled( true );
valueComboBox->setDisabled( true );
}

QgsAttributeTypeLoadDialog::~QgsAttributeTypeLoadDialog()
{

}


void QgsAttributeTypeLoadDialog::setVectorLayer( QgsVectorLayer *layer )
{
mLayer = layer;
}



void QgsAttributeTypeLoadDialog::previewButtonPushed()
{
createPreview( valueComboBox->currentIndex(), true );
}

void QgsAttributeTypeLoadDialog::fillLayerList()
{
layerComboBox->blockSignals( true );
layerComboBox->clear();
foreach ( QgsMapLayer *l, QgsMapLayerRegistry::instance()->mapLayers() )
{
QgsVectorLayer *vl = qobject_cast< QgsVectorLayer * >( l );
if ( vl )
layerComboBox->addItem( vl->name(), vl->id() );
}
layerComboBox->setCurrentIndex( -1 );
layerComboBox->blockSignals( false );
}

void QgsAttributeTypeLoadDialog::fillComboBoxes( int layerIndex )
{
keyComboBox->blockSignals( true );
valueComboBox->blockSignals( true );

//clear comboboxes first
keyComboBox->clear();
valueComboBox->clear();

if ( layerIndex < 0 )
QgsVectorLayer *vLayer = qobject_cast<QgsVectorLayer *>( layerIndex < 0 ? 0 : QgsMapLayerRegistry::instance()->mapLayer( layerComboBox->itemData( layerIndex ).toString() ) );
if ( vLayer )
{
return;
QMap<QString, int> fieldMap = vLayer->dataProvider()->fieldNameMap();
QMap<QString, int>::iterator it = fieldMap.begin();
for ( ; it != fieldMap.end(); ++it )
{
keyComboBox->addItem( it.key(), it.value() );
valueComboBox->addItem( it.key(), it.value() );
}
}

QgsMapLayer* dataLayer = QgsMapLayerRegistry::instance()->mapLayer( layerComboBox->itemData( layerComboBox->currentIndex() ).toString() );
QgsVectorLayer* vLayer = qobject_cast<QgsVectorLayer *>( dataLayer );
if ( vLayer == NULL )
{
return;
}
QMap<QString, int> fieldMap = vLayer->dataProvider()->fieldNameMap();
QMap<QString, int>::iterator it = fieldMap.begin();
for ( ; it != fieldMap.end(); ++it )
{
keyComboBox->addItem( it.key(), it.value() );
valueComboBox->addItem( it.key(), it.value() );
}
keyComboBox->setEnabled( vLayer != 0 );
valueComboBox->setEnabled( vLayer != 0 );

keyComboBox->setCurrentIndex( -1 );
valueComboBox->setEnabled( -1 );

keyComboBox->blockSignals( false );
valueComboBox->blockSignals( false );
}

void QgsAttributeTypeLoadDialog::createPreview( int fieldIndex, bool full )
Expand All @@ -122,10 +128,8 @@ void QgsAttributeTypeLoadDialog::createPreview( int fieldIndex, bool full )
int idx2 = valueComboBox->itemData( valueComboBox->currentIndex() ).toInt();
QgsMapLayer* dataLayer = QgsMapLayerRegistry::instance()->mapLayer( layerComboBox->itemData( layerComboBox->currentIndex() ).toString() );
QgsVectorLayer* vLayer = qobject_cast<QgsVectorLayer *>( dataLayer );
if ( vLayer == NULL )
{
if ( !vLayer )
return;
}

QgsAttributeList attributeList = QgsAttributeList();
attributeList.append( idx );
Expand Down Expand Up @@ -154,7 +158,6 @@ void QgsAttributeTypeLoadDialog::createPreview( int fieldIndex, bool full )
previewTableWidget->setItem( row, 0, new QTableWidgetItem( mit.value().toString() ) );
previewTableWidget->setItem( row, 1, new QTableWidgetItem( mit.key() ) );
}

}

QMap<QString, QVariant> &QgsAttributeTypeLoadDialog::valueMap()
Expand All @@ -174,10 +177,8 @@ void QgsAttributeTypeLoadDialog::loadDataToValueMap()
int idx2 = valueComboBox->itemData( valueComboBox->currentIndex() ).toInt();
QgsMapLayer* dataLayer = QgsMapLayerRegistry::instance()->mapLayer( layerComboBox->itemData( layerComboBox->currentIndex() ).toString() );
QgsVectorLayer* vLayer = qobject_cast<QgsVectorLayer *>( dataLayer );
if ( vLayer == NULL )
{
if ( !vLayer )
return;
}

QgsAttributeList attributeList = QgsAttributeList();
attributeList.append( idx );
Expand All @@ -196,12 +197,9 @@ void QgsAttributeTypeLoadDialog::loadDataToValueMap()
}
}



void QgsAttributeTypeLoadDialog::accept()
{
//store data to output variable
loadDataToValueMap();
QDialog::accept();
}

}

0 comments on commit 4eddfba

Please sign in to comment.