more onBookEvent implementations + updated TODO

TODO.libraryService

@@ -4,15 +4,12 @@ DONE summaries for libraryTrees
 DONE progress indicator in library activity
 DONE book search
 DONE books for series _and_ author
+* update book collection on path changing (from options dialog)
 
 new:
 * filesystem event listener
-* update book collection on path changing (from options dialog)
 DONE read book => go to library => go to book info => update book info => return to reading mode: book info is not updated
 * text search => found multiple occurencies => turn page right => right button is disabled
-* authors/books by author request
-* tags/books by tag request
-* series/books by series request
 * 'rescan' button in library menu
 * getHelpFile => getHelpBook
 * move BookCollection to libraryService package; change access rights to package local
@@ -27,6 +24,7 @@ small features:
 large features:
 * two page view
 * bookmarks highlighting
+* https://en.wikipedia.org/wiki/Alphabetical_order
 
 plugin:
 * bookmarks import/export

src/org/geometerplus/fbreader/library/AuthorListTree.java

@@ -46,7 +46,9 @@ public void waitForOpening() {
 	public boolean onBookEvent(BookEvent event, Book book) {
 		switch (event) {
 			case Added:
+			case Updated:
 			{
+				// TODO: remove empty authors tree after update (?)
 				final List<Author> bookAuthors = book.authors();
 				boolean changed = false;
 				if (bookAuthors.isEmpty()) {
@@ -57,11 +59,9 @@ public boolean onBookEvent(BookEvent event, Book book) {
 				return changed;
 			}
 			case Removed:
-				// TODO: implement
+				// TODO: remove empty authors tree (?)
 				return false;
 			default:
-			case Updated:
-				// TODO: implement
 				return false;
 		}
 	}

src/org/geometerplus/fbreader/library/SeriesListTree.java

@@ -55,16 +55,16 @@ public void waitForOpening() {
 	public boolean onBookEvent(BookEvent event, Book book) {
 		switch (event) {
 			case Added:
+			case Updated:
 			{
+				// TODO: remove empty series tree after update (?)
 				final SeriesInfo info = book.getSeriesInfo();
 				return info != null && createSeriesSubTree(info.Title);
 			}
 			case Removed:
-				// TODO: implement
+				// TODO: remove empty series tree (?)
 				return false;
 			default:
-			case Updated:
-				// TODO: implement
 				return false;
 		}
 	}

src/org/geometerplus/fbreader/library/TagListTree.java

@@ -48,7 +48,9 @@ public void waitForOpening() {
 	public boolean onBookEvent(BookEvent event, Book book) {
 		switch (event) {
 			case Added:
+			case Updated:
 			{
+				// TODO: remove empty tag trees after update (?)
 				final List<Tag> bookTags = book.tags();
 				boolean changed = false;
 				if (bookTags.isEmpty()) {
@@ -61,11 +63,9 @@ public boolean onBookEvent(BookEvent event, Book book) {
 				return changed;
 			}
 			case Removed:
-				// TODO: implement
+				// TODO: remove empty tag trees (?)
 				return false;
 			default:
-			case Updated:
-				// TODO: implement
 				return false;
 		}
 	}

src/org/geometerplus/fbreader/library/TagTree.java

@@ -116,9 +116,26 @@ public boolean onBookEvent(BookEvent event, Book book) {
 				return changed;
 			}
 			case Removed:
-				// TODO: implement
+				// TODO: remove empty tag trees (?)
+				return super.onBookEvent(event, book);
 			case Updated:
-				// TODO: implement
+			{
+				// TODO: remove empty tag trees (?)
+				boolean changed = removeBook(book);
+				final List<Tag> bookTags = book.tags();
+				if (bookTags.isEmpty()) {
+					changed &= Tag.NULL.equals(Tag) && createBookWithAuthorsSubTree(book);
+				} else {
+					for (Tag t : bookTags) {
+						if (Tag.equals(t)) {
+							changed &= createBookWithAuthorsSubTree(book);
+						} else if (Tag.equals(t.Parent)) {
+							changed &= createTagSubTree(t);
+						}
+					}
+				}
+				return changed;
+			}
 			default:
 				return super.onBookEvent(event, book);
 		}

src/org/geometerplus/fbreader/library/TitleListTree.java

@@ -74,12 +74,23 @@ public boolean onBookEvent(BookEvent event, Book book) {
 					return createBookWithAuthorsSubTree(book);
 				}
 			case Removed:
-				// TODO: implement
-				return false;
+				if (myDoGroupByFirstLetter) {
+					// TODO: remove old tree (?)
+					return false;
+				} else {
+					return super.onBookEvent(event, book);
+				}
 			default:
 			case Updated:
-				// TODO: implement
-				return false;
+				if (myDoGroupByFirstLetter) {
+					// TODO: remove old tree (?)
+					final String letter = TitleUtil.firstTitleLetter(book);
+					return letter != null && createTitleSubTree(letter);
+				} else {
+					boolean changed = removeBook(book);
+					changed |= containsBook(book) && createBookWithAuthorsSubTree(book);
+					return changed;
+				}
 		}
 	}