Add checks to prevent null pointer on alterations in geneFilterQuery

cBioPortal · Oct 13, 2024 · 4967744 · 4967744
1 parent 0ac1039
commit 4967744
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/main/resources/org/cbioportal/persistence/mybatisclickhouse/StudyViewFilterMapper.xml b/src/main/resources/org/cbioportal/persistence/mybatisclickhouse/StudyViewFilterMapper.xml
@@ -116,9 +116,11 @@
 
                             <foreach item="geneFilterQuery" collection="geneFilterQueryList" open=" AND ((" separator=") OR (" close="))">
                                 hugo_gene_symbol = #{geneFilterQuery.hugoGeneSymbol}
-                                <foreach item="alteration" collection="geneFilterQuery.getAlterations()" open="AND (" separator=") OR (" close=")">
-                                   cna_alteration = #{alteration.code}
-                                </foreach>
+                                <if test="geneFilterQuery.getAlterations() != null">
+                                    <foreach item="alteration" collection="geneFilterQuery.getAlterations()" open="AND (" separator=") OR (" close=")">
+                                        cna_alteration = #{alteration.code}
+                                    </foreach>
+                                </if>
                             </foreach>
                         </where>
                     </foreach>