Merge pull request #1621 from ghutchis/fix-start-up-crash

Fix crash at startup when opening a file from commandline

avogadro/qtgui/sceneplugin.h

@@ -77,13 +77,26 @@ class AVOGADROQTGUI_EXPORT ScenePlugin : public QObject
    */
   virtual void setEnabled(bool enable);
 
+  /**
+   * @return the widget for plugin settings (e.g., colors, widths, etc.)
+   */
   virtual QWidget* setupWidget();
+  /**
+   * This method exists to query without creating the widget.
+   * @return true if the plugin has a setup widget
+   */
+  virtual bool hasSetupWidget() const { return false; }
 
   /**
-  * Returns if this plugin should be considered in the default behavior,
-  * or it should reset to true or false.
-  */
-  enum DefaultBehavior { Ignore, False, True };
+   * Returns if this plugin should be considered in the default behavior,
+   * or it should reset to true or false.
+   */
+  enum DefaultBehavior
+  {
+    Ignore,
+    False,
+    True
+  };
   virtual DefaultBehavior defaultBehavior() const { return Ignore; }
 
 signals:

avogadro/qtgui/scenepluginmodel.cpp

@@ -86,7 +86,7 @@ QVariant ScenePluginModel::data(const QModelIndex& index_, int role) const
     switch (role) {
       case Qt::DisplayRole:
       case Qt::EditRole:
-        return (item->setupWidget() != nullptr) ? "•••" : " ";
+        return (item->hasSetupWidget()) ? "•••" : " ";
       case Qt::ToolTipRole:
       case Qt::WhatsThisRole:
         return tr("Settings");

avogadro/qtplugins/ballandstick/ballandstick.h

@@ -34,6 +34,7 @@ class BallAndStick : public QtGui::ScenePlugin
   }
 
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
   DefaultBehavior defaultBehavior() const override
   {

avogadro/qtplugins/cartoons/cartoons.h

@@ -27,14 +27,18 @@ class Cartoons : public QtGui::ScenePlugin
   void process(const QtGui::Molecule& molecule,
                Rendering::GroupNode& node) override;
 
-  QString name() const override { return tr("Cartoons", "protein ribbon / cartoon rendering"); }
+  QString name() const override
+  {
+    return tr("Cartoons", "protein ribbon / cartoon rendering");
+  }
 
   QString description() const override
   {
     return tr("Display of biomolecule ribbons / cartoons.");
   }
 
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
   DefaultBehavior defaultBehavior() const override
   {

avogadro/qtplugins/closecontacts/closecontacts.h

@@ -27,14 +27,18 @@ class CloseContacts : public QtGui::ScenePlugin
   void process(const QtGui::Molecule& molecule,
                Rendering::GroupNode& node) override;
 
-  QString name() const override { return tr("Close Contacts", "rendering of non-covalent close contacts"); }
+  QString name() const override
+  {
+    return tr("Close Contacts", "rendering of non-covalent close contacts");
+  }
 
   QString description() const override
   {
     return tr("Render close contacts between atoms.");
   }
-  
+
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
   DefaultBehavior defaultBehavior() const override
   {
@@ -47,10 +51,10 @@ public slots:
 
 private:
   std::string m_name = "Close Contacts";
-  
-  const std::array<QString, 3> INTERACTION_NAMES = {
-	tr("Contact"), tr("Salt Bridge"), tr("Repulsive")
-  };
+
+  const std::array<QString, 3> INTERACTION_NAMES = { tr("Contact"),
+                                                     tr("Salt Bridge"),
+                                                     tr("Repulsive") };
 
   std::array<double, 3> m_maximumDistances;
   std::array<Vector3ub, 3> m_lineColors;

avogadro/qtplugins/crystal/crystalscene.h

@@ -36,6 +36,7 @@ class CrystalScene : public QtGui::ScenePlugin
   }
 
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
 private slots:
   void setColor(const QColor& color);

avogadro/qtplugins/label/label.h

@@ -30,6 +30,8 @@ class Label : public QtGui::ScenePlugin
   }
 
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
+
   void process(const QtGui::Molecule& molecule,
                Rendering::GroupNode& node) override;
 

avogadro/qtplugins/meshes/meshes.h

@@ -31,6 +31,7 @@ class Meshes : public QtGui::ScenePlugin
   QString description() const override { return tr("Render polygon meshes."); }
 
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
   DefaultBehavior defaultBehavior() const override
   {

avogadro/qtplugins/noncovalent/noncovalent.h

@@ -33,8 +33,9 @@ class NonCovalent : public QtGui::ScenePlugin
   {
     return tr("Render a few non-covalent interactions.");
   }
-  
+
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
   DefaultBehavior defaultBehavior() const override
   {
@@ -48,11 +49,11 @@ public slots:
 
 private:
   const std::string m_name = "Non-Covalent";
-  
-  const std::array<QString, 3> INTERACTION_NAMES = {
-	tr("Hydrogen"), tr("Halogen"), tr("Chalcogen")
-  };
-  
+
+  const std::array<QString, 3> INTERACTION_NAMES = { tr("Hydrogen"),
+                                                     tr("Halogen"),
+                                                     tr("Chalcogen") };
+
   std::array<double, 3> m_angleTolerancesDegrees;
   std::array<double, 3> m_maximumDistances;
   std::array<Vector3ub, 3> m_lineColors;

avogadro/qtplugins/wireframe/wireframe.h

@@ -33,6 +33,7 @@ class Wireframe : public QtGui::ScenePlugin
   }
 
   QWidget* setupWidget() override;
+  bool hasSetupWidget() const override { return true; }
 
   DefaultBehavior defaultBehavior() const override
   {