Skip to content
Snippets Groups Projects
Commit aebb1bb7 authored by buczekp's avatar buczekp
Browse files

[LMS-1189] simplified code

SVN: 12705
parent 2888831a
No related branches found
Tags 0.2.3
No related merge requests found
...@@ -80,8 +80,6 @@ public class SecondaryEntityDAO ...@@ -80,8 +80,6 @@ public class SecondaryEntityDAO
return new SecondaryEntityDAO(query, daoFactory.getHomeDatabaseInstance()); return new SecondaryEntityDAO(query, daoFactory.getHomeDatabaseInstance());
} }
private final ISecondaryEntitySetListingQuery setQuery;
private final ISecondaryEntityListingQuery query; private final ISecondaryEntityListingQuery query;
private final DatabaseInstance databaseInstance; private final DatabaseInstance databaseInstance;
...@@ -90,28 +88,9 @@ public class SecondaryEntityDAO ...@@ -90,28 +88,9 @@ public class SecondaryEntityDAO
final DatabaseInstancePE databaseInstancePE) final DatabaseInstancePE databaseInstancePE)
{ {
this.query = query; this.query = query;
this.setQuery = createSetQuery(query, databaseInstancePE.getId());
this.databaseInstance = DatabaseInstanceTranslator.translate(databaseInstancePE); this.databaseInstance = DatabaseInstanceTranslator.translate(databaseInstancePE);
} }
private static ISecondaryEntitySetListingQuery createSetQuery(
ISecondaryEntityListingQuery query, long databaseInstanceId)
{
return asDatasetSetListingQuery(query);
}
private static ISecondaryEntitySetListingQuery asDatasetSetListingQuery(
final ISecondaryEntityListingQuery query)
{
return new ISecondaryEntitySetListingQuery()
{
public Iterable<SampleReferenceRecord> getSamples(LongSet sampleIds)
{
return query.getSamples(sampleIds);
}
};
}
public Experiment getExperiment(final long experimentId) public Experiment getExperiment(final long experimentId)
{ {
final ExperimentProjectGroupCodeRecord record = final ExperimentProjectGroupCodeRecord record =
...@@ -171,7 +150,7 @@ public class SecondaryEntityDAO ...@@ -171,7 +150,7 @@ public class SecondaryEntityDAO
public Long2ObjectMap<Sample> getSamples(LongSet sampleIds) public Long2ObjectMap<Sample> getSamples(LongSet sampleIds)
{ {
final Iterable<SampleReferenceRecord> sampleRecords = setQuery.getSamples(sampleIds); final Iterable<SampleReferenceRecord> sampleRecords = query.getSamples(sampleIds);
Long2ObjectMap<Sample> result = new Long2ObjectOpenHashMap<Sample>(); Long2ObjectMap<Sample> result = new Long2ObjectOpenHashMap<Sample>();
for (SampleReferenceRecord record : sampleRecords) for (SampleReferenceRecord record : sampleRecords)
{ {
...@@ -191,6 +170,7 @@ public class SecondaryEntityDAO ...@@ -191,6 +170,7 @@ public class SecondaryEntityDAO
sample.setGroup(tryCreateGroup(record.g_code, databaseInstance)); sample.setGroup(tryCreateGroup(record.g_code, databaseInstance));
sample.setDatabaseInstance(tryGetDatabaseInstance(record.g_code, databaseInstance)); sample.setDatabaseInstance(tryGetDatabaseInstance(record.g_code, databaseInstance));
sample.setPermId(escapeHtml(record.perm_id)); sample.setPermId(escapeHtml(record.perm_id));
// TODO 2009-09-25, Piotr Buczek: use container code
sample.setIdentifier(escapeHtml(createIdentifier(sample).toString())); sample.setIdentifier(escapeHtml(createIdentifier(sample).toString()));
return sample; return sample;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment