Fixes

Author: dcwhite

src/Interface/Application/NetworkEditor.h

@@ -413,6 +413,7 @@ namespace Gui {
     void setupPortHolder(const std::vector<SharedPointer<SCIRun::Dataflow::Networks::PortDescriptionInterface>>& ports, const QString& name,
       std::function<QPointF(const QRectF&)> position);
     PortRewiringMap portRewiringMap_;
+    QSet<QString> currentSubnetNames_;
 
     static NetworkEditor* inEditingContext_;
     struct InEditingContext

src/Interface/Application/Port.cc

@@ -482,7 +482,12 @@ void PortWidget::connectToSubnetPort(PortWidget* subnetPort)
   auto in = isInput_ ? this : subnetPort;
 
   ConnectionDescription cd { { out->moduleId_, out->portId_ }, { in->moduleId_, in->portId_ } };
-  connectionFactory_()->makeFinishedConnection(out, in, ConnectionId::create(cd));
+  if (connectionFactory_ && connectionFactory_())
+    connectionFactory_()->makeFinishedConnection(out, in, ConnectionId::create(cd));
+  else
+  {
+    qDebug() << "NO CONNECTION FACTORY AVAILABLE!!";
+  }
   //TODO: position provider needs adjustment
   //TODO: management of return value?
 }

src/Interface/Application/Subnetworks.cc

@@ -362,19 +362,25 @@ void NetworkEditor::makeSubnetwork()
     }
   }
 
-  if (underlyingModules.empty())
-  {
-    QMessageBox::information(this, "Make subnetwork", "Please select at least one module.");
-    return;
-  }
-
   bool ok;
   auto name = QInputDialog::getText(nullptr, "Make subnet", "Enter subnet name:", QLineEdit::Normal, "subnet", &ok);
   if (!ok || name.isEmpty())
   {
     QMessageBox::information(this, "Make subnetwork", "Invalid name.");
     return;
   }
+  
+  if (currentSubnetNames_.contains(name))
+  {
+    QMessageBox::information(this, "Make subnetwork", "A subnet by that name already exists.");
+    return;
+  }
+  
+  if (underlyingModules.empty())
+  {
+    QMessageBox::information(this, "Make subnetwork", "Please select at least one module.");
+    return;
+  }
 
   makeSubnetworkFromComponents(name, underlyingModules, includeConnections(items), rect);
 }
@@ -456,6 +462,8 @@ class SubnetModuleFactory : public Modules::Factory::HardCodedModuleFactory
 void NetworkEditor::makeSubnetworkFromComponents(const QString& name, const std::vector<ModuleHandle>& modules,
   QList<QGraphicsItem*> items, const QRectF& rect)
 {
+  currentSubnetNames_.insert(name);
+
   SubnetModuleFactory factory;
   auto subnetModule = factory.makeSubnet(name, modules, items);
   portRewiringMap_ = factory.getMap();
@@ -469,6 +477,7 @@ void NetworkEditor::makeSubnetworkFromComponents(const QString& name, const std:
   auto colorize = new QGraphicsDropShadowEffect;
   colorize->setColor(QColor(255,182,193,200));
   colorize->setOffset(8,8);
+  colorize->setBlurRadius(10);
   proxy->setGraphicsEffect(colorize);
 
   auto pic = grabSubnetPic(rect);
@@ -523,7 +532,7 @@ void NetworkEditor::makeSubnetworkFromComponents(const QString& name, const std:
   childrenNetworkItems_[name] = items;
 
   addSubnetChild(name, subnetModule);
-  qDebug() << "port repl map out of scope";
+  //qDebug() << "port repl map out of scope";
 }
 
 QPixmap NetworkEditor::grabSubnetPic(const QRectF& rect)