bookmarks:

-bugfixes
-deleting

git-svn-id: https://svn.berlios.de/svnroot/repos/yacy/trunk@1253 6c8d7289-2bf4-0310-a012-ef5d649a1542

htroot/Bookmarks_p.java

@@ -88,6 +88,10 @@ public class Bookmarks_p {
                 //ERROR
             }
         }
+        if(post.containsKey("delete")){
+            String urlHash=(String) post.get("delete");
+            switchboard.bookmarksDB.removeBookmark(urlHash);
+        }
         if(post.containsKey("tag")){
             tag=(String) post.get("tag");
         }
@@ -106,8 +110,13 @@ public class Bookmarks_p {
     boolean viewTag=false;
     if(!tag.equals("")){
         tagUrlHashes=switchboard.bookmarksDB.getTag(tag).getUrlHashes();
-        it=tagUrlHashes.iterator();
-        viewTag=true;
+        if(tagUrlHashes != null){
+            it=tagUrlHashes.iterator();
+            viewTag=true;
+        }else{ //tag does not exist
+            //TODO: Errorpage?
+            it=switchboard.bookmarksDB.bookmarkIterator(true);
+        }
     }else{
         it=switchboard.bookmarksDB.bookmarkIterator(true);
     }
@@ -129,6 +138,7 @@ public class Bookmarks_p {
             prop.put("bookmarks_"+count+"_link", bookmark.getUrl());
             prop.put("bookmarks_"+count+"_title", bookmark.getTitle());
             prop.put("bookmarks_"+count+"_tags", bookmark.getTags());
+            prop.put("bookmarks_"+count+"_hash", bookmark.getUrlHash());
             count++;
         }
     }

source/de/anomic/data/bookmarksDB.java

@@ -162,6 +162,15 @@ public class bookmarksDB {
         }
     }
     public void removeBookmark(String urlHash){
+        Bookmark bookmark = getBookmark(urlHash);
+        if(bookmark == null) return; //does not exist
+        String[] tags = bookmark.getTags().split(",");
+        bookmarksDB.Tag tag;
+        for(int i=0;i<tags.length;i++){
+            tag=getTag(tags[i]);
+            tag.delete(urlHash);
+            tag.setTagsTable();
+        }
         try {
             bookmarksTable.remove(urlHash);
         } catch (IOException e) {}
@@ -237,7 +246,7 @@ public class bookmarksDB {
             if(list.contains(urlHash)){
                 list.remove(urlHash);
             }
-            this.mem.put(URL_HASHES, list);
+            this.mem.put(URL_HASHES, vector2string(list));
         }
         public void setTagsTable(){
             try {