fixes for some changes in SVN 6797 that caused NPEs when the bookmark…

…s initialized git-svn-id: https://svn.berlios.de/svnroot/repos/yacy/trunk@6800 6c8d7289-2bf4-0310-a012-ef5d649a1542
yacy · Apr 13, 2010 · 4917f96 · 4917f96
1 parent dff6604
commit 4917f96
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/source/de/anomic/data/BookmarkDate.java b/source/de/anomic/data/BookmarkDate.java
@@ -72,6 +72,7 @@ public void init(Iterator<Bookmark> it) {
         int count = 0;
         while (it.hasNext()) {
             bookmark=it.next();
+            if (bookmark == null) continue;
             date = String.valueOf(bookmark.getTimeStamp());
             bmDate=getDate(date);
             if(bmDate==null){

diff --git a/source/de/anomic/data/bookmarksDB.java b/source/de/anomic/data/bookmarksDB.java
@@ -102,6 +102,7 @@ public bookmarksDB(final File bookmarksFile, final File datesFile) throws IOExce
         String[] tagArray;
         while(it.hasNext()){
             bookmark = it.next();
+            if (bookmark == null) continue;
             tagArray = BookmarkHelper.cleanTagsString(bookmark.getTagsString() + bookmark.getFoldersString()).split(",");
             tag = null;
             for (final String element : tagArray) {
@@ -332,7 +333,7 @@ public String addBookmark(final Bookmark bookmark){
     public Bookmark getBookmark(final String urlHash){
         try {
             final Map<String, String> map = bookmarks.get(urlHash);
-            return (map == null) ? new Bookmark(map) : null;
+            return (map == null) ? null : new Bookmark(map);
         } catch (final IOException e) {
             return null;
         }
@@ -373,6 +374,7 @@ public Iterator<String> getBookmarksIterator(final boolean priv) {
         Bookmark bm;
         while(it.hasNext()){
             bm=it.next();
+            if (bm == null) continue;
             if(priv || bm.getPublic()){
             	set.add(bm.getUrlHash());
             }
@@ -866,17 +868,17 @@ public boolean hasNext() {
             try {
                 return this.bookmarkIter.hasNext();
             } catch (final kelondroException e) {
-                //resetDatabase();
+            	Log.logException(e);
                 return false;
             }
         }
 
         public Bookmark next() {
-            if (hasNext()) {
+        	try {
                 String s = new String(this.bookmarkIter.next());
                 return getBookmark(s);
-            } else {
-                //resetDatabase();
+        	} catch (final kelondroException e) {
+                Log.logException(e);
                 return null;
             }
         }