Skip to content

Commit

Permalink
Merge pull request #1668 from m-kuhn/action-menu-crash
Browse files Browse the repository at this point in the history
Alternative approach to f9fcee9
  • Loading branch information
jef-n committed Oct 29, 2014
2 parents 4cf08c5 + a7f889d commit cc5d3bc
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
13 changes: 7 additions & 6 deletions src/gui/qgsattributedialog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
QgsAttributeDialog::QgsAttributeDialog( QgsVectorLayer* vl, QgsFeature* thepFeature, bool featureOwner, QgsDistanceArea myDa, QWidget* parent, bool showDialogButtons )
: QDialog( parent )
, mHighlight( 0 )
, mOwnedFeature( featureOwner ? thepFeature : 0 )
{
QgsAttributeEditorContext context;
context.setDistanceArea( myDa );
Expand All @@ -37,17 +36,22 @@ QgsAttributeDialog::QgsAttributeDialog( QgsVectorLayer* vl, QgsFeature* thepFeat

if ( !showDialogButtons )
mAttributeForm->hideButtonBox();

if ( featureOwner )
delete thepFeature;
}

QgsAttributeDialog::QgsAttributeDialog( QgsVectorLayer* vl, QgsFeature* thepFeature, bool featureOwner, QWidget* parent, bool showDialogButtons, QgsAttributeEditorContext context )
: QDialog( parent )
, mHighlight( 0 )
, mOwnedFeature( featureOwner ? thepFeature : 0 )
{
init( vl, thepFeature, context, parent );

if ( !showDialogButtons )
mAttributeForm->hideButtonBox();

if ( featureOwner )
delete thepFeature;
}

QgsAttributeDialog::~QgsAttributeDialog()
Expand All @@ -58,9 +62,6 @@ QgsAttributeDialog::~QgsAttributeDialog()
delete mHighlight;
}

if( mOwnedFeature )
delete mOwnedFeature;

saveGeometry();
}

Expand Down Expand Up @@ -110,7 +111,7 @@ void QgsAttributeDialog::init( QgsVectorLayer* layer, QgsFeature* feature, QgsAt
connect( buttonBox, SIGNAL( accepted() ), this, SLOT( accept() ) );

mMenuBar = new QMenuBar( this );
QgsActionMenu* menu = new QgsActionMenu( layer, feature, this );
QgsActionMenu* menu = new QgsActionMenu( layer, &mAttributeForm->feature(), this );
mMenuBar->addMenu( menu );
layout()->setMenuBar( mMenuBar );

Expand Down
1 change: 0 additions & 1 deletion src/gui/qgsattributedialog.h
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,6 @@ class GUI_EXPORT QgsAttributeDialog : public QDialog
QString mReturnvarname;
QgsAttributeForm* mAttributeForm;
QMenuBar* mMenuBar;
QgsFeature *mOwnedFeature;

// true if this dialog is editable
bool mEditable;
Expand Down

0 comments on commit cc5d3bc

Please sign in to comment.