Author: sshafroi
Date: 2008-12-17 12:04:36 +0100 (Wed, 17 Dec 2008)
New Revision: 7107

Modified:
   
branches/2.18/result-spi/src/main/java/no/sesat/search/result/BasicNavigationItem.java
Log:
SEARCH-5292 - NullPointerException on 
BasicNavigationItem.getChildByTitle(BasicNavigationItem.java:75) in catalina.out

Protect against titles that are null


Modified: 
branches/2.18/result-spi/src/main/java/no/sesat/search/result/BasicNavigationItem.java
===================================================================
--- 
branches/2.18/result-spi/src/main/java/no/sesat/search/result/BasicNavigationItem.java
      2008-12-15 21:50:15 UTC (rev 7106)
+++ 
branches/2.18/result-spi/src/main/java/no/sesat/search/result/BasicNavigationItem.java
      2008-12-17 11:04:36 UTC (rev 7107)
@@ -1,4 +1,4 @@
-/* Copyright (2007) Schibsted Søk AS
+/* Copyright (2007-2008) Schibsted Søk AS
  *   This file is part of SESAT.
  *
  *   SESAT is free software: you can redistribute it and/or modify
@@ -71,9 +71,11 @@
     }
 
     public NavigationItem getChildByTitle(String title) {
-        for (NavigationItem navigationItem : getResults()) {
-            if (navigationItem.getTitle().equals(title)) {
-                return navigationItem;
+        if (title != null) {
+            for (NavigationItem navigationItem : getResults()) {
+                if (title.equals(navigationItem.getTitle())) {
+                    return navigationItem;
+                }
             }
         }
         return null;

_______________________________________________
Kernel-commits mailing list
[email protected]
http://sesat.no/mailman/listinfo/kernel-commits

Reply via email to