Skip to content

Commit 35e3d2c

Browse files
committed
Remove TODOs
1 parent ec71a72 commit 35e3d2c

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

SequenceAnalysis/resources/schemas/sequenceanalysis.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -523,9 +523,8 @@
523523
<onClick>SequenceAnalysis.window.RunExportWindow.downloadFilesForAnalysis(dataRegionName);</onClick>
524524
</item>
525525
<item text="Alignment Reports">
526-
<!-- TODO: use analysisId-->
527526
<item text="Matching Haplotypes">
528-
<onClick>SequenceAnalysis.Buttons.viewQuery(dataRegionName, {queryName: 'haplotypeMatches'})</onClick>
527+
<onClick>SequenceAnalysis.Buttons.viewMatchingHaplotypes(dataRegionName)</onClick>
529528
</item>
530529
<item text="View Coverage">
531530
<onClick>SequenceAnalysis.Buttons.viewQuery(dataRegionName, {queryName: 'sequence_coverage'})</onClick>

SequenceAnalysis/resources/web/SequenceAnalysis/sequenceanalysisButtons.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -163,19 +163,24 @@ SequenceAnalysis.Buttons = new function(){
163163
);
164164
},
165165

166-
viewAlignmentsPivoted: function(dataRegionName){
166+
viewMatchingHaplotypes: function(dataRegionName){
167167
var dataRegion = LABKEY.DataRegions[dataRegionName];
168168
var checked = dataRegion.getChecked();
169169
if (!checked.length){
170170
alert('Must select one or more rows');
171171
return;
172172
}
173173

174+
if (checked.length !== 1) {
175+
alert('Only one row at a time can be selected');
176+
return;
177+
}
178+
174179
window.location = LABKEY.ActionURL.buildURL(
175180
'sequenceanalysis',
176-
'lineagePivot',
181+
'haplotypeMatches',
177182
dataRegion.containerPath,
178-
{analysisIds: checked.join(';')}
183+
{'query.param.AnalysisId': checked[0]}
179184
);
180185
},
181186

0 commit comments

Comments
 (0)