Merge pull request #4956 from jgrocha/profilemenu

elpaso · web-flow · commit 34ef954edcc2 · 2017-08-04T21:32:44.000+02:00
Fix #16970 (related to PR#4184)
diff --git a/src/app/qgisapp.cpp b/src/app/qgisapp.cpp
@@ -2298,49 +2298,43 @@ void QgisApp::refreshProfileMenu()
   QString activeName = profile->name();
   mConfigMenu->setTitle( tr( "&User Profiles" ) );
 
-  mConfigMenu->addAction( new QgsMenuHeaderWidgetAction( tr( "Active Profile" ), mConfigMenu ) );
-
-  mConfigMenu->addAction( new QgsMenuHeaderWidgetAction( tr( "Profiles" ), mConfigMenu ) );
-  QAction *profileSection = mConfigMenu->actions().at( 1 );
-
-  mConfigMenu->addAction( new QgsMenuHeaderWidgetAction( tr( "Config" ), mConfigMenu ) );
-  QAction *configSection = mConfigMenu->actions().at( 2 );
-
-  QAction *openProfileFolderAction = mConfigMenu->addAction( tr( "Open active profile folder" ) );
-  connect( openProfileFolderAction, &QAction::triggered, this, [this]()
-  {
-    QDesktopServices::openUrl( QUrl::fromLocalFile( userProfileManager()->userProfile()->folder() ) );
-  } );
-
-  QAction *newProfileAction = mConfigMenu->addAction( tr( "New profile" ) );
-  connect( newProfileAction, &QAction::triggered, this, &QgisApp::newProfile );
+  QActionGroup *profileGroup = new QActionGroup( this );
+  profileGroup->setExclusive( true );
 
   Q_FOREACH ( const QString &name, userProfileManager()->allProfiles() )
   {
     profile = userProfileManager()->profileForName( name );
     // Qt 5.5 has no parent default as nullptr
     QAction *action = new QAction( profile->icon(), profile->alias(), nullptr );
     action->setToolTip( profile->folder() );
+    action->setCheckable( true );
+    profileGroup->addAction( action );
+    mConfigMenu->addAction( action );
     delete profile;
 
     if ( name == activeName )
     {
-      mConfigMenu->insertAction( profileSection, action );
+      action->setChecked( true );
     }
     else
     {
-      mConfigMenu->insertAction( configSection, action );
+      connect( action, &QAction::triggered, this, [this, name]()
+      {
+        userProfileManager()->loadUserProfile( name );
+      } );
     }
-    connect( action, &QAction::triggered, this, [this, name]()
-    {
-      userProfileManager()->loadUserProfile( name );
-    } );
   }
 
-  if ( userProfileManager()->allProfiles().count() == 1 )
+  mConfigMenu->addSeparator( );
+
+  QAction *openProfileFolderAction = mConfigMenu->addAction( tr( "Open active profile folder" ) );
+  connect( openProfileFolderAction, &QAction::triggered, this, [this]()
   {
-    profileSection->setVisible( false );
-  }
+    QDesktopServices::openUrl( QUrl::fromLocalFile( userProfileManager()->userProfile()->folder() ) );
+  } );
+
+  QAction *newProfileAction = mConfigMenu->addAction( tr( "New profile" ) );
+  connect( newProfileAction, &QAction::triggered, this, &QgisApp::newProfile );
 }
 
 void QgisApp::createProfileMenu()
