From 496774452c4fbe120d0e78e80f664929d14d2571 Mon Sep 17 00:00:00 2001
From: Charles Haynes <haynescd@chop.edu>
Date: Sat, 12 Oct 2024 22:40:21 -0400
Subject: [PATCH] Add checks to prevent null pointer on alterations in
 geneFilterQuery

---
 .../mybatisclickhouse/StudyViewFilterMapper.xml           | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/main/resources/org/cbioportal/persistence/mybatisclickhouse/StudyViewFilterMapper.xml b/src/main/resources/org/cbioportal/persistence/mybatisclickhouse/StudyViewFilterMapper.xml
index 4e4fb92d61b..95e66f20d3c 100644
--- 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>