Changeset 31686

Timestamp:

2017-05-18T10:39:22+12:00 (7 years ago)

Author:

kjdon

Message:

diego was sometimes getting arraylist outof bounds exception - we are getting an arraylist and then getting the first element without checking it at all. So added in a check for size > 0. not sure why its empty, is it a symptom of something else going wrong? don't have time to look at it now, and anyway, I want to rewrite query term highlighting one day.

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/action/DocumentAction.java (modified) (2 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/action/DocumentAction.java

@@ -1193 +1193 @@
             {
                 ArrayList phrase_query_p_term_variants_list = phrase_query_term_variants_hierarchy.get(p);
+                if (phrase_query_p_term_variants_list.size()>0) {
                 HashSet phrase_query_p_term_1_variants = (HashSet) phrase_query_p_term_variants_list.get(0);
                 if (phrase_query_p_term_1_variants.contains(word_match.word))
@@ -1208 +1209 @@
                     }
                 }
+                }
             }
         }