Downloader Bug fix

UI Optimized
Scroller autohide, track disable

.idea/vcs.xml

@@ -2,6 +2,5 @@
 <project version="4">
   <component name="VcsDirectoryMappings">
     <mapping directory="" vcs="Git" />
-    <mapping directory="$PROJECT_DIR$/gh-pages" vcs="Git" />
   </component>
 </project>

app/build.gradle

@@ -21,7 +21,7 @@ android {
         minSdkVersion 16
         targetSdkVersion 30
         versionCode 59
-        versionName "5.0.1-hotfix2"
+        versionName "5.0.2"
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         vectorDrawables.useSupportLibrary = true
     }

app/src/main/java/xyz/quaver/pupil/Pupil.kt

@@ -71,11 +71,9 @@ val client: OkHttpClient
 
 class Pupil : Application() {
 
-    init {
-        AppCompatDelegate.setCompatVectorFromResourcesEnabled(true)
-    }
-
     override fun onCreate() {
+        AppCompatDelegate.setCompatVectorFromResourcesEnabled(true)
+
         preferences = PreferenceManager.getDefaultSharedPreferences(this)
 
         val userID = Preferences["user_id", ""].let {  userID ->

app/src/main/java/xyz/quaver/pupil/adapters/GalleryBlockAdapter.kt

@@ -20,6 +20,7 @@ package xyz.quaver.pupil.adapters
 
 import android.content.Context
 import android.graphics.drawable.Drawable
+import android.util.Log
 import android.util.SparseBooleanArray
 import android.view.LayoutInflater
 import android.view.View
@@ -125,6 +126,8 @@ class GalleryBlockAdapter(private val glide: RequestManager, private val galleri
         }
 
         fun bind(galleryID: Int) {
+            val time = System.currentTimeMillis()
+
             val cache = Cache.getInstance(view.context, galleryID)
 
             val galleryBlock = cache.metadata.galleryBlock ?: return
@@ -219,13 +222,15 @@ class GalleryBlockAdapter(private val glide: RequestManager, private val galleri
                 }
 
                 galleryblock_tag_group.removeAllViews()
+                CoroutineScope(Dispatchers.Default).launch {
                     galleryBlock.relatedTags.forEach {
-                    galleryblock_tag_group.addView(TagChip(context, Tag.parse(it)).apply {
+                        TagChip(context, Tag.parse(it)).apply {
                             setOnClickListener { view ->
                                 for (callback in onChipClickedHandler)
                                     callback.invoke((view as TagChip).tag)
                             }
-                    })
+                        }.let { launch(Dispatchers.Main) { galleryblock_tag_group.addView(it) } }
+                    }
                 }
 
                 galleryblock_id.text = galleryBlock.id.toString()
@@ -274,6 +279,7 @@ class GalleryBlockAdapter(private val glide: RequestManager, private val galleri
                     galleryblock_tag_group.visibility = View.GONE
                 }
             }
+            Log.i("PUPILD", "${System.currentTimeMillis() - time}")
         }
     }
     class NextViewHolder(view: LinearLayout) : RecyclerView.ViewHolder(view)

app/src/main/java/xyz/quaver/pupil/adapters/ReaderAdapter.kt

@@ -41,7 +41,6 @@ import xyz.quaver.hitomi.Reader
 import xyz.quaver.hitomi.getReferer
 import xyz.quaver.hitomi.imageUrlFromImage
 import xyz.quaver.hiyobi.createImgList
-import xyz.quaver.io.util.readBytes
 import xyz.quaver.pupil.BuildConfig
 import xyz.quaver.pupil.R
 import xyz.quaver.pupil.services.DownloadService
@@ -143,7 +142,7 @@ class ReaderAdapter(private val activity: ReaderActivity,
 
                 CoroutineScope(Dispatchers.IO).launch {
                     glide
-                        .load(image.readBytes())
+                        .load(image.uri)
                         .diskCacheStrategy(DiskCacheStrategy.NONE)
                         .skipMemoryCache(true)
                         .apply {

app/src/main/java/xyz/quaver/pupil/services/DownloadService.kt

@@ -309,11 +309,25 @@ class DownloadService : Service() {
 
         progress.put(galleryID, MutableList(reader.galleryInfo.files.size) { 0F })
 
-        cache.metadata.imageList?.forEachIndexed { index, image ->
+        cache.metadata.imageList?.let {
+            if (progress[galleryID]?.size != it.size) {
+                cache.metadata.imageList?.filterNotNull()?.forEach { file ->
+                    cache.findFile(file)?.delete()
+                }
+                cache.metadata.imageList = MutableList(reader.galleryInfo.files.size) { null }
+                return@let
+            }
+
+            it.forEachIndexed { index, image ->
                 progress[galleryID]?.set(index, if (image != null) Float.POSITIVE_INFINITY else 0F)
             }
+        }
 
         if (isCompleted(galleryID)) {
+            if (DownloadManager.getInstance(this@DownloadService)
+                    .getDownloadFolder(galleryID) != null)
+                Cache.getInstance(this@DownloadService, galleryID).moveToDownload()
+
             notificationManager.cancel(galleryID)
             startId?.let { stopSelf(it) }
             return@launch

app/src/main/java/xyz/quaver/pupil/ui/MainActivity.kt

@@ -26,6 +26,7 @@ import android.text.InputType
 import android.view.*
 import android.widget.*
 import androidx.appcompat.app.AlertDialog
+import androidx.appcompat.app.AppCompatDelegate
 import androidx.cardview.widget.CardView
 import androidx.core.view.GravityCompat
 import com.arlib.floatingsearchview.FloatingSearchView
@@ -97,7 +98,9 @@ class MainActivity :
 
 
     override fun onCreate(savedInstanceState: Bundle?) {
+        AppCompatDelegate.setCompatVectorFromResourcesEnabled(true)
         super.onCreate(savedInstanceState)
+        setContentView(R.layout.activity_main)
 
         if (intent.action == Intent.ACTION_VIEW) {
             intent.dataString?.let { url ->
@@ -111,8 +114,6 @@ class MainActivity :
             }
         }
 
-        setContentView(R.layout.activity_main)
-
         if (Preferences["download_folder", ""].isEmpty())
             DownloadLocationDialogFragment().show(supportFragmentManager, "Download Location Dialog")
 

app/src/main/java/xyz/quaver/pupil/util/downloader/Cache.kt

@@ -200,24 +200,42 @@ class Cache private constructor(context: Context, val galleryID: Int) : ContextW
     fun moveToDownload() = CoroutineScope(Dispatchers.IO).launch {
         val downloadFolder = downloadFolder ?: return@launch
 
+        if (downloadFolder.getChild(".metadata").exists())
+            return@launch
+
         metadata.imageList?.forEach { imageName ->
             imageName ?: return@forEach
             val target = downloadFolder.getChild(imageName)
             val source = cacheFolder.getChild(imageName)
 
-            if (!source.exists())
+            if (!source.exists() || target.exists())
                 return@forEach
 
             kotlin.runCatching {
                 target.createNewFile()
-                source.readBytes()?.let { target.writeBytes(it) }
+                target.outputStream()?.use { target -> source.inputStream()?.use { source ->
+                    source.copyTo(target)
+                } }
+            }
+        }
+
+        val cacheThumbnail = cacheFolder.getChild(".thumbnail")
+        val downloadThumbnail = downloadFolder.getChild(".thumbnail")
+
+        if (cacheThumbnail.exists() && !downloadThumbnail.exists()) {
+            kotlin.runCatching {
+                downloadThumbnail.createNewFile()
+                downloadThumbnail.outputStream()?.use { target -> cacheThumbnail.inputStream()?.use { source ->
+                    source.copyTo(target)
+                } }
+                cacheThumbnail.delete()
             }
         }
 
         val cacheMetadata = cacheFolder.getChild(".metadata")
         val downloadMetadata = downloadFolder.getChild(".metadata")
 
-        if (cacheMetadata.exists()) {
+        if (cacheMetadata.exists() && !downloadMetadata.exists()) {
             kotlin.runCatching {
                 downloadMetadata.createNewFile()
                 downloadMetadata.writeText(Json.encodeToString(metadata))

app/src/main/res/layout-v23/activity_main_content.xml

@@ -70,6 +70,8 @@
             app:handleHasFixedSize="true"
             app:handleHeight="72dp"
             app:handleWidth="24dp"
+            app:disableTrack="true"
+            app:hideHandleAfter="1000"
             app:trackMarginStart="64dp"
             app:addLastItemPadding="true"
             app:popupDrawable="@color/transparent">

app/src/main/res/layout/activity_main_content.xml

@@ -70,6 +70,8 @@
             app:handleHasFixedSize="true"
             app:handleHeight="72dp"
             app:handleWidth="24dp"
+            app:disableTrack="true"
+            app:hideHandleAfter="1000"
             app:trackMarginStart="64dp"
             app:addLastItemPadding="true"
             app:popupDrawable="@color/transparent">

app/src/main/res/layout/activity_reader.xml

@@ -33,6 +33,8 @@
         app:handleDrawable="@drawable/thumb"
         app:handleHeight="72dp"
         app:handleWidth="24dp"
+        app:disableTrack="true"
+        app:hideHandleAfter="1000"
         app:handleHasFixedSize="true"
         app:addLastItemPadding="true"
         app:popupDrawable="@color/transparent">

app/src/main/res/layout/item_galleryblock.xml

@@ -25,7 +25,8 @@
 	android:layout_height="wrap_content"
 	android:layout_margin="8dp"
 	app:cardCornerRadius="8dp"
-	android:clipChildren="true">
+	android:clipChildren="true"
+	tools:ignore="RtlHardcoded">
 
 	<com.daimajia.swipe.SwipeLayout
 		android:id="@+id/galleryblock_swipe_layout"
@@ -70,22 +71,13 @@
 
 		</LinearLayout>
 
-		<LinearLayout
-			android:id="@+id/galleryblock_primary"
-			android:layout_width="match_parent"
-			android:layout_height="wrap_content"
-			android:orientation="vertical"
-			android:foreground="?attr/selectableItemBackground"
-			android:focusable="true"
-			android:clickable="true"
-			tools:ignore="UnusedAttribute">
-
 		<androidx.constraintlayout.widget.ConstraintLayout
+			android:id="@+id/galleryblock_primary"
 			android:layout_width="match_parent"
 			android:layout_height="wrap_content">
 
-				<ProgressBar
+			<androidx.core.widget.ContentLoadingProgressBar
-					style="@style/Widget.AppCompat.ProgressBar.Horizontal"
+				style="?android:attr/progressBarStyleHorizontal"
 				android:id="@+id/galleryblock_progressbar"
 				android:layout_width="match_parent"
 				android:layout_height="4dp"
@@ -108,8 +100,7 @@
 				android:contentDescription="@string/galleryblock_thumbnail_description"
 				android:adjustViewBounds="true"
 				app:layout_constraintLeft_toLeftOf="parent"
-					app:layout_constraintTop_toBottomOf="@id/galleryblock_progressbar"
+				app:layout_constraintTop_toBottomOf="@id/galleryblock_progressbar"/>
-					app:layout_constraintBottom_toBottomOf="parent"/>
 
 			<TextView
 				style="@style/TextAppearance.AppCompat.Headline"
@@ -117,18 +108,16 @@
 				android:layout_width="0dp"
 				android:layout_height="wrap_content"
 				android:layout_marginTop="8dp"
-					android:layout_marginStart="8dp"
 				android:layout_marginLeft="8dp"
 				app:layout_constraintLeft_toRightOf="@id/galleryblock_thumbnail"
 				app:layout_constraintRight_toRightOf="parent"
-					app:layout_constraintTop_toTopOf="parent"/>
+				app:layout_constraintTop_toBottomOf="@id/galleryblock_progressbar"/>
 
 			<TextView
 				style="@style/TextAppearance.AppCompat.Medium"
 				android:id="@+id/galleryblock_artist"
 				android:layout_width="0dp"
 				android:layout_height="wrap_content"
-					android:layout_marginStart="8dp"
 				android:layout_marginLeft="8dp"
 				app:layout_constraintLeft_toRightOf="@id/galleryblock_thumbnail"
 				app:layout_constraintRight_toRightOf="parent"
@@ -138,37 +127,34 @@
 				android:id="@+id/galleryblock_series"
 				android:layout_width="0dp"
 				android:layout_height="wrap_content"
-					android:layout_marginStart="8dp"
 				android:layout_marginLeft="8dp"
 				app:layout_constraintTop_toBottomOf="@id/galleryblock_artist"
-					app:layout_constraintStart_toEndOf="@id/galleryblock_thumbnail"
+				app:layout_constraintLeft_toRightOf="@id/galleryblock_thumbnail"
-					app:layout_constraintEnd_toEndOf="parent"/>
+				app:layout_constraintRight_toRightOf="parent"/>
 
 			<TextView
 				android:id="@+id/galleryblock_type"
 				android:layout_width="wrap_content"
 				android:layout_height="wrap_content"
-					android:layout_marginStart="8dp"
 				android:layout_marginLeft="8dp"
 				app:layout_constraintTop_toBottomOf="@id/galleryblock_series"
-					app:layout_constraintStart_toEndOf="@id/galleryblock_thumbnail" />
+				app:layout_constraintLeft_toRightOf="@id/galleryblock_thumbnail" />
 
 			<TextView
 				android:id="@+id/galleryblock_language"
 				android:layout_width="wrap_content"
 				android:layout_height="wrap_content"
-					android:layout_marginStart="8dp"
 				android:layout_marginLeft="8dp"
 				android:layout_marginBottom="8dp"
 				app:layout_constraintTop_toBottomOf="@id/galleryblock_type"
 				app:layout_constraintBottom_toTopOf="@id/galleryblock_padding"
-					app:layout_constraintStart_toEndOf="@id/galleryblock_thumbnail" />
+				app:layout_constraintLeft_toRightOf="@id/galleryblock_thumbnail" />
 
 			<View
 				android:id="@+id/galleryblock_padding"
 				android:layout_width="0dp"
 				android:layout_height="0dp"
-					app:layout_constraintEnd_toEndOf="parent"
+				app:layout_constraintRight_toRightOf="parent"
 				app:layout_constraintTop_toBottomOf="@id/galleryblock_language"
 				app:layout_constraintBottom_toTopOf="@id/galleryblock_tag_group"/>
 
@@ -177,62 +163,64 @@
 				android:layout_width="0dp"
 				android:layout_height="wrap_content"
 				android:layout_marginLeft="8dp"
-					android:layout_marginStart="8dp"
 				android:layout_marginTop="16dp"
 				android:layout_marginBottom="16dp"
 				app:chipSpacing="4dp"
 				app:layout_constraintTop_toBottomOf="@id/galleryblock_padding"
 				app:layout_constraintLeft_toRightOf="@id/galleryblock_thumbnail"
-					app:layout_constraintRight_toRightOf="parent"
+				app:layout_constraintRight_toRightOf="parent"/>
-					app:layout_constraintBottom_toBottomOf="parent"/>
 
-			</androidx.constraintlayout.widget.ConstraintLayout>
+			<androidx.constraintlayout.widget.Barrier
+				android:id="@+id/barrier"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				app:barrierDirection="bottom"
+				app:constraint_referenced_ids="galleryblock_thumbnail,galleryblock_tag_group"/>
 
 			<View
+				android:id="@+id/divider"
 				android:layout_width="match_parent"
 				android:layout_height="1dp"
-				android:layout_margin="8dp"
+				android:background="@color/light_gray"
-				android:background="@android:color/darker_gray"/>
+				app:layout_constraintTop_toBottomOf="@id/barrier"
-
+				android:layout_margin="8dp"/>
-			<LinearLayout
-				android:layout_width="match_parent"
-				android:layout_height="wrap_content"
-				android:paddingLeft="8dp"
-				android:paddingRight="8dp"
-				android:paddingBottom="8dp"
-				android:orientation="horizontal"
-				android:gravity="center_vertical">
 
 			<TextView
 				android:id="@+id/galleryblock_id"
 				android:layout_width="wrap_content"
-					android:layout_height="wrap_content"/>
+				android:layout_height="wrap_content"
-
+				android:layout_marginLeft="8dp"
-				<View
+				android:layout_marginTop="8dp"
-					android:layout_width="0dp"
+				android:layout_marginBottom="8dp"
-					android:layout_height="1dp"
+				app:layout_constraintTop_toBottomOf="@id/divider"
-					android:layout_weight="1"/>
+				app:layout_constraintBottom_toBottomOf="parent"
+				app:layout_constraintLeft_toLeftOf="parent"/>
 
 			<TextView
 				android:id="@+id/galleryblock_pagecount"
 				android:layout_width="wrap_content"
-					android:layout_height="wrap_content"/>
+				android:layout_height="wrap_content"
-
+				android:layout_marginTop="8dp"
-				<View
+				android:layout_marginBottom="8dp"
-					android:layout_width="0dp"
+				app:layout_constraintTop_toBottomOf="@id/divider"
-					android:layout_height="1dp"
+				app:layout_constraintBottom_toBottomOf="parent"
-					android:layout_weight="1"/>
+				app:layout_constraintLeft_toRightOf="parent"
+				app:layout_constraintRight_toLeftOf="parent" />
 
 			<ImageView
 				android:id="@+id/galleryblock_favorite"
 				android:contentDescription="@string/app_name"
 				android:layout_width="32dp"
 				android:layout_height="32dp"
-					app:srcCompat="@drawable/ic_star_empty"/>
+				android:layout_marginRight="8dp"
+				android:layout_marginTop="8dp"
+				android:layout_marginBottom="8dp"
+				app:srcCompat="@drawable/ic_star_empty"
+				app:layout_constraintTop_toBottomOf="@id/divider"
+				app:layout_constraintBottom_toBottomOf="parent"
+				app:layout_constraintRight_toRightOf="parent" />
 
-			</LinearLayout>
+		</androidx.constraintlayout.widget.ConstraintLayout>
-
-		</LinearLayout>
 
 	</com.daimajia.swipe.SwipeLayout>