From f19fab5285c1e59e0e096668438bba7abf3666ce Mon Sep 17 00:00:00 2001
From: EveX1 <evekill@free.fr>
Date: Sat, 29 Jan 2022 02:07:12 +0100
Subject: [PATCH] UPDATE compressImage() - ADD Compressor Resolution Constraint
 as inner class - ADD two util (private) methods from same package - ADD logic
 to cut the width / height by half if image size > 2 * targeted size - UPDATE
 comment from #285

---
 .../smsmessenger/helpers/ImageCompressor.kt   | 69 +++++++++++++++++--
 1 file changed, 65 insertions(+), 4 deletions(-)

diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/ImageCompressor.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/ImageCompressor.kt
index 66ad7e7b..363c2960 100644
--- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/ImageCompressor.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/ImageCompressor.kt
@@ -38,9 +38,19 @@ class ImageCompressor(private val context: Context) {
                         val byteArray = contentResolver.openInputStream(uri)?.readBytes()!!
                         var destinationFile = File(outputDirectory, System.currentTimeMillis().toString().plus(mimeType.getExtensionFromMimeType()))
                         destinationFile.writeBytes(byteArray)
-                        val constraint = SizeConstraint(compressSize)
-                        while (constraint.isSatisfied(destinationFile).not()) {
-                            destinationFile = constraint.satisfy(destinationFile)
+                        val sizeConstraint = SizeConstraint(compressSize)
+                        val bitmap = loadBitmap(destinationFile)
+
+                        // if image weight > * 2 targeted size: cut down resolution by 2
+                        if (fileSize > 2 * compressSize) {
+                            val resConstraint = ResolutionConstraint(bitmap.width / 2, bitmap.height / 2)
+                            while (resConstraint.isSatisfied(destinationFile).not()) {
+                                destinationFile = resConstraint.satisfy(destinationFile)
+                            }
+                        }
+                        // do compression
+                        while (sizeConstraint.isSatisfied(destinationFile).not()) {
+                            destinationFile = sizeConstraint.satisfy(destinationFile)
                         }
                         callback.invoke(context.getMyFileUri(destinationFile))
                     } else {
@@ -106,7 +116,7 @@ class ImageCompressor(private val context: Context) {
         private var iteration: Int = 0
 
         fun isSatisfied(imageFile: File): Boolean {
-            // If size requirement is not met and iteration is maxed
+            // If size requirement is not met and maxIteration is reached
             if(iteration >= maxIteration && imageFile.length() >= maxFileSize) {
                 throw Exception("Unable to compress image to targeted size")
             }
@@ -120,4 +130,55 @@ class ImageCompressor(private val context: Context) {
         }
     }
 
+    private inner class ResolutionConstraint(private val width: Int, private val height: Int) {
+
+        private fun decodeSampledBitmapFromFile(imageFile: File, reqWidth: Int, reqHeight: Int): Bitmap {
+            return BitmapFactory.Options().run {
+                inJustDecodeBounds = true
+                BitmapFactory.decodeFile(imageFile.absolutePath, this)
+
+                inSampleSize = calculateInSampleSize(this, reqWidth, reqHeight)
+
+                inJustDecodeBounds = false
+                BitmapFactory.decodeFile(imageFile.absolutePath, this)
+            }
+        }
+
+        private fun calculateInSampleSize(options: BitmapFactory.Options, reqWidth: Int, reqHeight: Int): Int {
+            // Raw height and width of image
+            val (height: Int, width: Int) = options.run { outHeight to outWidth }
+            var inSampleSize = 1
+
+            if (height > reqHeight || width > reqWidth) {
+
+                val halfHeight: Int = height / 2
+                val halfWidth: Int = width / 2
+
+                // Calculate the largest inSampleSize value that is a power of 2 and keeps both
+                // height and width larger than the requested height and width.
+                while (halfHeight / inSampleSize >= reqHeight && halfWidth / inSampleSize >= reqWidth) {
+                    inSampleSize *= 2
+                }
+            }
+
+            return inSampleSize
+        }
+
+        fun isSatisfied(imageFile: File): Boolean {
+            return BitmapFactory.Options().run {
+                inJustDecodeBounds = true
+                BitmapFactory.decodeFile(imageFile.absolutePath, this)
+                calculateInSampleSize(this, width, height) <= 1
+            }
+        }
+
+        fun satisfy(imageFile: File): File {
+            return decodeSampledBitmapFromFile(imageFile, width, height).run {
+                determineImageRotation(imageFile, this).run {
+                    overWrite(imageFile, this)
+                }
+            }
+        }
+    }
+
 }