Jim review modifications II

fixed missed compiler errors, and remove bad comment

Author: CodeByDrescher

vcell-cli/src/main/java/org/vcell/cli/run/results/SpatialResultsConverter.java

@@ -188,7 +188,9 @@ private static ValueHolder<LazySBMLSpatialDataAccessor> getSpatialValueHolderFor
         // get the list of variables associated with the data reference
         for (Variable var : dataGen.getVariables()) {
             // for each variable we recover the task
-            Task baseTask = sedml.findBaseTaskByAbstractTaskId(var.getTaskReference());
+            AbstractTask topLevelTask = sedml.findAbstractTaskById(var.getTaskReference());
+            if (null == topLevelTask) throw new RuntimeException("Task referenced by variable could not be found!");
+            Task baseTask = sedml.findBaseTaskByAbstractTaskId(topLevelTask.getId());
             if (baseTask == null) throw new RuntimeException("Unable to find task referenced by var: " + var.getId().string());
 
             // from the task we get the sbml model

vcell-core/src/main/java/org/vcell/sedml/SedMLImporter.java

@@ -322,7 +322,6 @@ private SedMLDataContainer verifyOrModifyOrRejectSedml(SedMLDataContainer provid
         Queue<RepeatedTask> badRepeatedTasks = repeatedTasks.stream().filter((task)->task.getSubTasks().size() > 1).collect(Collectors.toCollection(LinkedList::new));
         if (!badRepeatedTasks.isEmpty()){
             // Uh-oh, multi-subTasks detected!
-            // Short circuit here matters; if `condenseRedundantSubTasks == true`, then we can modify the sedml, and safely call the method to do so
             if (policy.multipleSubTaskPolicy != StrictnessPolicy.MultipleSubTaskPolicy.REJECT_IMMEDIATELY){
                 while (!badRepeatedTasks.isEmpty()){
                     boolean success = this.successfullyReducedRedundantSubTasks(badRepeatedTasks.poll());