From b6e40c7a845f7c66d384877bde89af86505a35b0 Mon Sep 17 00:00:00 2001
From: Nikolay Pultsin <geometer@fbreader.org>
Date: Thu, 1 Oct 2015 02:00:35 +0100
Subject: [PATCH] code simplification (sync with premium version)

---
 AndroidManifest.xml                           |  7 +++++-
 .../fbreader/library/BookInfoActivity.java    | 25 ++-----------------
 2 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index fa7799f48..4ecd902af 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -214,7 +214,12 @@
 				<action android:name="android.fbreader.action.sync.QUICK_SYNC"/>
 			</intent-filter>
 		</service>
-		<activity android:name="org.geometerplus.android.fbreader.library.BookInfoActivity" android:theme="@style/FBReader.Activity" android:process=":library" android:configChanges="orientation|keyboardHidden|screenSize">
+		<activity
+			android:name="org.geometerplus.android.fbreader.library.BookInfoActivity"
+			android:theme="@style/FBReader.Activity"
+			android:process=":library"
+			android:configChanges="orientation|keyboardHidden|screenSize"
+		>
 			<intent-filter>
 				<action android:name="android.fbreader.action.BOOK_INFO"/>
 				<category android:name="android.intent.category.DEFAULT"/>
diff --git a/fbreader/app/src/main/java/org/geometerplus/android/fbreader/library/BookInfoActivity.java b/fbreader/app/src/main/java/org/geometerplus/android/fbreader/library/BookInfoActivity.java
index 9e3f4632c..6b742f0fb 100644
--- a/fbreader/app/src/main/java/org/geometerplus/android/fbreader/library/BookInfoActivity.java
+++ b/fbreader/app/src/main/java/org/geometerplus/android/fbreader/library/BookInfoActivity.java
@@ -241,16 +241,7 @@ public class BookInfoActivity extends Activity implements IBookCollection.Listen
 		((TextView)findViewById(R.id.book_info_title)).setText(myResource.getResource("bookInfo").getValue());
 
 		setupInfoPair(R.id.book_title, "title", book.getTitle());
-
-		final StringBuilder buffer = new StringBuilder();
-		final List<Author> authors = book.authors();
-		for (Author a : authors) {
-			if (buffer.length() > 0) {
-				buffer.append(", ");
-			}
-			buffer.append(a.DisplayName);
-		}
-		setupInfoPair(R.id.book_authors, "authors", buffer, authors.size());
+		setupInfoPair(R.id.book_authors, "authors", book.authorsString(", "), book.authors().size());
 
 		final SeriesInfo series = book.getSeriesInfo();
 		setupInfoPair(R.id.book_series, "series", series == null ? null : series.Series.getTitle());
@@ -259,19 +250,7 @@ public class BookInfoActivity extends Activity implements IBookCollection.Listen
 			seriesIndexString = series.Index.toPlainString();
 		}
 		setupInfoPair(R.id.book_series_index, "indexInSeries", seriesIndexString);
-
-		buffer.delete(0, buffer.length());
-		final HashSet<String> tagNames = new HashSet<String>();
-		for (Tag tag : book.tags()) {
-			if (!tagNames.contains(tag.Name)) {
-				if (buffer.length() > 0) {
-					buffer.append(", ");
-				}
-				buffer.append(tag.Name);
-				tagNames.add(tag.Name);
-			}
-		}
-		setupInfoPair(R.id.book_tags, "tags", buffer, tagNames.size());
+		setupInfoPair(R.id.book_tags, "tags", book.tagsString(", "), book.tags().size());
 		String language = book.getLanguage();
 		if (!ZLLanguageUtil.languageCodes().contains(language)) {
 			language = Language.OTHER_CODE;