added ability to include or exclude switches from netlist analyzer

diylc/diylc-core/src/org/diylc/common/INetlistProcessor.java

@@ -35,9 +35,10 @@ public interface INetlistProcessor {
   /**
    * Finds all {@link Netlist}s for each switch position combination. 
    * 
+   * @param includeSwitches
    * @return
    */
-  List<Netlist> extractNetlists();
+  List<Netlist> extractNetlists(boolean includeSwitches);
   
   /**
    * Finds all available {@link INetlistAnalyzer} implementations.

diylc/diylc-core/src/org/diylc/presenter/Presenter.java

@@ -1906,7 +1906,7 @@ public class Presenter implements IPlugInPort {
   public void expandSelection(ExpansionMode expansionMode) {
     LOG.info(String.format("expandSelection(%s)", expansionMode));
     List<IDIYComponent<?>> newSelection = new ArrayList<IDIYComponent<?>>(this.selectedComponents);
-    List<Netlist> netlists = extractNetlists();
+    List<Netlist> netlists = extractNetlists(false);
     List<Set<IDIYComponent<?>>> allGroups = NetlistAnalyzer.extractComponentGroups(netlists);
     // Find control points of all selected components and all types
     Set<String> selectedNamePrefixes = new HashSet<String>();
@@ -2699,11 +2699,12 @@ public class Presenter implements IPlugInPort {
   
   @SuppressWarnings("unchecked")
   @Override
-  public List<Netlist> extractNetlists() {    
+  public List<Netlist> extractNetlists(boolean includeSwitches) {    
     Map<Netlist, Netlist> result = new HashMap<Netlist, Netlist>();
     List<Node> nodes = new ArrayList<Node>();
     
     List<ISwitch> switches = new ArrayList<ISwitch>();
+    
     for (IDIYComponent<?> c : currentProject.getComponents()) {
       ComponentType type =
           ComponentProcessor.getInstance().extractComponentTypeFrom((Class<? extends IDIYComponent<?>>) c.getClass());
@@ -2712,14 +2713,14 @@ public class Presenter implements IPlugInPort {
       if (!(c instanceof IContinuity)) {
         for (int i = 0; i < c.getControlPointCount(); i++) {
           String nodeName = c.getControlPointNodeName(i);
-          if (nodeName != null) {
+          if (nodeName != null && (!includeSwitches || !ISwitch.class.isAssignableFrom(type.getInstanceClass()))) {
             nodes.add(new Node(c, i));
           }
         }
       }
 
       // extract switches
-      if (ISwitch.class.isAssignableFrom(type.getInstanceClass()))
+      if (includeSwitches && ISwitch.class.isAssignableFrom(type.getInstanceClass()))
         switches.add((ISwitch) c);
     }   
     

diylc/diylc-library/src/org/diylc/components/AbstractLeadedComponent.java

@@ -338,7 +338,7 @@ public abstract class AbstractLeadedComponent<T> extends AbstractTransparentComp
       drawCenteredText(g2d, label, offset, 0, HorizontalAlignment.CENTER, VerticalAlignment.CENTER);
       g2d.setTransform(oldTransform);
     } else {
-      if (getLabelOriantation() == LabelOriantation.Horizontal) {
+      if (isStanding() || getLabelOriantation() == LabelOriantation.Horizontal) {
         g2d.setTransform(oldTransform);
         double x = (getNewPoints()[0].x + getNewPoints()[1].x - length) / 2.0;
         double y = (getNewPoints()[0].y + getNewPoints()[1].y - width) / 2.0;

diylc/diylc-library/src/org/diylc/components/electromechanical/MiniToggleSwitch.java

@@ -393,12 +393,12 @@ public class MiniToggleSwitch extends AbstractTransparentComponent<ToggleSwitchT
   }
   
   // switch stuff
-  
+//  
-  @Override
+//  @Override
-  public String getControlPointNodeName(int index) {
+//  public String getControlPointNodeName(int index) {
-    // we don't want the switch to produce any nodes, it just makes connections
+//    // we don't want the switch to produce any nodes, it just makes connections
-    return null;
+//    return null;
-  }
+//  }
   
   @Override
   public boolean canPointMoveFreely(int pointIndex) {

diylc/diylc-library/src/org/diylc/components/guitar/LPSwitch.java

@@ -328,12 +328,12 @@ public class LPSwitch extends AbstractTransparentComponent<String> implements IS
     body = null;
   }
   
-  @Override
+//  @Override
-  public String getControlPointNodeName(int index) {
+//  public String getControlPointNodeName(int index) {
-    // we don't want the switch to produce any nodes, it just makes connections
+//    // we don't want the switch to produce any nodes, it just makes connections
-    return null;
+//    return null;
-  }
+//  }
-  
+//  
   // switch stuff
 
   @Override

diylc/diylc-library/src/org/diylc/components/guitar/LeverSwitch.java

@@ -428,11 +428,11 @@ public class LeverSwitch extends AbstractTransparentComponent<String> implements
     }
   }
   
-  @Override
+//  @Override
-  public String getControlPointNodeName(int index) {
+//  public String getControlPointNodeName(int index) {
-    // we don't want the switch to produce any nodes, it just makes connections
+//    // we don't want the switch to produce any nodes, it just makes connections
-    return null;
+//    return null;
-  }
+//  }
   
   // switch stuff
 

diylc/diylc-swing/src/org/diylc/swing/ActionFactory.java

@@ -1667,7 +1667,7 @@ public class ActionFactory {
 
         @Override
         public List<Netlist> doInBackground() throws Exception {
-          return plugInPort.extractNetlists();
+          return plugInPort.extractNetlists(true);
         }
 
         @Override
@@ -1722,7 +1722,7 @@ public class ActionFactory {
 
         @Override
         public List<Summary> doInBackground() throws Exception {
-          List<Netlist> netlists = plugInPort.extractNetlists();
+          List<Netlist> netlists = plugInPort.extractNetlists(true);
           if (netlists == null || netlists.isEmpty()) {
             throw new Exception("The generated netlist is empty, nothing to show.");            
           }