diff --git a/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoader.java b/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoader.java index 3e88ac38407f002331c2e580f3d5d81f2deed6dd..65338dfe0b47c84df4b4dbfa05292a22073329cd 100644 --- a/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoader.java +++ b/screening/source/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoader.java @@ -37,6 +37,8 @@ import ch.systemsx.cisd.openbis.generic.shared.basic.dto.ExternalData; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.ListMaterialCriteria; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.Material; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.TableModel; +import ch.systemsx.cisd.openbis.generic.shared.basic.dto.TableModelColumnHeader; +import ch.systemsx.cisd.openbis.generic.shared.basic.dto.TableModelRow; import ch.systemsx.cisd.openbis.generic.shared.dto.Session; import ch.systemsx.cisd.openbis.plugin.screening.server.IScreeningBusinessObjectFactory; import ch.systemsx.cisd.openbis.plugin.screening.server.dataaccess.IScreeningQuery; @@ -78,6 +80,9 @@ public class ExperimentFeatureVectorSummaryLoader extends AbstractContentLoader { List<ExternalData> matchingDataSets = getMatchingDataSets(experimentId, analysisProcedureCriteria); + TableModel tabelModel = + new TableModel(Collections.<TableModelColumnHeader> emptyList(), + Collections.<TableModelRow> emptyList()); if (matchingDataSets.size() == 1) { ExternalData ds = matchingDataSets.get(0); @@ -89,12 +94,9 @@ public class ExperimentFeatureVectorSummaryLoader extends AbstractContentLoader IDataSetTable dataSetTable = businessObjectFactory.createDataSetTable(session); try { - TableModel tabelModel = - dataSetTable.createReportFromDatasets(reportingPluginKey, dataStore, - codes); - return new ExperimentFeatureVectorSummary(experiment, - Collections.<MaterialFeatureVectorSummary> emptyList(), - Collections.<CodeAndLabel> emptyList(), tabelModel); + tabelModel = + dataSetTable.createReportFromDatasets(reportingPluginKey, + dataStore, codes); } catch (UserFailureException ex) { String message = ex.getMessage(); @@ -112,6 +114,9 @@ public class ExperimentFeatureVectorSummaryLoader extends AbstractContentLoader } } } + return new ExperimentFeatureVectorSummary(experiment, + Collections.<MaterialFeatureVectorSummary> emptyList(), + Collections.<CodeAndLabel> emptyList(), tabelModel); } return calculatedSummary(experimentId, analysisProcedureCriteria, experiment); diff --git a/screening/sourceTest/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoaderTest.java b/screening/sourceTest/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoaderTest.java index d7906782b9a0a408ad5f4d9b712a271b384b1e79..8b00bb59509df7b495269e96e17094bef4e1e551 100644 --- a/screening/sourceTest/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoaderTest.java +++ b/screening/sourceTest/java/ch/systemsx/cisd/openbis/plugin/screening/server/logic/ExperimentFeatureVectorSummaryLoaderTest.java @@ -222,7 +222,7 @@ public class ExperimentFeatureVectorSummaryLoaderTest extends AbstractServerTest AnalysisProcedureCriteria.createFromCode("extern"), new AnalysisSettings(properties)); - assertEquals(null, summary); + assertEquals(0, summary.getTableModelOrNull().getRows().size()); context.assertIsSatisfied(); } @@ -244,7 +244,7 @@ public class ExperimentFeatureVectorSummaryLoaderTest extends AbstractServerTest AnalysisProcedureCriteria.createFromCode("intern"), new AnalysisSettings(properties)); - assertEquals(null, summary); + assertEquals(0, summary.getTableModelOrNull().getRows().size()); context.assertIsSatisfied(); }