Skip to content

Commit d5cb20d

Browse files
committed
Bugfix to readset import when one pair is archived to SRA
1 parent 90dc3ff commit d5cb20d

File tree

2 files changed

+16
-6
lines changed

2 files changed

+16
-6
lines changed

SequenceAnalysis/src/org/labkey/sequenceanalysis/ReadDataImpl.java

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -201,18 +201,28 @@ public void setModifiedBy(Integer modifiedBy)
201201
_modifiedBy = modifiedBy;
202202
}
203203

204+
public File getFile1(boolean allowArchived)
205+
{
206+
return getFile(1, _fileId1, allowArchived);
207+
}
208+
204209
@Override
205210
@Transient
206211
public File getFile1()
207212
{
208-
return getFile(1, _fileId1);
213+
return getFile1(false);
209214
}
210215

211216
@Override
212217
@Transient
213218
public File getFile2()
214219
{
215-
return getFile(2, _fileId2);
220+
return getFile2(false);
221+
}
222+
223+
public File getFile2(boolean allowArchived)
224+
{
225+
return getFile(2, _fileId2, false);
216226
}
217227

218228
public void setFile(File f, int fileIdx)
@@ -248,9 +258,9 @@ public Integer getTotalReads()
248258
}
249259

250260
@Transient
251-
private File getFile(int fileIdx, Integer fileId)
261+
private File getFile(int fileIdx, Integer fileId, boolean allowArchived)
252262
{
253-
if (isArchived())
263+
if (isArchived() && !allowArchived)
254264
{
255265
return null;
256266
}

SequenceAnalysis/src/org/labkey/sequenceanalysis/pipeline/ReadsetInitTask.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -505,10 +505,10 @@ private void checkForDuplicateFileNames(List<SequenceReadsetImpl> readsets, List
505505
{
506506
Map<String, File> existingFileNames = new HashMap<>();
507507
preexistingReadData.forEach(rd -> {
508-
existingFileNames.put(rd.getFile1().getName(), rd.getFile1());
508+
existingFileNames.put(rd.getFile1(true).getName(), rd.getFile1(true));
509509
if (rd.getFile2() != null)
510510
{
511-
existingFileNames.put(rd.getFile2().getName(), rd.getFile2());
511+
existingFileNames.put(rd.getFile2(true).getName(), rd.getFile2(true));
512512
}
513513
});
514514

0 commit comments

Comments
 (0)