Skip to content
Permalink
Browse files

Fix addDbLayerAction insertion point when arcgis provider is enabled

  • Loading branch information
manisandro committed Jun 3, 2016
1 parent 8f15877 commit 04e1e96859ca6d66ccab5507d20eb37dd3795e7f
Showing with 3 additions and 1 deletion.
  1. +3 −1 src/app/qgisapp.cpp
@@ -2175,6 +2175,8 @@ void QgisApp::createToolBars()
mLayerToolBar->removeAction( mActionAddWfsLayer );
featureServiceAction->setObjectName( "ActionFeatureService" );
connect( bt, SIGNAL( triggered( QAction * ) ), this, SLOT( toolButtonActionTriggered( QAction * ) ) );
#else
QAction* mapServiceAction = mActionAddWmsLayer;
#endif

// add db layer button
@@ -2206,7 +2208,7 @@ void QgisApp::createToolBars()
}
if ( defAddDbLayerAction )
bt->setDefaultAction( defAddDbLayerAction );
QAction* addDbLayerAction = mLayerToolBar->insertWidget( mActionAddWmsLayer, bt );
QAction* addDbLayerAction = mLayerToolBar->insertWidget( mapServiceAction, bt );
addDbLayerAction->setObjectName( "ActionAddDbLayer" );
connect( bt, SIGNAL( triggered( QAction * ) ), this, SLOT( toolButtonActionTriggered( QAction * ) ) );

3 comments on commit 04e1e96

@nirvn

This comment has been minimized.

Copy link
Contributor

@nirvn nirvn replied Jun 3, 2016

@manisandro thanks.

@nirvn

This comment has been minimized.

Copy link
Contributor

@nirvn nirvn replied Jun 3, 2016

@manisandro , BTW, @nyalldawson recently renamed [ delete ] buttons to [ remove ] recently, in a bid to avoid confusion as to what the button does. Might be worth updating your dialogs too. Cheers.

@manisandro

This comment has been minimized.

Copy link
Member Author

@manisandro manisandro replied Jun 3, 2016

@nirvn Thanks for the heads up, done in 6a46e71

Please sign in to comment.
You can’t perform that action at this time.